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

Unified Diff: chrome/browser/signin/signin_tracker_unittest.cc

Issue 9956097: suppress user/password dialog when re-enabling sync (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More OS_CHROMEOS guard for compilation. Created 8 years, 9 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/browser/signin/signin_tracker_unittest.cc
diff --git a/chrome/browser/signin/signin_tracker_unittest.cc b/chrome/browser/signin/signin_tracker_unittest.cc
index 8ae1235ff44f306114af8f402a1475da30bb5e44..ec2d14cec2fb173604da26ab54903f4ffc09c337 100644
--- a/chrome/browser/signin/signin_tracker_unittest.cc
+++ b/chrome/browser/signin/signin_tracker_unittest.cc
@@ -274,3 +274,4 @@ TEST_F(SigninTrackerTest, SyncSigninError) {
tracker_->OnStateChanged();
}
+// TODO(kochi) Add tests with initial state != WATIING_FOR_GAIA_VALIDATION.
arv (Not doing code reviews) 2012/04/04 17:04:21 Can you do this in this patch?
kochi 2012/04/04 17:29:46 Will file a issue on the bug tracker.

Powered by Google App Engine
This is Rietveld 408576698