Activate the VariationsService for ChromeOS and ensure that it does not ping the server until the E…

chrome/browser/metrics/variations/variations_service.h

 // Copyright (c) 2012 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_METRICS_VARIATIONS_VARIATIONS_SERVICE_H_
 #define CHROME_BROWSER_METRICS_VARIATIONS_VARIATIONS_SERVICE_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/metrics/field_trial.h"
 #include "base/time.h"
 #include "base/timer.h"
 #include "chrome/browser/metrics/proto/study.pb.h"
 #include "chrome/browser/metrics/proto/trials_seed.pb.h"
 #include "chrome/common/chrome_version_info.h"
 #include "googleurl/src/gurl.h"
 #include "net/base/network_change_notifier.h"
 #include "net/url_request/url_fetcher_delegate.h"
 
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/login/wizard_controller.h"
+#endif
+
 class PrefService;
 
 namespace chrome_variations {
 
 // Used to setup field trials based on stored variations seed data, and fetch
 // new seed data from the variations server.
 class VariationsService
     : public net::URLFetcherDelegate,
-      public net::NetworkChangeNotifier::ConnectionTypeObserver{
+#if defined(OS_CHROMEOS)
+      public chromeos::WizardController::Observer,
+#endif
+      public net::NetworkChangeNotifier::ConnectionTypeObserver {
  public:
   VariationsService();
   virtual ~VariationsService();
 
   // Creates field trials based on Variations Seed loaded from local prefs. If
   // there is a problem loading the seed data, all trials specified by the seed
   // may not be created.
   bool CreateTrialsFromSeed(PrefService* local_prefs);
 
   // Calls FetchVariationsSeed once and repeats this periodically. See
   // implementation for details on the period. Must be called after
   // |CreateTrialsFromSeed|.
   void StartRepeatedVariationsSeedFetch();
 
   // Starts the fetching process once, where |OnURLFetchComplete| is called with
   // the response. If the network is down at the time, sets a flag to retry when
   // the network is back online. This is virtual so it can be overriden for
   // testing.
   virtual void FetchVariationsSeed();
 
   // Exposed for testing.
   void SetWasOfflineDuringLastRequestAttemptForTesting(bool offline);
 
+#if defined(OS_CHROMEOS)
+  // chromeos::WizardController::Observer implementation:
+  virtual void OnScreenChanged(chromeos::WizardScreen* next_screen) OVERRIDE;
+
+  virtual void OnSessionStart() OVERRIDE;

[Alexei Svitkine (slow), 2012/09/10 16:50:11]

Remove empty lines between these functions.

[SteveT, 2012/09/10 19:33:52]

Done.

+
+  virtual void OnEulaAccepted() OVERRIDE;
+#endif
+
   // Register Variations related prefs in Local State.
   static void RegisterPrefs(PrefService* prefs);
 
  private:
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyChannel);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyLocale);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyPlatform);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyVersion);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyVersionWildcards);
   FRIEND_TEST_ALL_PREFIXES(VariationsServiceTest, CheckStudyStartDate);
@@ skipping 74 matching lines @@
   // The URL to use for querying the Variations server.
   GURL variations_server_url_;
 
   // Cached serial number from the most recently fetched Variations seed.
   std::string variations_serial_number_;
 
   // Tracks whether |CreateTrialsFromSeed| has been called, to ensure that
   // it gets called prior to |StartRepeatedVariationsSeedFetch|.
   bool create_trials_from_seed_called_;
 
+#if defined(OS_CHROMEOS)
+  // Tracks whether or not the service is waiting for the user to accept the
+  // EULA before performing the next request.
+  bool waiting_for_user_to_accept_eula_;
+#endif
+
   // Tracks whether or not the last seed request attempt failed due to being
   // offline.
   bool was_offline_during_last_request_attempt_;
 
   // The timer used to repeatedly ping the server. Keep this as an instance
   // member so if VariationsService goes out of scope, the timer is
   // automatically canceled.
   base::RepeatingTimer<VariationsService> timer_;
 };
 
 }  // namespace chrome_variations
 
 #endif  // CHROME_BROWSER_METRICS_VARIATIONS_VARIATIONS_SERVICE_H_