Record policy usage statistics every 24 hours.

chrome/browser/browser_process_impl.h

Index: chrome/browser/browser_process_impl.h
diff --git a/chrome/browser/browser_process_impl.h b/chrome/browser/browser_process_impl.h
index 404db256e13952aa8e1e3bb2830ff0734bb5a484..d72c80c91d48bb202ca3fb6e3582a1b62fdffc78 100644
--- a/chrome/browser/browser_process_impl.h
+++ b/chrome/browser/browser_process_impl.h
@@ -31,6 +31,7 @@ class RemoteDebuggingServer;
 namespace policy {
 class BrowserPolicyConnector;
 class PolicyService;
+class PolicyStatisticsCollector;
 };
 
 // Real implementation of BrowserProcess that creates and returns the services.
@@ -162,6 +163,10 @@ class BrowserProcessImpl : public BrowserProcess,
   bool created_local_state_;
   scoped_ptr<PrefService> local_state_;
 
+  // Must be destroyed before |local_state_| and |policy_statistics_collector_|.

[Mattias Nissler (ping if slow), 2012/09/12 13:06:27]

Doesn't make sense.

+  // Note that currently this is enforced by StartTearDown.
+  scoped_ptr<policy::PolicyStatisticsCollector> policy_statistics_collector_;

[Joao da Silva, 2012/09/12 13:29:38]

Why doesn't this hang off the BrowserPolicyConnector?

UMA_HISTOGRAM_ENUMERATION can still be called after the MetricsService is shut
down, so policy_statistics_collector_ doesn't really depend on the
metrics_service, as implied on browser_process_impl.cc.

+
   bool created_icon_manager_;
   scoped_ptr<IconManager> icon_manager_;