[Sync Promo UI] Changing sync promo to honor brand codes.

chrome/browser/ui/webui/sync_promo_trial.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_trial.h"
 
 #include "base/metrics/field_trial.h"
+#include "base/string_util.h"
+#include "chrome/browser/google/google_util.h"
 #include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "grit/generated_resources.h"
 
 namespace sync_promo_trial {
 
 // Field trial IDs of the control and experiment groups. Though they are not
 // literally "const", they are set only once, in Activate() below. See the .h
 // file for what these groups represent.
 int g_sync_promo_experiment_a = 0;
 int g_sync_promo_experiment_b = 0;
 int g_sync_promo_experiment_c = 0;
 int g_sync_promo_experiment_d = 0;
 
+const char kSyncPromoEnabledTrialName[] = "SyncPromoEnabled";
 const char kSyncPromoMsgTrialName[] = "SyncPromoMsg";
 
+const char kSyncPromoEnabledWithApps[] = "EnabledWithDefaultApps";
+const char kSyncPromoEnabledWithoutApps[] = "EnabledWithoutDefaultApps";
+const char kSyncPromoDisabledWithApps[] = "DisabledWithDefaultApps";
+const char kSyncPromoDisabledWithoutApps[] = "DisabledWithoutDefaultApps";
+
 void Activate() {
   // The end date (February 21, 2012) is approximately 2 weeks into M17 stable.
   scoped_refptr<base::FieldTrial> trial(
       new base::FieldTrial(kSyncPromoMsgTrialName, 1000, "MsgA", 2012, 2, 21));
   g_sync_promo_experiment_a = base::FieldTrial::kDefaultGroupNumber;
 
   // Try to give the user a consistent experience, if possible.
   if (base::FieldTrialList::IsOneTimeRandomizationEnabled())
     trial->UseOneTimeRandomization();
 
   // Each group has probability 0.5%, leaving the control with 98.5%.
   g_sync_promo_experiment_b = trial->AppendGroup("MsgB", 50);
   g_sync_promo_experiment_c = trial->AppendGroup("MsgC", 50);
   g_sync_promo_experiment_d = trial->AppendGroup("MsgD", 50);
+
+  // Determine whether we should show the sync promo via brand code.
+  std::string brand;
+  google_util::GetBrand(&brand);
+
+  brand = "ECSB";
+  LOG(ERROR) << "brand: " << brand;

[SteveT, 2011/12/13 15:19:03]

Don't forget to clean this up at the end of the review. I believe you can also
set the brand code through configuration, for testing purposes. Not sure how it
works on linux, but in Windows there is a reg key that contains the brand value.
Roger - do you remember where this is set in win and perhaps linux?

Some digging through GetBrand may also help.

[Roger Tawa OOO till Jul 10th, 2011/12/13 16:00:53]

See GetBrand() for details, but for linux, the brand is always empty.

[Dan Beam, 2011/12/14 00:16:54]

Done.

+
+  // Create a field trial based on the brand code.
+  if (LowerCaseEqualsASCII(brand, "ecba")) {
+    base::FieldTrialList::CreateFieldTrial(kSyncPromoEnabledTrialName,
+                                           kSyncPromoEnabledWithApps);
+  } else if (LowerCaseEqualsASCII(brand, "ecsa")) {
+    base::FieldTrialList::CreateFieldTrial(kSyncPromoEnabledTrialName,
+                                           kSyncPromoEnabledWithoutApps);
+  } else if (LowerCaseEqualsASCII(brand, "ecsb")) {
+    base::FieldTrialList::CreateFieldTrial(kSyncPromoEnabledTrialName,
+                                           kSyncPromoDisabledWithApps);
+  } else if (LowerCaseEqualsASCII(brand, "ecbb")) {
+    base::FieldTrialList::CreateFieldTrial(kSyncPromoEnabledTrialName,
+                                           kSyncPromoDisabledWithoutApps);
+  }
 }
 
 bool IsExperimentActive() {
   return base::FieldTrialList::FindValue(kSyncPromoMsgTrialName) !=
       base::FieldTrial::kNotFinalized;
 }
 
+bool IsPartOfBrandTrialToEnable() {
+  return base::FieldTrialList::TrialExists(kSyncPromoEnabledTrialName);
+}
+
 Group GetGroup() {
   // Promo message A is also the default value, so display it if there is no
   // active experiment.
   if (!IsExperimentActive())
     return PROMO_MSG_A;
 
   const int group = base::FieldTrialList::FindValue(kSyncPromoMsgTrialName);
   if (group == g_sync_promo_experiment_a)
     return PROMO_MSG_A;
   else if (group == g_sync_promo_experiment_b)
     return PROMO_MSG_B;
   else if (group == g_sync_promo_experiment_c)
     return PROMO_MSG_C;
   else if (group == g_sync_promo_experiment_d)
     return PROMO_MSG_D;
 
   NOTREACHED();
   return PROMO_MSG_A;
 }
 
+int GetSyncPromoAndDefaultAppsCombination() {
+  // This should never happen, but it doesn't hurt to CHECK().
+  DCHECK(IsPartOfBrandTrialToEnable());
+
+  std::string group =
+      base::FieldTrialList::Find(kSyncPromoEnabledTrialName)->group_name();
+
+  if (group == kSyncPromoEnabledWithApps)
+    return WITH_SYNC_PROMO_WITH_DEFAULT_APPS;
+  else if (group == kSyncPromoEnabledWithoutApps)
+    return WITH_SYNC_PROMO_WITHOUT_DEFAULT_APPS;
+  else if (group == kSyncPromoDisabledWithApps)
+    return WITHOUT_SYNC_PROMO_WITH_DEFAULT_APPS;
+  else if (group == kSyncPromoDisabledWithoutApps)
+    return WITH_SYNC_PROMO_WITH_DEFAULT_APPS;
+
+  NOTREACHED();
+  return SYNC_PROMO_AND_DEFAULT_APPS_BOUNDARY + 1;
+}
+
 int GetMessageBodyResID() {
   // Note that GetGroup and the switch will take care of the !IsExperimentActive
   // case for us.
   Group group = GetGroup();
   switch (group) {
     case PROMO_MSG_A:
       return IDS_SYNC_PROMO_MESSAGE_BODY_A;
     case PROMO_MSG_B:
       return IDS_SYNC_PROMO_MESSAGE_BODY_B;
     case PROMO_MSG_C:
@@ skipping 15 matching lines @@
                             PROMO_MSG_MAX);
 }
 
 void RecordUserSignedIn() {
   DCHECK(IsExperimentActive());
   UMA_HISTOGRAM_ENUMERATION("SyncPromo.MessageOnSignIn",
                             GetGroup(),
                             PROMO_MSG_MAX);
 }
 
+void RecordUserSignedInWithTrialBrand() {
+  DCHECK(IsPartOfBrandTrialToEnable());
+  UMA_HISTOGRAM_ENUMERATION("SyncPromo.SignedInWithBrand",
+                            GetSyncPromoAndDefaultAppsCombination(),
+                            SYNC_PROMO_AND_DEFAULT_APPS_BOUNDARY);

[SteveT, 2011/12/13 15:19:03]

A couple questions/points about this UMA histogram...

First:
This gives us a count for the number of users with a given brand code that sign
in using the startup sync promo page. How do we normalize this value across the
total number of browsers with this brand code? I'm guessing we have some
RLZ-related facility for getting the number of installs and first runs for a
given brand code?

In my experiment, I had a second UMA stat (SyncPromo.MessageDisplayed, seen
above), that we used to normalize values. Maybe we don't need something like
that for this new experiment since we have our own counts of installs/first-runs
by brand code.

Second:
If you haven't already, don't forget to the corresponding enum values and
descriptions in src/tools/histograms/histograms.xml, and have jar@ review it.
This has to be checked in before we see changes reported in our histograms dash.

My corresponding patch was here: https://chromereviews.googleplex.com/3739013/

[Dan Beam, 2011/12/14 00:16:54]

Done.
Yeah, I've done this before (in fact, I created all the other SyncPromo.*
histograms around your CL, :P).  Thank you for the reminder, though.

+}
+
+bool ShouldShowAtStartupBasedOnBrand() {
+  DCHECK(IsPartOfBrandTrialToEnable());
+  std::string name =
+      base::FieldTrialList::Find(kSyncPromoEnabledTrialName)->group_name();
+  return name == kSyncPromoEnabledWithApps ||
+         name == kSyncPromoEnabledWithoutApps;
+}
+
 }  // namespace sync_promo_trial