views: Support Desktop Aura creation on Ozone

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

Index: chrome/browser/ui/views/frame/browser_desktop_root_window_host_ozone.cc
diff --git a/chrome/browser/ui/views/frame/browser_desktop_root_window_host_ozone.cc b/chrome/browser/ui/views/frame/browser_desktop_root_window_host_ozone.cc
new file mode 100644
index 0000000000000000000000000000000000000000..dc4a06d1e85b60ea2ed3a707d06bda637d5d57e3
--- /dev/null
+++ b/chrome/browser/ui/views/frame/browser_desktop_root_window_host_ozone.cc
@@ -0,0 +1,70 @@
+// Copyright (c) 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.
+
+#include "chrome/browser/ui/views/frame/browser_desktop_root_window_host_ozone.h"
+
+////////////////////////////////////////////////////////////////////////////////
+// BrowserDesktopRootWindowHostOzone, public:
+
+BrowserDesktopRootWindowHostOzone::BrowserDesktopRootWindowHostOzone(
+    views::internal::NativeWidgetDelegate* native_widget_delegate,
+    views::DesktopNativeWidgetAura* desktop_native_widget_aura,
+    BrowserView* browser_view)
+      : browser_view_(browser_view),
+      desktop_root_window_host_(NULL) {
+  desktop_root_window_host_ = views::DesktopRootWindowHost::Create(
+      native_widget_delegate,
+      desktop_native_widget_aura);
+}
+
+BrowserDesktopRootWindowHostOzone::~BrowserDesktopRootWindowHostOzone() {
+  desktop_root_window_host_ = NULL;
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// BrowserDesktopRootWindowHostOzone,
+//     BrowserDesktopRootWindowHost implementation:
+
+views::DesktopRootWindowHost*
+    BrowserDesktopRootWindowHostOzone::AsDesktopRootWindowHost() {
+  return desktop_root_window_host_;

[Elliot Glaysher, 2013/10/28 19:50:00]

You haven't fixed the major problem here. You're returning an object that isn't
this. The whole point of this method is to downcast this object. Returning a
different object here is probably *really* wrong.

[vignatti (out of this project), 2013/10/29 16:08:38]

I see now.

rjkroege@, it all boils down to the fact that we don't have a
DesktopRootWindowHostOzone class and therefore we cannot make
BrowserDesktopRootWindowHostOzone inherit it just like in the other platforms.
Now I'm not very sure the consequences of simply adding that class in views, and
specially the impact for the downstream Ozone implementations. I need you input
here. Thank you.

+}
+
+int BrowserDesktopRootWindowHostOzone::GetMinimizeButtonOffset() const {
+  return 0;
+}
+
+bool BrowserDesktopRootWindowHostOzone::UsesNativeSystemMenu() const {
+  return false;
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// BrowserDesktopRootWindowHostOzone,
+//     views::DesktopRootWindowHostOzone implementation:
+
+void BrowserDesktopRootWindowHostOzone::Init(
+    aura::Window* content_window,
+    const views::Widget::InitParams& params,
+    aura::RootWindow::CreateParams* rw_create_params) {
+  desktop_root_window_host_->Init(content_window, params, rw_create_params);
+}
+
+void BrowserDesktopRootWindowHostOzone::CloseNow() {
+  desktop_root_window_host_->CloseNow();
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// BrowserDesktopRootWindowHost, public:
+
+// static
+BrowserDesktopRootWindowHost*
+    BrowserDesktopRootWindowHost::CreateBrowserDesktopRootWindowHost(
+        views::internal::NativeWidgetDelegate* native_widget_delegate,
+        views::DesktopNativeWidgetAura* desktop_native_widget_aura,
+        BrowserView* browser_view,
+        BrowserFrame* browser_frame) {
+  return new BrowserDesktopRootWindowHostOzone(native_widget_delegate,
+                                               desktop_native_widget_aura,
+                                               browser_view);
+}