Tests for cloud policy UMA metrics.

chrome/browser/policy/device_management_service.h

Index: chrome/browser/policy/device_management_service.h
diff --git a/chrome/browser/policy/device_management_service.h b/chrome/browser/policy/device_management_service.h
index bb74aec61cd0052885eccfe347af348a4b1dafe6..9029ace14c8dd61db1a9edfe1c36460050521d6d 100644
--- a/chrome/browser/policy/device_management_service.h
+++ b/chrome/browser/policy/device_management_service.h
@@ -72,13 +72,14 @@ class DeviceManagementService : public URLFetcher::Delegate {
   // callback.
   void RemoveJob(DeviceManagementJob* job);
 
+ protected:
+  // Starts the given job.
+  virtual void StartJob(DeviceManagementJob* job);

[gfeher, 2011/07/19 13:45:33]

Maybe you can make this private now?

[Joao da Silva, 2011/07/19 15:19:39]

Not really, since the Mock needs to override this method, and sometimes calls it
directly (StartJobNotMocked). That's why it's now protected and virtual.

+
  private:
   typedef std::map<const URLFetcher*, DeviceManagementJob*> JobFetcherMap;
   typedef std::deque<DeviceManagementJob*> JobQueue;
 
-  // Starts the given job.
-  void StartJob(DeviceManagementJob* job);
-
   // URLFetcher::Delegate override.
   virtual void OnURLFetchComplete(const URLFetcher* source,
                                   const GURL& url,