Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1767)

Unified Diff: chrome/browser/ui/webui/sync_promo_trial.h

Issue 8689006: Create a field test for sync sign in promo strings. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: addressed nits and removed NTP cache stuff Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/sync_promo_trial.h
===================================================================
--- chrome/browser/ui/webui/sync_promo_trial.h (revision 0)
+++ chrome/browser/ui/webui/sync_promo_trial.h (revision 0)
@@ -0,0 +1,61 @@
+// 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.
+
+#ifndef CHROME_BROWSER_UI_WEBUI_SYNC_PROMO_TRIAL_H_
+#define CHROME_BROWSER_UI_WEBUI_SYNC_PROMO_TRIAL_H_
+
+#include "base/basictypes.h"
+
+// This class manages the Sync Promo field trial.
+//
+// The sync promo involves changing the message body in the sign in promo (see
+// SyncPromoHandler) to see if that has any effect on sign in.
+class SyncPromoTrial {
James Hawkins 2011/11/28 21:42:09 Please get rid of the class and move these methods
SteveT 2011/11/28 22:11:15 Done.
+ public:
+ enum Group {
+ // "Get your bookmarks, history, and settings on all your devices."
+ // This is also the default message group.
+ PROMO_MSG_A = 0,
+
+ // "Back up your bookmarks, history, and settings to the web."
+ PROMO_MSG_B,
+
+ // "Sync your personalized browser features between all your devices."
+ PROMO_MSG_C,
+
+ // "You'll be automatically signed into to your favorite Google services."
+ PROMO_MSG_D,
+
+ // Bounding max value needed for UMA histogram macro.
+ PROMO_MSG_MAX,
+ };
+
+ // Activate the field trial. Before this call, all calls to GetGroup will
+ // return PROMO_MSG_A. *** MUST NOT BE CALLED MORE THAN ONCE. ***
+ static void Activate();
+
+ // Returns true iff the experiment has been set up and is active. If this
+ // is false, the caller should not record stats for this experiment.
+ static bool IsExperimentActive();
+
+ // Return the field trial group this client belongs to.
+ static Group GetGroup();
+
+ // Return the resource ID for the Sync Promo message body associated with this
+ // client.
+ static int GetMessageBodyResID();
+
+ // Record the appropriate UMA stat for when a user sees the sync promo
+ // message.
+ static void RecordUserSawMessage();
+
+ // Record the appropriate UMA stat for when a user successfully signs in to
+ // GAIA.
+ static void RecordUserSignedIn();
+
+ private:
+ DISALLOW_IMPLICIT_CONSTRUCTORS(SyncPromoTrial);
+};
+
+#endif // CHROME_BROWSER_UI_WEBUI_SYNC_PROMO_TRIAL_H_
Property changes on: chrome/browser/ui/webui/sync_promo_trial.h
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698