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

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

Issue 2468173004: MD Settings: show restart button when user toggles metrics reporting (Closed)
Patch Set: Created 4 years, 1 month 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
Index: chrome/test/data/webui/settings/metrics_reporting_tests.js
diff --git a/chrome/test/data/webui/settings/metrics_reporting_tests.js b/chrome/test/data/webui/settings/metrics_reporting_tests.js
index 5ee30c769e07aa7bb25719c51f917535e1bf1ba7..1c612a7bc5b081f94fcbcd55f3640f87a594f061 100644
--- a/chrome/test/data/webui/settings/metrics_reporting_tests.js
+++ b/chrome/test/data/webui/settings/metrics_reporting_tests.js
@@ -21,6 +21,7 @@ suite('metrics reporting', function() {
test('changes to whether metrics reporting is enabled/managed', function() {
return testBrowserProxy.whenCalled('getMetricsReporting').then(function() {
Polymer.dom.flush();
+ assertFalse(!!page.$$('#metricsReporting paper-button'));
var checkbox = page.$.metricsReportingCheckbox;
assertEquals(testBrowserProxy.metricsReporting.enabled, checkbox.checked);
@@ -34,6 +35,8 @@ suite('metrics reporting', function() {
cr.webUIListenerCallback('metrics-reporting-change', changedMetrics);
Polymer.dom.flush();
+ assertTrue(!!page.$$('#metricsReporting paper-button'));
+
assertEquals(changedMetrics.enabled, checkbox.checked);
indicatorVisible = !!page.$$('#indicator');
assertEquals(changedMetrics.managed, indicatorVisible);
@@ -42,6 +45,8 @@ suite('metrics reporting', function() {
MockInteractions.tap(checkbox);
return testBrowserProxy.whenCalled('setMetricsReportingEnabled', toggled);
+ }).then(function() {
+ assertTrue(!!page.$$('#metricsReporting paper-button'));
});
});
});

Powered by Google App Engine
This is Rietveld 408576698