Enhanced Sync Confirmation modal

chrome/browser/ui/webui/signin/sync_confirmation_handler.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/signin/sync_confirmation_handler.h"
 
+#include <vector>
+
 #include "base/bind.h"
 #include "chrome/browser/profiles/profile_avatar_icon_util.h"
 #include "chrome/browser/signin/account_tracker_service_factory.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/webui/signin/login_ui_service_factory.h"
 #include "components/signin/core/browser/account_tracker_service.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/browser/web_contents.h"
@@ skipping 17 matching lines @@
       base::Bind(&SyncConfirmationHandler::HandleUndo, base::Unretained(this)));
   web_ui()->RegisterMessageCallback("initialized",
       base::Bind(&SyncConfirmationHandler::HandleInitialized,
                  base::Unretained(this)));
   web_ui()->RegisterMessageCallback("goToSettings",
       base::Bind(&SyncConfirmationHandler::HandleGoToSettings,
                  base::Unretained(this)));
 }
 
 void SyncConfirmationHandler::HandleConfirm(const base::ListValue* args) {
-  CloseModalSigninWindow(LoginUIService::SYNC_WITH_DEFAULT_SETTINGS);
+  bool open_activity_controls_url = false;
+  bool success = args->GetBoolean(0, &open_activity_controls_url);
+  DCHECK(success);
+  CloseModalSigninWindow(LoginUIService::SYNC_WITH_DEFAULT_SETTINGS,
+                         open_activity_controls_url);
 }
 
 void SyncConfirmationHandler::HandleGoToSettings(const base::ListValue* args) {
-  CloseModalSigninWindow(LoginUIService::CONFIGURE_SYNC_FIRST);
+  bool open_activity_controls_url = false;
+  bool success = args->GetBoolean(0, &open_activity_controls_url);
+  DCHECK(success);
+  CloseModalSigninWindow(LoginUIService::CONFIGURE_SYNC_FIRST,
+                         open_activity_controls_url);
 }
 
 void SyncConfirmationHandler::HandleUndo(const base::ListValue* args) {
   content::RecordAction(base::UserMetricsAction("Signin_Undo_Signin"));
   Browser* browser = GetDesktopBrowser();
   LoginUIServiceFactory::GetForProfile(browser->profile())->
-      SyncConfirmationUIClosed(LoginUIService::ABORT_SIGNIN);
+      SyncConfirmationUIClosed(LoginUIService::ABORT_SIGNIN, false);
   SigninManagerFactory::GetForProfile(Profile::FromWebUI(web_ui()))->SignOut(
       signin_metrics::ABORT_SIGNIN,
       signin_metrics::SignoutDelete::IGNORE_METRIC);
   browser->CloseModalSigninWindow();
 }
 
 void SyncConfirmationHandler::HandleInitialized(const base::ListValue* args) {
   Browser* browser = GetDesktopBrowser();
   Profile* profile = browser->profile();
   std::vector<AccountInfo> accounts =
@@ skipping 32 matching lines @@
 Browser* SyncConfirmationHandler::GetDesktopBrowser() {
   Browser* browser = chrome::FindBrowserWithWebContents(
       web_ui()->GetWebContents());
   if (!browser)
     browser = chrome::FindLastActiveWithProfile(Profile::FromWebUI(web_ui()));
   DCHECK(browser);
   return browser;
 }
 
 void SyncConfirmationHandler::CloseModalSigninWindow(
-    LoginUIService::SyncConfirmationUIClosedResults results) {
+    LoginUIService::SyncConfirmationUIClosedResults results,

[Evan Stade, 2016/03/29 22:50:21]

nit: SyncConfirmationUIClosedResults made me think this was a struct. It should
be renamed SyncConfirmationUiClosedResult and |results| should be |result| (here
and many places).

Also, not a nit: maybe make this a bitmask, so you are calling with, e.g.,

LoginUIService::CONFIGURE_SYNC_FIRST |
LoginUIService::OPEN_ACTIVITY_CONTROLS_URL

instead of forcing all callers to update, often with a mysterious "false".

[Moe, 2016/03/30 18:07:47]

Done.

+    bool open_activity_controls_url) {
   Browser* browser = GetDesktopBrowser();
   LoginUIServiceFactory::GetForProfile(browser->profile())->
-      SyncConfirmationUIClosed(results);
+      SyncConfirmationUIClosed(results, open_activity_controls_url);
   browser->CloseModalSigninWindow();
 }