Chromium Code Reviews| Index: chrome/browser/ui/webui/sync_setup_handler.cc |
| diff --git a/chrome/browser/ui/webui/sync_setup_handler.cc b/chrome/browser/ui/webui/sync_setup_handler.cc |
| index 7e71099760de32741241536d032a4f58bf537b81..49954d8568115156ed4b83f9a71a7955b6a77a6d 100644 |
| --- a/chrome/browser/ui/webui/sync_setup_handler.cc |
| +++ b/chrome/browser/ui/webui/sync_setup_handler.cc |
| @@ -42,6 +42,7 @@ |
| #include "chrome/common/url_constants.h" |
| #include "components/signin/core/browser/profile_oauth2_token_service.h" |
| #include "components/signin/core/browser/signin_error_controller.h" |
| +#include "components/signin/core/browser/signin_metrics.h" |
| #include "components/signin/core/common/profile_management_switches.h" |
| #include "components/sync_driver/sync_prefs.h" |
| #include "content/public/browser/render_view_host.h" |
| @@ -805,6 +806,7 @@ void SyncSetupHandler::HandleStartSignin(const base::ListValue* args) { |
| void SyncSetupHandler::HandleStopSyncing(const base::ListValue* args) { |
| if (GetSyncService()) |
| ProfileSyncService::SyncEvent(ProfileSyncService::STOP_FROM_OPTIONS); |
| + signin_metrics::LogSignout(signin_metrics::USER_CLICKED_SIGNOUT); |
| SigninManagerFactory::GetForProfile(GetProfile())->SignOut(); |
|
Evan Stade
2014/06/19 01:55:12
I think you should make SignOut take a parameter (
Mike Lerman
2014/06/19 17:55:48
Done.
|
| bool delete_profile = false; |
| @@ -849,8 +851,10 @@ void SyncSetupHandler::CloseSyncSetup() { |
| // Sign out the user on desktop Chrome if they click cancel during |
| // initial setup. |
| // TODO(rsimha): Revisit this for M30. See http://crbug.com/252049. |
| - if (sync_service->FirstSetupInProgress()) |
| + if (sync_service->FirstSetupInProgress()) { |
| + signin_metrics::LogSignout(signin_metrics::ABORT_SIGNIN); |
| SigninManagerFactory::GetForProfile(GetProfile())->SignOut(); |
| + } |
| #endif |
| } |
| } |