assistant: cleanup dead code

Bug: None
Test: locally build and run
Change-Id: Ifdb6e31b926da18c6da47e7a2a5540f58e9fa9a9
Reviewed-on: https://chromium-review.googlesource.com/c/1423620Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#625391}

chrome/browser/resources/settings/google_assistant_page/google_assistant_browser_proxy.js

@@ -10,18 +10,6 @@
 cr.define('settings', function() {
   /** @interface */
   class GoogleAssistantBrowserProxy {
-    /**
-     * Enables or disables the Google Assistant.
-     * @param {boolean} enabled
-     */
-    setGoogleAssistantEnabled(enabled) {}
-
-    /**
-     * Enables or disables screen context for the Google Assistant.
-     * @param {boolean} enabled
-     */
-    setGoogleAssistantContextEnabled(enabled) {}
-
     /** Launches into the Google Assistant app settings. */
     launchGoogleAssistantSettings() {}
 
@@ -34,16 +22,6 @@ cr.define('settings', function() {
 
   /** @implements {settings.GoogleAssistantBrowserProxy} */
   class GoogleAssistantBrowserProxyImpl {
-    /** @override */
-    setGoogleAssistantEnabled(enabled) {
-      chrome.send('setGoogleAssistantEnabled', [enabled]);
-    }
-
-    /** @override */
-    setGoogleAssistantContextEnabled(enabled) {
-      chrome.send('setGoogleAssistantContextEnabled', [enabled]);
-    }
-
     /** @override */
     showGoogleAssistantSettings() {
       chrome.send('showGoogleAssistantSettings');

chrome/browser/resources/settings/google_assistant_page/google_assistant_page.html

@@ -19,16 +19,14 @@
         class="first primary-toggle"
         pref="{{prefs.settings.voice_interaction.enabled}}"
         label="[[getAssistantOnOffLabel_(
-            prefs.settings.voice_interaction.enabled.value)]]"
-        on-change="onGoogleAssistantEnableChange_">
+            prefs.settings.voice_interaction.enabled.value)]]">
     </settings-toggle-button>
     <template is="dom-if"
         if="[[prefs.settings.voice_interaction.enabled.value]]">
       <settings-toggle-button id="googleAssistantContextEnable"
           pref="{{prefs.settings.voice_interaction.context.enabled}}"
           label="$i18n{googleAssistantEnableContext}"
-          sub-label="$i18n{googleAssistantEnableContextDescription}"
-          on-change="onGoogleAssistantContextEnableChange_">
+          sub-label="$i18n{googleAssistantEnableContextDescription}">
       </settings-toggle-button>
       <template is="dom-if" if="[[assistantFeatureEnabled_]]">
         <settings-toggle-button id="googleAssistantHotwordEnable"

chrome/browser/resources/settings/google_assistant_page/google_assistant_page.js

@@ -96,18 +96,6 @@ Polymer({
     return this.i18n(toggleValue ? 'toggleOn' : 'toggleOff');
   },
 
-  /** @private */
-  onGoogleAssistantEnableChange_: function() {
-    this.browserProxy_.setGoogleAssistantEnabled(
-        !!this.getPref('settings.voice_interaction.enabled.value'));
-  },
-
-  /** @private */
-  onGoogleAssistantContextEnableChange_: function() {
-    this.browserProxy_.setGoogleAssistantContextEnabled(
-        !!this.getPref('settings.voice_interaction.context.enabled.value'));
-  },
-
   /** @private */
   onGoogleAssistantSettingsTapped_: function() {
     this.browserProxy_.showGoogleAssistantSettings();

chrome/test/data/webui/settings/google_assistant_page_test.js

@@ -8,22 +8,10 @@
 class TestGoogleAssistantBrowserProxy extends TestBrowserProxy {
   constructor() {
     super([
-      'setGoogleAssistantEnabled',
-      'setGoogleAssistantContextEnabled',
       'showGoogleAssistantSettings',
     ]);
   }
 
-  /** @override */
-  setGoogleAssistantEnabled(enabled) {
-    this.methodCalled('setGoogleAssistantEnabled', enabled);
-  }
-
-  /** @override */
-  setGoogleAssistantContextEnabled(enabled) {
-    this.methodCalled('setGoogleAssistantContextEnabled', enabled);
-  }
-
   /** @override */
   showGoogleAssistantSettings() {
     this.methodCalled('showGoogleAssistantSettings');
@@ -69,8 +57,6 @@ suite('GoogleAssistantHandler', function() {
     button.click();
     Polymer.dom.flush();
     assertTrue(button.checked);
-    return browserProxy.whenCalled('setGoogleAssistantEnabled')
-        .then(assertTrue);
   });
 
   test('toggleAssistantContext', function() {
@@ -86,8 +72,6 @@ suite('GoogleAssistantHandler', function() {
     button.click();
     Polymer.dom.flush();
     assertTrue(button.checked);
-    return browserProxy.whenCalled('setGoogleAssistantContextEnabled')
-        .then(assertTrue);
   });
 
   test('tapOnAssistantSettings', function() {