Add method to invoke mobile only field trials. Add a field trial that controls rollout of data comp…

chrome/browser/chrome_browser_field_trials.h

Index: chrome/browser/chrome_browser_field_trials.h
diff --git a/chrome/browser/chrome_browser_field_trials.h b/chrome/browser/chrome_browser_field_trials.h
index bd0cf6274a365b29c9e48a9a0e30b67779269b59..932c742007095faa5d51eb1a914a7e3dd175e88f 100644
--- a/chrome/browser/chrome_browser_field_trials.h
+++ b/chrome/browser/chrome_browser_field_trials.h
@@ -33,6 +33,10 @@ class ChromeBrowserFieldTrials {
   // |local_state| is needed by some other methods called from within this one.
   void SetupDesktopFieldTrials(PrefService* local_state);
 
+  // Sets up mobile-only field trials.
+  // Add invocation of your field trial initialization code in this method.
+  void SetupMobileFieldTrials();

[SteveT, 2013/04/01 19:26:51]

Do we want to declare/define this at all on desktop platforms? The code doesn't
get used.

That is, should we be wrapping this and the definition and appropriate IFDEFs?

[vagababov, 2013/04/01 19:34:56]

Makes sense. Wrapped declarations and definitions in the ifdef guards as well.

+
   // This is not quite a field trial initialization, but it's an initialization
   // that depends on a field trial, so why not? :-)
   // |local_state| is needed to reset a local pref based on the chosen group.
@@ -41,6 +45,9 @@ class ChromeBrowserFieldTrials {
   // A/B test for spdy when --use-spdy not set.
   void SpdyFieldTrial();
 
+  // Mobile compression rollout field trial.
+  void DataCompressionProxyFieldTrial();
+
   // A/B test for warmest socket vs. most recently used socket.
   void WarmConnectionFieldTrial();