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

Unified Diff: chrome/test/data/webui/settings/system_page_tests.js

Issue 2055923002: MD Settings: Hook up system_page to new LifetimeBrowserProxy. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lifetime_handler
Patch Set: Fix closure compilation. Created 4 years, 6 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/test/data/webui/settings/cr_settings_browsertest.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/settings/system_page_tests.js
diff --git a/chrome/test/data/webui/settings/system_page_tests.js b/chrome/test/data/webui/settings/system_page_tests.js
index 27ff9a50e51fc2a9ff8a0f55a852f43fc9a3b7a8..a9f2e219961abcc37e3373512f938c8bee38ebb9 100644
--- a/chrome/test/data/webui/settings/system_page_tests.js
+++ b/chrome/test/data/webui/settings/system_page_tests.js
@@ -12,7 +12,7 @@ cr.define('settings_system_page', function() {
* @implements {settings.SystemPageBrowserProxy}
*/
function TestSystemPageBrowserProxy() {
- settings.TestBrowserProxy.call(this, ['restartBrowser']);
+ settings.TestBrowserProxy.call(this, []);
}
TestSystemPageBrowserProxy.prototype = {
@@ -22,26 +22,26 @@ cr.define('settings_system_page', function() {
changeProxySettings: assertNotReached,
/** @override */
- restartBrowser: function() {
- this.methodCalled('restartBrowser');
- },
-
- /** @override */
wasHardwareAccelerationEnabledAtStartup: function() {
return HARDWARE_ACCELERATION_AT_STARTUP;
},
};
suite('SettingsDevicePage', function() {
- /** @type {settings.TestSystemPageBrowserProxy} */
- var testBrowserProxy;
+ /** @type {TestSystemPageBrowserProxy} */
+ var systemBrowserProxy;
+
+ /** @type {settings.TestLifetimeBrowserProxy} */
+ var lifetimeBrowserProxy;
/** @type {SettingsSystemPageElement} */
var systemPage;
setup(function() {
- testBrowserProxy = new TestSystemPageBrowserProxy;
- settings.SystemPageBrowserProxyImpl.instance_ = testBrowserProxy;
+ lifetimeBrowserProxy = new settings.TestLifetimeBrowserProxy();
+ settings.LifetimeBrowserProxyImpl.instance_ = lifetimeBrowserProxy;
+ settings.SystemPageBrowserProxyImpl.instance_ =
+ new TestSystemPageBrowserProxy();
systemPage = document.createElement('settings-system-page');
systemPage.set('prefs', {
@@ -81,7 +81,7 @@ cr.define('settings_system_page', function() {
expectTrue(!!restart); // The "RESTART" button should be showing now.
MockInteractions.tap(restart);
- return testBrowserProxy.whenCalled('restartBrowser');
+ return lifetimeBrowserProxy.whenCalled('restart');
});
});
});
« no previous file with comments | « chrome/test/data/webui/settings/cr_settings_browsertest.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698