Enable Chrome Signin from Webstore

chrome/browser/ui/webui/sync_promo/sync_promo_trial.cc

Index: chrome/browser/ui/webui/sync_promo/sync_promo_trial.cc
diff --git a/chrome/browser/ui/webui/sync_promo/sync_promo_trial.cc b/chrome/browser/ui/webui/sync_promo/sync_promo_trial.cc
index 819b38b80a7363501c5c381677620b2752cf35c9..81f39e7abd2d8143af579a247f5c71dff0809ff9 100644
--- a/chrome/browser/ui/webui/sync_promo/sync_promo_trial.cc
+++ b/chrome/browser/ui/webui/sync_promo/sync_promo_trial.cc
@@ -23,6 +23,8 @@ enum {
   UMA_SETTINGS_SIGNED_IN,
   UMA_EXTENSION_INSTALL_BUBBLE_SHOWN,
   UMA_EXTENSION_INSTALL_BUBBLE_SIGNED_IN,
+  UMA_WEBSTORE_INSTALL_SHOWN,
+  UMA_WEBSTORE_INSTALL_SIGNED_IN,

[Roger Tawa OOO till Jul 10th, 2013/01/31 13:27:16]

please add these right before UMA_MAX.

[guohui, 2013/01/31 16:44:02]

Done.

   UMA_UNKNOWN_SHOWN,
   UMA_UNKNOWN_SIGNED_IN,
   UMA_MAX,
@@ -52,6 +54,9 @@ void RecordUserShownPromo(content::WebUI* web_ui) {
     case SyncPromoUI::SOURCE_EXTENSION_INSTALL_BUBBLE:
       uma = UMA_EXTENSION_INSTALL_BUBBLE_SHOWN;
       break;
+    case SyncPromoUI::SOURCE_WEBSTORE_INSTALL:
+      uma = UMA_WEBSTORE_INSTALL_SHOWN;
+      break;
     case SyncPromoUI::SOURCE_UNKNOWN:
       uma = UMA_UNKNOWN_SHOWN;
       break;
@@ -59,7 +64,7 @@ void RecordUserShownPromo(content::WebUI* web_ui) {
       // If this assert hits, then the SyncPromoUI::Source enum has changed and
       // the UMA enum above, this switch statement and histograms.xml all need
       // to be updated to reflect that.
-      COMPILE_ASSERT(SyncPromoUI::SOURCE_UNKNOWN == 5,
+      COMPILE_ASSERT(SyncPromoUI::SOURCE_UNKNOWN == 6,
                      kSourceEnumHasChangedButNotThisSwitchStatement);
       NOTREACHED();
       break;
@@ -90,9 +95,12 @@ void RecordUserSignedIn(content::WebUI* web_ui) {
     case SyncPromoUI::SOURCE_UNKNOWN:
       uma = UMA_UNKNOWN_SIGNED_IN;
       break;
+    case SyncPromoUI::SOURCE_WEBSTORE_INSTALL:
+      uma = UMA_WEBSTORE_INSTALL_SIGNED_IN;
+      break;
     default:
       // This switch statement needs to be updated when the enum Source changes.
-      COMPILE_ASSERT(SyncPromoUI::SOURCE_UNKNOWN == 5,
+      COMPILE_ASSERT(SyncPromoUI::SOURCE_UNKNOWN == 6,
                      kSourceEnumHasChangedButNotThisSwitchStatement);
       NOTREACHED();
       break;