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

Unified Diff: chrome/browser/ui/sync/one_click_signin_sync_observer.cc

Issue 277563008: [Signin] Remove a reference to an obsolete bug. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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/ui/sync/one_click_signin_sync_observer.cc
diff --git a/chrome/browser/ui/sync/one_click_signin_sync_observer.cc b/chrome/browser/ui/sync/one_click_signin_sync_observer.cc
index 376baeb05053159db327700d975c1448a3128a7b..851a1d3a71a1c48830cb14e937f5b2cb3339f499 100644
--- a/chrome/browser/ui/sync/one_click_signin_sync_observer.cc
+++ b/chrome/browser/ui/sync/one_click_signin_sync_observer.cc
@@ -82,12 +82,8 @@ void OneClickSigninSyncObserver::OnStateChanged() {
if (sync_service->sync_initialized() &&
signin::GetSourceForPromoURL(continue_url_)
!= signin::SOURCE_SETTINGS) {
- // TODO(isherman): Redirecting after Sync is set up still has some bugs:
- // http://crbug.com/355885
- // Having multiple settings pages open can cause issues.
- // http://crbug.com/357901
- // Selecting anything other than "Sync Everything" when configuring Sync
- // prevents the redirect.
+ // TODO(isherman): Having multiple settings pages open can cause issues
+ // redirecting after Sync is set up: http://crbug.com/355885
LoadContinueUrl();
}
}
« chrome/browser/services/gcm/gcm_service.cc ('K') | « chrome/browser/services/gcm/gcm_service.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698