Create a field test for sync sign in promo strings.

chrome/browser/ui/webui/sync_promo_handler.cc

 // Copyright (c) 2011 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/sync_promo_handler.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/metrics/histogram.h"
 #include "base/time.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/sync/sync_setup_flow.h"
 #include "chrome/browser/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/webui/sync_promo_trial.h"
 #include "chrome/browser/ui/webui/sync_promo_ui.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_service.h"
 
 namespace {
@@ skipping 85 matching lines @@
                  base::Unretained(this)));
   web_ui_->RegisterMessageCallback("SyncPromo:UserFlowAction",
       base::Bind(&SyncPromoHandler::HandleUserFlowAction,
                  base::Unretained(this)));
   web_ui_->RegisterMessageCallback("SyncPromo:UserSkipped",
       base::Bind(&SyncPromoHandler::HandleUserSkipped,
                  base::Unretained(this)));
   SyncSetupHandler::RegisterMessages();
 }
 
+void SyncPromoHandler::ShowGaiaSuccessAndClose() {
+  if (SyncPromoTrial::IsExperimentActive())
+    SyncPromoTrial::RecordUserSignedIn();
+
+  SyncSetupHandler::ShowGaiaSuccessAndClose();
+}
+
+void SyncPromoHandler::ShowGaiaSuccessAndSettingUp() {
+  if (SyncPromoTrial::IsExperimentActive())
+    SyncPromoTrial::RecordUserSignedIn();
+
+  SyncSetupHandler::ShowGaiaSuccessAndSettingUp();
+}
+
 void SyncPromoHandler::ShowConfigure(const base::DictionaryValue& args) {
   bool usePassphrase = false;
   args.GetBoolean("usePassphrase", &usePassphrase);
 
   if (usePassphrase) {
     // If a passphrase is required then we must show the configure pane.
     SyncSetupHandler::ShowConfigure(args);
   } else {
     // If no passphrase is required then skip the configure pane and sync
     // everything by default. This makes the first run experience simpler.
@@ skipping 24 matching lines @@
         window_already_closed_ = true;
       }
       break;
     }
     default: {
       NOTREACHED();
     }
   }
 }
 
+void SyncPromoHandler::HandleSubmitAuth(const base::ListValue* args) {
+  // Call the original implementation.

[sail, 2011/11/28 16:34:25]

comment should say why you're doing something not what you're doing

[SteveT, 2011/11/28 20:07:32]

Oops sorry - this was from the remnants of some earlier implementation which is
not needed and removed now.

+  SyncSetupHandler::HandleSubmitAuth(args);
+}
+
 void SyncPromoHandler::ShowSetupUI() {
   ProfileSyncService* service =
       Profile::FromWebUI(web_ui_)->GetProfileSyncService();
   service->get_wizard().Step(SyncSetupWizard::GetLoginState());
 }
 
 void SyncPromoHandler::HandleCloseSyncPromo(const base::ListValue* args) {
   CloseSyncSetup();
 
   // If the user has signed in then set the pref to show them NTP bubble
   // confirming that they're signed in.
   std::string username = prefs_->GetString(prefs::kGoogleServicesUsername);
   if (!username.empty())
     prefs_->SetBoolean(prefs::kSyncPromoShowNTPBubble, true);
 
   GURL url = SyncPromoUI::GetNextPageURLForSyncPromoURL(
       web_ui_->tab_contents()->GetURL());
   web_ui_->tab_contents()->OpenURL(url, GURL(), CURRENT_TAB,
                                    content::PAGE_TRANSITION_LINK);
 }
 
 void SyncPromoHandler::HandleInitializeSyncPromo(const base::ListValue* args) {
-  base::FundamentalValue visible(SyncPromoUI::GetShowTitleForSyncPromoURL(
-      web_ui_->tab_contents()->GetURL()));
+  // If the promo is also the Chrome launch page, we want to show the title and
+  // load the promo experiments for the page.
+  bool is_launch_page = SyncPromoUI::GetLaunchPageForSyncPromoURL(
+      web_ui_->tab_contents()->GetURL());
+  if (is_launch_page)
+    LoadPromoExperiments();
+  base::FundamentalValue visible(is_launch_page);
   web_ui_->CallJavascriptFunction("SyncSetupOverlay.setPromoTitleVisible",
                                   visible);
 
   OpenSyncSetup();
   // We don't need to compute anything for this, just do this every time.
   RecordUserFlowAction(SYNC_PROMO_VIEWED);
   // Increment view count first and show natural numbers in stats rather than 0
   // based starting point (if it happened to be our first time showing this).
   IncrementViewCountBy(1);
   // Record +1 for every view.  This is the only thing we record that's not part
   // of the user flow histogram.
   UMA_HISTOGRAM_COUNTS("SyncPromo.NumTimesViewed", GetViewCount());
 }
 
+void SyncPromoHandler::LoadPromoExperiments() {
+  // If we're not running the trial, we don't have to set the string since
+  // it is already set to the default.
+  if (!SyncPromoTrial::IsExperimentActive())
+    return;
+
+  std::string resource_name;
+  SyncPromoTrial::Group group = SyncPromoTrial::GetGroup();
+  switch (group) {
+    case SyncPromoTrial::PROMO_MSG_A:
+      resource_name = "promoMessageBodyA";
+      break;
+    case SyncPromoTrial::PROMO_MSG_B:
+      resource_name = "promoMessageBodyB";
+      break;
+    case SyncPromoTrial::PROMO_MSG_C:
+      resource_name = "promoMessageBodyC";
+      break;
+    case SyncPromoTrial::PROMO_MSG_D:
+      resource_name = "promoMessageBodyD";
+      break;
+    default:
+      NOTREACHED() << "Unrecognized SyncPromoTrial::Group.";
+      resource_name = "promoMessageBodyA";
+      break;
+  }
+
+  SyncPromoTrial::RecordUserSawMessage();
+
+  // Call back to javascript with the resource name and let them set it.
+  StringValue result(resource_name);
+  web_ui_->CallJavascriptFunction("SyncSetupOverlay.populatePromoMessage",
+                                  result);
+}
+
 void SyncPromoHandler::HandleShowAdvancedSettings(
     const base::ListValue* args) {
   CloseSyncSetup();
   std::string url(chrome::kChromeUISettingsURL);
   url += chrome::kSyncSetupSubPage;
   web_ui_->tab_contents()->OpenURL(GURL(url), GURL(), CURRENT_TAB,
                                    content::PAGE_TRANSITION_LINK);
   RecordUserFlowAction(SYNC_PROMO_ADVANCED_CLICKED);
 }
 
@@ skipping 39 matching lines @@
   int adjusted = GetViewCount() + amount;
   prefs_->SetInteger(prefs::kSyncPromoViewCount, adjusted);
   return adjusted;
 }
 
 void SyncPromoHandler::RecordUserFlowAction(int action) {
   // Send an enumeration to our single user flow histogram.
   UMA_HISTOGRAM_ENUMERATION("SyncPromo.UserFlow", action,
                             SYNC_PROMO_BUCKET_BOUNDARY);
 }