[win8] Force browsers created with Chrome in Metro mode to be on the Ash desktop.

chrome/browser/ui/views/frame/native_browser_frame_factory.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_NATIVE_BROWSER_FRAME_FACTORY_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_NATIVE_BROWSER_FRAME_FACTORY_H_
 
 #include "base/basictypes.h"
 
 class BrowserFrame;
@@ skipping 12 matching lines @@
   // factory. Use NULL to go back to default factory.
   static void Set(NativeBrowserFrameFactory* new_factory);
 
   virtual NativeBrowserFrame* Create(BrowserFrame* browser_frame,
                                      BrowserView* browser_view);
 
  protected:
   NativeBrowserFrameFactory() {}
   virtual ~NativeBrowserFrameFactory() {}
 
+ private:
+  // For Chrome running on desktop platforms, returns true if the factory should
+  // create an ash browser frame for the provided |browser_view|.
+  static bool ShouldCreateForAshDesktop(BrowserView* browser_view);

[tapted, 2013/11/15 12:47:11]

Note: the current approach leaves this undefined on chromeos, and on use_ash==0.
Alternative is a header like native_browser_frame_factory_nonchromeosash.h or
_ashwinanddesktoplinux.h. Or an #ifdef here, but it's not strictly needed..

[sky, 2013/11/15 17:31:27]

Is there a reason not to move it to native_browser_frame_factory.cc and always
compile it?

[tapted, 2013/11/18 11:13:48]

It needs to pull in some ash-specific files - if that's OK I can merge the
definition from native_browser_frame_factory_desktop_ash.cc into
native_browser_frame_factory.cc. I've now added the missing definition there to
compare (Done.), but it is just NOTREACHED(). I kept the definition out
initially because, without it, the NOTREACHED can be a link error rather than a
runtime error (but maybe that's more confusing).

+
   DISALLOW_COPY_AND_ASSIGN(NativeBrowserFrameFactory);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_NATIVE_BROWSER_FRAME_FACTORY_H_