Chromium Code Reviews| Index: components/password_manager/core/browser/password_manager_metrics_util.h |
| diff --git a/components/password_manager/core/browser/password_manager_metrics_util.h b/components/password_manager/core/browser/password_manager_metrics_util.h |
| index 9f22f7920d632ab43c55699aa1c620d1517fe020..222c57026b812ef306a50df7ef006343970b7207 100644 |
| --- a/components/password_manager/core/browser/password_manager_metrics_util.h |
| +++ b/components/password_manager/core/browser/password_manager_metrics_util.h |
| @@ -130,6 +130,13 @@ enum AccountChooserUserAction { |
| ACCOUNT_CHOOSER_ACTION_COUNT |
| }; |
| +enum ChromeSignInUserAction { |
|
Ilya Sherman
2016/06/03 21:21:24
nit: What is the "Chrome" part of this enum name r
vasilii
2016/06/06 15:08:46
I think SyncSignIn is better. It has nothing to do
|
| + CHROME_SIGNIN_DISMISSED, |
| + CHROME_SIGNIN_OK, |
| + CHROME_SIGNIN_CANCEL, |
| + CHROME_SIGNIN_ACTION_COUNT |
| +}; |
| + |
| // A version of the UMA_HISTOGRAM_BOOLEAN macro that allows the |name| |
| // to vary over the program's runtime. |
| void LogUMAHistogramBoolean(const std::string& name, bool sample); |
| @@ -171,6 +178,9 @@ void LogAutoSigninPromoUserAction(AutoSigninPromoUserAction action); |
| void LogAccountChooserUserActionOneAccount(AccountChooserUserAction action); |
| void LogAccountChooserUserActionManyAccounts(AccountChooserUserAction action); |
| +// Log a user action on showing the Chrome sign in promo. |
| +void LogAutoSigninPromoUserAction(ChromeSignInUserAction action); |
| + |
| } // namespace metrics_util |
| } // namespace password_manager |