Ensure that on Windows 7, relaunching the browser via chrome://restart or changing about:flags rela…

win8/metro_driver/chrome_app_view_ash.cc

 // 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.
 
 #include "win8/metro_driver/stdafx.h"
 #include "win8/metro_driver/chrome_app_view_ash.h"
 
 #include <corewindow.h>
 #include <shellapi.h>
 #include <windows.foundation.h>
@@ skipping 953 matching lines @@
       SendKeySequence(VK_F4, ALT);
       if (ui_channel_)
         ui_channel_->Close();
     } else {
       globals.app_exit->Exit();
     }
   } else {
     if (ui_channel_)
       ui_channel_->Close();
 
+    HWND core_window = core_window_hwnd();
+    ::PostMessage(core_window, WM_CLOSE, 0, 0);
+

[Shrikant Kelkar, 2014/09/12 21:53:29]

Instead of using multiple close methods may be merge it into above
TERMINATE_USING_KEY_SEQUENCE? Or do it during app_exit somewhere?

[ananta, 2014/09/12 23:49:58]

This function is fragile and the ordering of statements causes timing and
related failures. Doing it in app exit will not work on Windows 8 as that is 
a windows object providing this functionality.

     globals.app_exit->Exit();
   }
 }
 
 void ChromeAppViewAsh::OnInputSourceChanged() {
   if (!input_source_)
     return;
 
   LANGID langid = 0;
   bool is_ime = false;
@@ skipping 419 matching lines @@
   mswr::ComPtr<winapp::Core::ICoreApplicationExit> app_exit;
   CheckHR(core_app.As(&app_exit));
   globals.app_exit = app_exit.Detach();
 }
 
 IFACEMETHODIMP
 ChromeAppViewFactory::CreateView(winapp::Core::IFrameworkView** view) {
   *view = mswr::Make<ChromeAppViewAsh>().Detach();
   return (*view) ? S_OK :  E_OUTOFMEMORY;
 }