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

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

Issue 2341493003: MD Settings: Deflake attempt for CrSettingsPrivacyPageTest.PrivacyPage. (Closed)
Patch Set: Check for specific value. Created 4 years, 3 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/privacy_page_test.js
diff --git a/chrome/test/data/webui/settings/privacy_page_test.js b/chrome/test/data/webui/settings/privacy_page_test.js
index c42bbe0f9ba0a1f3d96285ec1cd70d4c7d0ddebd..0e8afd3c34ec3663e4370653f17e0fc91d3c459b 100644
--- a/chrome/test/data/webui/settings/privacy_page_test.js
+++ b/chrome/test/data/webui/settings/privacy_page_test.js
@@ -105,7 +105,14 @@ cr.define('settings_privacy_page', function() {
assertFalse(!!page.$$('settings-clear-browsing-data-dialog'));
MockInteractions.tap(page.$.clearBrowsingData);
Polymer.dom.flush();
- assertTrue(!!page.$$('settings-clear-browsing-data-dialog'));
+
+ var dialog = page.$$('settings-clear-browsing-data-dialog');
+ assertTrue(!!dialog);
+
+ // Ensure that the dialog is fully opened before returning from this
+ // test, otherwise asynchronous code run in attached() can cause flaky
+ // errors.
+ return test_util.whenAttributeIs(dialog.$.dialog, 'open', true);
});
});
}
« no previous file with comments | « chrome/test/data/webui/settings/cr_settings_browsertest.js ('k') | chrome/test/data/webui/settings/test_util.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698