Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(423)

Side by Side Diff: chrome/browser/resources/options/browser_options.js

Issue 780703004: [Hotword] Ok Google options disappear if search provider is changed from (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: check for no default url Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | chrome/browser/search/hotword_service.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 cr.exportPath('options'); 5 cr.exportPath('options');
6 6
7 /** 7 /**
8 * @typedef {{actionLinkText: (string|undefined), 8 * @typedef {{actionLinkText: (string|undefined),
9 * hasError: (boolean|undefined), 9 * hasError: (boolean|undefined),
10 * hasUnrecoverableError: (boolean|undefined), 10 * hasUnrecoverableError: (boolean|undefined),
(...skipping 1178 matching lines...) Expand 10 before | Expand all | Expand 10 after
1189 * @private 1189 * @private
1190 */ 1190 */
1191 showHotwordNoDspSection_: function(opt_error) { 1191 showHotwordNoDspSection_: function(opt_error) {
1192 this.showHotwordCheckboxAndIndicator_( 1192 this.showHotwordCheckboxAndIndicator_(
1193 'hotword-no-dsp-search', 1193 'hotword-no-dsp-search',
1194 'hotword-no-dsp-search-setting-indicator', 1194 'hotword-no-dsp-search-setting-indicator',
1195 opt_error); 1195 opt_error);
1196 }, 1196 },
1197 1197
1198 /** 1198 /**
1199 * Controls the visibility of all the hotword sections.
1200 * @param {boolean} visible Whether to show hotword sections.
1201 * @private
1202 */
1203 setHotwordSectionVisible_: function(visible) {
1204 $('hotword-search').hidden = !visible;
1205 $('hotword-always-on-search').hidden = !visible;
1206 $('hotword-no-dsp-search').hidden = !visible;
1207 $('audio-history').hidden = !visible;
1208 },
1209
1210 /**
1199 * Shows or hides the hotword retrain link 1211 * Shows or hides the hotword retrain link
1200 * @param {boolean} visible Whether to show the link. 1212 * @param {boolean} visible Whether to show the link.
1201 * @private 1213 * @private
1202 */ 1214 */
1203 setHotwordRetrainLinkVisible_: function(visible) { 1215 setHotwordRetrainLinkVisible_: function(visible) {
1204 $('hotword-retrain-link').hidden = !visible; 1216 $('hotword-retrain-link').hidden = !visible;
1205 }, 1217 },
1206 1218
1207 /** 1219 /**
1208 * Event listener for the 'hotword always on search enabled' preference. 1220 * Event listener for the 'hotword always on search enabled' preference.
(...skipping 897 matching lines...) Expand 10 before | Expand all | Expand 10 after
2106 'setAutoOpenFileTypesDisplayed', 2118 'setAutoOpenFileTypesDisplayed',
2107 'setBatteryIcon', 2119 'setBatteryIcon',
2108 'setBatteryStatusText', 2120 'setBatteryStatusText',
2109 'setBluetoothState', 2121 'setBluetoothState',
2110 'setCanSetTime', 2122 'setCanSetTime',
2111 'setFontSize', 2123 'setFontSize',
2112 'setHotwordRetrainLinkVisible', 2124 'setHotwordRetrainLinkVisible',
2113 'setNativeThemeButtonEnabled', 2125 'setNativeThemeButtonEnabled',
2114 'setNetworkPredictionValue', 2126 'setNetworkPredictionValue',
2115 'setHighContrastCheckboxState', 2127 'setHighContrastCheckboxState',
2128 'setHotwordSectionVisible',
2116 'setMetricsReportingCheckboxState', 2129 'setMetricsReportingCheckboxState',
2117 'setMetricsReportingSettingVisibility', 2130 'setMetricsReportingSettingVisibility',
2118 'setProfilesInfo', 2131 'setProfilesInfo',
2119 'setSpokenFeedbackCheckboxState', 2132 'setSpokenFeedbackCheckboxState',
2120 'setThemesResetButtonEnabled', 2133 'setThemesResetButtonEnabled',
2121 'setVirtualKeyboardCheckboxState', 2134 'setVirtualKeyboardCheckboxState',
2122 'setupPageZoomSelector', 2135 'setupPageZoomSelector',
2123 'setupProxySettingsButton', 2136 'setupProxySettingsButton',
2124 'showAudioHistorySection', 2137 'showAudioHistorySection',
2125 'showBluetoothSettings', 2138 'showBluetoothSettings',
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
2189 } 2202 }
2190 button.textContent = loadTimeData.getString(strId); 2203 button.textContent = loadTimeData.getString(strId);
2191 }; 2204 };
2192 } 2205 }
2193 2206
2194 // Export 2207 // Export
2195 return { 2208 return {
2196 BrowserOptions: BrowserOptions 2209 BrowserOptions: BrowserOptions
2197 }; 2210 };
2198 }); 2211 });
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/search/hotword_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698