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

Unified Diff: chrome/test/data/webui/md_history/history_synced_tabs_test.js

Issue 2171513004: MD History: Hide sign in guide in guest mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add test Created 4 years, 5 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/md_history/history_synced_tabs_test.js
diff --git a/chrome/test/data/webui/md_history/history_synced_tabs_test.js b/chrome/test/data/webui/md_history/history_synced_tabs_test.js
index c1b0301d35d54d482cceabd5832e7bfa35362a07..0e4a7144d063d3ca81298fb38820a994c4c491cf 100644
--- a/chrome/test/data/webui/md_history/history_synced_tabs_test.js
+++ b/chrome/test/data/webui/md_history/history_synced_tabs_test.js
@@ -253,6 +253,22 @@ cr.define('md_history.history_synced_tabs_test', function() {
});
});
+ test('hide sign in promo in guest mode', function() {
+ // Initial state is signed in.
+ element.signInState_ = true;
+ // Turn to guest mode, which is also signed out state.
+ element.guestSession_ = true;
+ updateSignInState(false);
+ return flush().then(function() {
+ assertTrue(element.$['sign-in-guide'].hidden);
+ var noSyncedResults = loadTimeData.getString('noSyncedResults');
+ assertNotEquals(
+ -1,
+ element.$['no-synced-tabs'].textContent.indexOf(noSyncedResults));
calamity 2016/07/21 04:56:49 I think it's enough to just check this is hidden.
lshang 2016/07/21 07:09:55 Done.
+ element.guestSession_ = false;
calamity 2016/07/21 04:56:49 Put this in the teardown?
lshang 2016/07/21 07:09:55 Done.
+ });
+ });
+
teardown(function() {
element.syncedDevices = [];
element.searchedTerm = '';

Powered by Google App Engine
This is Rietveld 408576698