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

Side by Side Diff: chrome/browser/ui/gtk/constrained_window_gtk.cc

Issue 9586009: Move TabContentsViewGtk's delegate to content\public\browser. Also name it a delegate instead of a … (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 9 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
« no previous file with comments | « chrome/browser/ui/gtk/constrained_window_gtk.h ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/gtk/constrained_window_gtk.h" 5 #include "chrome/browser/ui/gtk/constrained_window_gtk.h"
6 6
7 #include <gdk/gdkkeysyms.h> 7 #include <gdk/gdkkeysyms.h>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/message_loop.h" 10 #include "base/message_loop.h"
11 #include "chrome/browser/ui/browser_list.h" 11 #include "chrome/browser/ui/browser_list.h"
12 #include "chrome/browser/ui/constrained_window_tab_helper.h" 12 #include "chrome/browser/ui/constrained_window_tab_helper.h"
13 #include "chrome/browser/ui/gtk/gtk_util.h" 13 #include "chrome/browser/ui/gtk/gtk_util.h"
14 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h" 14 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
15 #include "content/public/browser/browser_thread.h" 15 #include "content/public/browser/browser_thread.h"
16 #include "content/public/browser/web_contents.h" 16 #include "content/public/browser/web_contents.h"
17 #include "ui/base/gtk/gtk_compat.h" 17 #include "ui/base/gtk/gtk_compat.h"
18 #include "ui/base/gtk/gtk_hig_constants.h" 18 #include "ui/base/gtk/gtk_hig_constants.h"
19 19
20 #if defined(TOOLKIT_VIEWS) 20 #if defined(TOOLKIT_VIEWS)
21 #include "chrome/browser/ui/views/tab_contents/native_tab_contents_view_gtk.h" 21 #include "chrome/browser/ui/views/tab_contents/native_tab_contents_view_gtk.h"
22 #include "chrome/browser/ui/views/tab_contents/tab_contents_view_views.h" 22 #include "chrome/browser/ui/views/tab_contents/tab_contents_view_views.h"
23 #else 23 #else
24 #include "chrome/browser/tab_contents/chrome_tab_contents_view_wrapper_gtk.h" 24 #include "chrome/browser/tab_contents/chrome_web_contents_view_gtk_delegate.h"
25 #include "content/browser/tab_contents/tab_contents_view_gtk.h" 25 #include "content/browser/tab_contents/tab_contents_view_gtk.h"
26 #endif 26 #endif
27 27
28 using content::BrowserThread; 28 using content::BrowserThread;
29 29
30 ConstrainedWindowGtkDelegate::~ConstrainedWindowGtkDelegate() { 30 ConstrainedWindowGtkDelegate::~ConstrainedWindowGtkDelegate() {
31 } 31 }
32 32
33 bool ConstrainedWindowGtkDelegate::GetBackgroundColor(GdkColor* color) { 33 bool ConstrainedWindowGtkDelegate::GetBackgroundColor(GdkColor* color) {
34 return false; 34 return false;
(...skipping 99 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 134
135 ConstrainedWindowGtk::TabContentsViewType* 135 ConstrainedWindowGtk::TabContentsViewType*
136 ConstrainedWindowGtk::ContainingView() { 136 ConstrainedWindowGtk::ContainingView() {
137 #if defined(TOOLKIT_VIEWS) 137 #if defined(TOOLKIT_VIEWS)
138 return static_cast<NativeTabContentsViewGtk*>( 138 return static_cast<NativeTabContentsViewGtk*>(
139 static_cast<TabContentsViewViews*>(wrapper_->web_contents()->GetView())-> 139 static_cast<TabContentsViewViews*>(wrapper_->web_contents()->GetView())->
140 native_tab_contents_view()); 140 native_tab_contents_view());
141 #else 141 #else
142 return static_cast<TabContentsViewType*>( 142 return static_cast<TabContentsViewType*>(
143 static_cast<content::TabContentsViewGtk*>( 143 static_cast<content::TabContentsViewGtk*>(
144 wrapper_->web_contents()->GetView())->wrapper()); 144 wrapper_->web_contents()->GetView())->delegate());
145 #endif 145 #endif
146 } 146 }
147 147
148 gboolean ConstrainedWindowGtk::OnKeyPress(GtkWidget* sender, 148 gboolean ConstrainedWindowGtk::OnKeyPress(GtkWidget* sender,
149 GdkEventKey* key) { 149 GdkEventKey* key) {
150 if (key->keyval == GDK_Escape) { 150 if (key->keyval == GDK_Escape) {
151 // Let the stack unwind so the event handler can release its ref 151 // Let the stack unwind so the event handler can release its ref
152 // on widget(). 152 // on widget().
153 MessageLoop::current()->PostTask( 153 MessageLoop::current()->PostTask(
154 FROM_HERE, 154 FROM_HERE,
155 base::Bind(&ConstrainedWindowGtk::CloseConstrainedWindow, 155 base::Bind(&ConstrainedWindowGtk::CloseConstrainedWindow,
156 weak_factory_.GetWeakPtr())); 156 weak_factory_.GetWeakPtr()));
157 return TRUE; 157 return TRUE;
158 } 158 }
159 159
160 return FALSE; 160 return FALSE;
161 } 161 }
162 162
163 void ConstrainedWindowGtk::OnHierarchyChanged(GtkWidget* sender, 163 void ConstrainedWindowGtk::OnHierarchyChanged(GtkWidget* sender,
164 GtkWidget* previous_toplevel) { 164 GtkWidget* previous_toplevel) {
165 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 165 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
166 if (!gtk_widget_is_toplevel(gtk_widget_get_toplevel(widget()))) 166 if (!gtk_widget_is_toplevel(gtk_widget_get_toplevel(widget())))
167 return; 167 return;
168 168
169 FocusConstrainedWindow(); 169 FocusConstrainedWindow();
170 } 170 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/gtk/constrained_window_gtk.h ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698