mus: aura::Env should be destroyed last in the mus client.

content/browser/browser_main_loop.cc

Index: content/browser/browser_main_loop.cc
diff --git a/content/browser/browser_main_loop.cc b/content/browser/browser_main_loop.cc
index aed3789f7230625bd929d0c4a7decd3adfa30f0e..7a75766b02b4c4032c3d8eee6f7e6deccda7c126 100644
--- a/content/browser/browser_main_loop.cc
+++ b/content/browser/browser_main_loop.cc
@@ -1206,10 +1206,6 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() {
     parts_->PostMainMessageLoopRun();
   }
 
-#if defined(USE_AURA)
-  env_.reset();

[msw, 2016/12/21 17:56:35]

aura::Env doesn't have any deps on objects explicitly destroyed below?
(probably hard to ascertain, keep an eye out for shutdown crashes?)

[sadrul, 2016/12/21 18:03:36]

Yep. I will keep an eye on it (the trybots seem happy though)

-#endif
-
   system_stats_monitor_.reset();
 
   // Destroying the GpuProcessHostUIShims on the UI thread posts a task to