In WinAura, also kill the Metro viewer process in AttemptExit().

chrome/browser/browser_process.h

Index: chrome/browser/browser_process.h
diff --git a/chrome/browser/browser_process.h b/chrome/browser/browser_process.h
index 0b4c6746d1fadd6a8b75964c9aedb406b2e399b2..609196b26fe5c78fc3124f965bddbb08dda54f29 100644
--- a/chrome/browser/browser_process.h
+++ b/chrome/browser/browser_process.h
@@ -225,6 +225,9 @@ class BrowserProcess {
 #if defined(OS_WIN) && defined(USE_AURA)
   // Invoked when the ASH metro viewer process on Windows 8 exits.
   virtual void OnMetroViewerProcessTerminated() = 0;
+
+  // Terminates the metro viewer process if there is one.
+  virtual void TerminateMetroViewerProcess() = 0;

[sky, 2013/05/14 17:17:51]

Move these two into a win-aura specific BrowserProcessPlatformPart. There isn't
one yet, so you'll have to create it. Follow the pattern of the chromeos
specific one.

[gab, 2013/05/14 17:45:25]

These methods need to have access to metro_viewer_process_host_ which is a
member of BrowserProcessImpl.

The only way I can think of to move these into a win-aura specific
BrowserProcessPlatformPart would be to pass a pointer the this pointer of
BrowserProcessImpl when constructing a BrowserProcessPlatformPart and making
BrowserProcessPlatformPart a friend of BrowserProcessImpl (or publicly expose a
non-const getter to the internal pointer =S), but that feels like a worst hack
than the current one...

WDYT?

[sky, 2013/05/14 20:47:45]

Can metro_viewer_process_host_ be moved too? Ideally this file would have no
ifdefs.

[gab, 2013/05/15 17:13:23]

Of course, not sure why I thought that wouldn't work... anyways did it in a
separate CL (https://codereview.chromium.org/14698027/) and rebased this one on
top of that one.

PTAL.

Thanks!
Gab

 #endif
 
  private: