Fix crash in policy code caused by early shutdown.

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 425cecca1696636c5de04636367844ebfd7e1449..75ac4b67e71e45422274d2e1f912f638a9ab357e 100644
--- a/chrome/browser/browser_process_impl.h
+++ b/chrome/browser/browser_process_impl.h
@@ -154,6 +154,11 @@ class BrowserProcessImpl : public BrowserProcess,
   bool created_watchdog_thread_;
   scoped_ptr<WatchDogThread> watchdog_thread_;
 
+  // Must be destroyed after |policy_service_| if StartTearDown() isn't invoked
+  // during to an early shutdown.

[Mattias Nissler (ping if slow), 2012/05/21 16:33:37]

nit: grammar.

Also, can you please file a bug for fixing up initialization and cleanup order
for the policy stuff?

[Joao da Silva, 2012/05/21 17:14:34]

Done.

+  bool created_browser_policy_connector_;
+  scoped_ptr<policy::BrowserPolicyConnector> browser_policy_connector_;
+
   // Must be destroyed after |local_state_|.
   scoped_ptr<policy::PolicyService> policy_service_;
 
@@ -171,9 +176,6 @@ class BrowserProcessImpl : public BrowserProcess,
 
   scoped_ptr<RemoteDebuggingServer> remote_debugging_server_;
 
-  bool created_browser_policy_connector_;
-  scoped_ptr<policy::BrowserPolicyConnector> browser_policy_connector_;
-
   scoped_refptr<printing::PrintPreviewTabController>
       print_preview_tab_controller_;