Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(675)

Side by Side Diff: chrome/browser/ui/views/frame/browser_desktop_root_window_host_x11.cc

Issue 31043006: Moves creation of various clients to DesktopNativeWidgetAura (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: resolve merge Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/frame/browser_desktop_root_window_host_x11.h" 5 #include "chrome/browser/ui/views/frame/browser_desktop_root_window_host_x11.h"
6 6
7 //////////////////////////////////////////////////////////////////////////////// 7 ////////////////////////////////////////////////////////////////////////////////
8 // BrowserDesktopRootWindowHostX11, public: 8 // BrowserDesktopRootWindowHostX11, public:
9 9
10 BrowserDesktopRootWindowHostX11::BrowserDesktopRootWindowHostX11( 10 BrowserDesktopRootWindowHostX11::BrowserDesktopRootWindowHostX11(
11 views::internal::NativeWidgetDelegate* native_widget_delegate, 11 views::internal::NativeWidgetDelegate* native_widget_delegate,
12 views::DesktopNativeWidgetAura* desktop_native_widget_aura, 12 views::DesktopNativeWidgetAura* desktop_native_widget_aura,
13 const gfx::Rect& initial_bounds,
14 BrowserView* browser_view) 13 BrowserView* browser_view)
15 : DesktopRootWindowHostX11(native_widget_delegate, 14 : DesktopRootWindowHostX11(native_widget_delegate,
16 desktop_native_widget_aura, 15 desktop_native_widget_aura),
17 initial_bounds),
18 browser_view_(browser_view) { 16 browser_view_(browser_view) {
19 } 17 }
20 18
21 BrowserDesktopRootWindowHostX11::~BrowserDesktopRootWindowHostX11() { 19 BrowserDesktopRootWindowHostX11::~BrowserDesktopRootWindowHostX11() {
22 } 20 }
23 21
24 //////////////////////////////////////////////////////////////////////////////// 22 ////////////////////////////////////////////////////////////////////////////////
25 // BrowserDesktopRootWindowHostX11, 23 // BrowserDesktopRootWindowHostX11,
26 // BrowserDesktopRootWindowHost implementation: 24 // BrowserDesktopRootWindowHost implementation:
27 25
28 views::DesktopRootWindowHost* 26 views::DesktopRootWindowHost*
29 BrowserDesktopRootWindowHostX11::AsDesktopRootWindowHost() { 27 BrowserDesktopRootWindowHostX11::AsDesktopRootWindowHost() {
30 return this; 28 return this;
31 } 29 }
32 30
33 int BrowserDesktopRootWindowHostX11::GetMinimizeButtonOffset() const { 31 int BrowserDesktopRootWindowHostX11::GetMinimizeButtonOffset() const {
34 return 0; 32 return 0;
35 } 33 }
36 34
37 bool BrowserDesktopRootWindowHostX11::UsesNativeSystemMenu() const { 35 bool BrowserDesktopRootWindowHostX11::UsesNativeSystemMenu() const {
38 return false; 36 return false;
39 } 37 }
40 38
41 //////////////////////////////////////////////////////////////////////////////// 39 ////////////////////////////////////////////////////////////////////////////////
42 // BrowserDesktopRootWindowHostX11, 40 // BrowserDesktopRootWindowHostX11,
43 // views::DesktopRootWindowHostX11 implementation: 41 // views::DesktopRootWindowHostX11 implementation:
44 42
45 aura::RootWindow* BrowserDesktopRootWindowHostX11::Init( 43 void BrowserDesktopRootWindowHostX11::Init(
46 aura::Window* content_window, 44 aura::Window* content_window,
47 const views::Widget::InitParams& params) { 45 const views::Widget::InitParams& params,
48 aura::RootWindow* root_window = views::DesktopRootWindowHostX11::Init( 46 aura::RootWindow::CreateParams* rw_create_params) {
49 content_window, params); 47 views::DesktopRootWindowHostX11::Init(content_window, params,
48 rw_create_params);
50 49
51 // We have now created our backing X11 window. We now need to (possibly) 50 // We have now created our backing X11 window. We now need to (possibly)
52 // alert Unity that there's a menu bar attached to it. 51 // alert Unity that there's a menu bar attached to it.
53 global_menu_bar_x11_.reset(new GlobalMenuBarX11(browser_view_, this)); 52 global_menu_bar_x11_.reset(new GlobalMenuBarX11(browser_view_, this));
54
55 return root_window;
56 } 53 }
57 54
58 void BrowserDesktopRootWindowHostX11::CloseNow() { 55 void BrowserDesktopRootWindowHostX11::CloseNow() {
59 global_menu_bar_x11_.reset(); 56 global_menu_bar_x11_.reset();
60 DesktopRootWindowHostX11::CloseNow(); 57 DesktopRootWindowHostX11::CloseNow();
61 } 58 }
62 59
63 //////////////////////////////////////////////////////////////////////////////// 60 ////////////////////////////////////////////////////////////////////////////////
64 // BrowserDesktopRootWindowHost, public: 61 // BrowserDesktopRootWindowHost, public:
65 62
66 // static 63 // static
67 BrowserDesktopRootWindowHost* 64 BrowserDesktopRootWindowHost*
68 BrowserDesktopRootWindowHost::CreateBrowserDesktopRootWindowHost( 65 BrowserDesktopRootWindowHost::CreateBrowserDesktopRootWindowHost(
69 views::internal::NativeWidgetDelegate* native_widget_delegate, 66 views::internal::NativeWidgetDelegate* native_widget_delegate,
70 views::DesktopNativeWidgetAura* desktop_native_widget_aura, 67 views::DesktopNativeWidgetAura* desktop_native_widget_aura,
71 const gfx::Rect& initial_bounds,
72 BrowserView* browser_view, 68 BrowserView* browser_view,
73 BrowserFrame* browser_frame) { 69 BrowserFrame* browser_frame) {
74 return new BrowserDesktopRootWindowHostX11(native_widget_delegate, 70 return new BrowserDesktopRootWindowHostX11(native_widget_delegate,
75 desktop_native_widget_aura, 71 desktop_native_widget_aura,
76 initial_bounds,
77 browser_view); 72 browser_view);
78 } 73 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698