Remove unnecessary valueExists calls

Remove unnecessary valueExists calls in OS Settings as they are always defined.

Change-Id: I916c2194a02ce8aef997d9fb62752c20da8f1924
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1636996Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org>
Cr-Commit-Position: refs/heads/master@{#664971}

chrome/browser/resources/settings/chromeos/os_settings_ui/os_settings_ui.js

@@ -162,19 +162,13 @@ Polymer({
       vpnNameTemplate: loadTimeData.getString('vpnNameTemplate'),
     };
 
-    // TODO(crbug.com/961514) remove unnecessary valueExists() calls.
-    this.showApps_ = loadTimeData.valueExists('showApps') &&
-        loadTimeData.getBoolean('showApps');
-    this.showAndroidApps_ = loadTimeData.valueExists('androidAppsVisible') &&
-        loadTimeData.getBoolean('androidAppsVisible');
+    this.showApps_ = loadTimeData.getBoolean('showApps');
+    this.showAndroidApps_ = loadTimeData.getBoolean('androidAppsVisible');
     this.showKioskNextShell_ = loadTimeData.valueExists('showKioskNextShell') &&
         loadTimeData.getBoolean('showKioskNextShell');
-    this.showCrostini_ = loadTimeData.valueExists('showCrostini') &&
-        loadTimeData.getBoolean('showCrostini');
-    this.showPluginVm_ = loadTimeData.valueExists('showPluginVm') &&
-        loadTimeData.getBoolean('showPluginVm');
-    this.havePlayStoreApp_ = loadTimeData.valueExists('havePlayStoreApp') &&
-        loadTimeData.getBoolean('havePlayStoreApp');
+    this.showCrostini_ = loadTimeData.getBoolean('showCrostini');
+    this.showPluginVm_ = loadTimeData.getBoolean('showPluginVm');
+    this.havePlayStoreApp_ = loadTimeData.getBoolean('havePlayStoreApp');
 
     this.addEventListener('show-container', () => {
       this.$.container.style.visibility = 'visible';

chrome/browser/ui/webui/settings/settings_ui.cc

@@ -455,6 +455,7 @@ void SettingsUI::InitOSWebUIHandlers(Profile* profile,
   html_source->AddBoolean("hasInternalStylus",
                           ash::stylus_utils::HasInternalStylus());
 #if defined(KIOSK_NEXT)
+  // Remove valueExists call from os_settings_ui.js when the #define is removed.
   html_source->AddBoolean(
       "showKioskNextShell",
       base::FeatureList::IsEnabled(ash::features::kKioskNextShell) &&