Make it possible to launch chrome --mash with ozone_platform={wayland|x11}, chromeos=0

chrome/browser/ui/views/frame/native_browser_frame_factory_auralinux.cc

Index: chrome/browser/ui/views/frame/native_browser_frame_factory_auralinux.cc
diff --git a/chrome/browser/ui/views/frame/native_browser_frame_factory_auralinux.cc b/chrome/browser/ui/views/frame/native_browser_frame_factory_auralinux.cc
index 7515c037292a73af9769b0c19ce9dc311ceb9b20..cce91a0fe2af4c7fcb0bbe93be1389312c128344 100644
--- a/chrome/browser/ui/views/frame/native_browser_frame_factory_auralinux.cc
+++ b/chrome/browser/ui/views/frame/native_browser_frame_factory_auralinux.cc
@@ -13,5 +13,10 @@ NativeBrowserFrame* NativeBrowserFrameFactory::Create(
     BrowserView* browser_view) {
   if (shell::ShellIsRemote())
     return new BrowserFrameMus(browser_frame, browser_view);
+
+#if !defined(USE_OZONE)

[rjkroege, 2016/10/11 18:49:47]

this looks strange. i.e.: something is not right here but I'm not the right
person. +sadrul to comment more about how this should fit?

[sadrul, 2016/10/11 19:14:14]

What happens if you use this piece of code with ozone?

[tonikitoo, 2016/10/12 13:44:23]

In patchset #3, in if you look at chrome/browser/ui/BUILD.gn ,
views/frame/desktop_browser_frame_auralinux.cc is not built for 'use_ozone'.

Some reasons:

- desktop_browser_frame_auralinux.cc contains some X11 code. Well, we could
maybe ifdef it out, but decided not to for now.

- DesktopBrowserFrameAuraLinux ctor calls out
BrowserDesktopWindowTreeHost::CreateBrowserDesktopWindowTreeHost which is only
implemented for win and x11 in
chrome/browser/ui/views/frame/browser_desktop_window_tree_host_{win,x11}.cc

My understanding is that for "mus" we do not want this method implemented.

All in all, since we are targeting launching with "--mash", we would enter the
"if" clause in line 14 above.

patchset 4 adds NOTREACHED with some text to explain it.

   return new DesktopBrowserFrameAuraLinux(browser_frame, browser_view);
+#else
+  return nullptr;
+#endif
 }