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

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

Issue 2847583002: [MD settings] css for row separator (Closed)
Patch Set: browser tests Created 3 years, 8 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
Index: chrome/test/data/webui/settings/appearance_page_test.js
diff --git a/chrome/test/data/webui/settings/appearance_page_test.js b/chrome/test/data/webui/settings/appearance_page_test.js
index 59ef458cef62f1a2a2469f31ff15a0587d3d3a61..2a72e8484485864443e9a82e904516512824fb3d 100644
--- a/chrome/test/data/webui/settings/appearance_page_test.js
+++ b/chrome/test/data/webui/settings/appearance_page_test.js
@@ -181,14 +181,14 @@ suite('AppearanceHandler', function() {
assertFalse(!!appearancePage.$$('#useDefault'));
assertFalse(!!appearancePage.$$('#useSystem'));
// If there's no "USE" buttons, the container should be hidden.
- assertTrue(appearancePage.$$('.secondary-action').hidden);
+ assertTrue(appearancePage.$$('#themesSecondarActions').hidden);
Dan Beam 2017/05/01 15:32:31 at least you're consistent...
dschuyler 2017/05/01 19:26:11 Done.
appearanceBrowserProxy.setIsSupervised(false);
appearancePage.set(THEME_ID_PREF, 'fake theme id');
Polymer.dom.flush();
// If there's "USE" buttons again, the container should be visible.
assertTrue(!!appearancePage.$$('#useDefault'));
- assertFalse(appearancePage.$$('.secondary-action').hidden);
+ assertFalse(appearancePage.$$('#themesSecondarActions').hidden);
var button = appearancePage.$$('#useSystem');
assertTrue(!!button);

Powered by Google App Engine
This is Rietveld 408576698