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

Unified Diff: chrome/browser/renderer_host/render_widget_host_view_gtk.cc

Issue 6024008: Consider the popup window position when the window shows upward. This patch depends on WebKit patch. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Fix format errors. Created 9 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/renderer_host/render_widget_host_view_gtk.cc
diff --git a/chrome/browser/renderer_host/render_widget_host_view_gtk.cc b/chrome/browser/renderer_host/render_widget_host_view_gtk.cc
index f6b16188d3d285daf0917e7387880e6463151d7a..cff39cbb1224b38cb4aa73111fcded74109d5c9b 100644
--- a/chrome/browser/renderer_host/render_widget_host_view_gtk.cc
+++ b/chrome/browser/renderer_host/render_widget_host_view_gtk.cc
@@ -602,7 +602,6 @@ void RenderWidgetHostViewGtk::WasHidden() {
}
void RenderWidgetHostViewGtk::SetSize(const gfx::Size& size) {
- // This is called when webkit has sent us a Move message.
int width = std::min(size.width(), kMaxWindowWidth);
int height = std::min(size.height(), kMaxWindowHeight);
if (IsPopup()) {
@@ -616,6 +615,7 @@ void RenderWidgetHostViewGtk::SetSize(const gfx::Size& size) {
gtk_widget_set_size_request(view_.get(), width, height);
#endif
}
+
// Update the size of the RWH.
if (requested_size_.width() != width ||
requested_size_.height() != height) {
@@ -624,6 +624,16 @@ void RenderWidgetHostViewGtk::SetSize(const gfx::Size& size) {
}
}
+void RenderWidgetHostViewGtk::SetBounds(const gfx::Rect& rect) {
+ // This is called when webkit has sent us a Move message.
+ if (IsPopup()) {
+ gtk_window_move(GTK_WINDOW(gtk_widget_get_toplevel(view_.get())),
+ rect.x(), rect.y());
+ }
+
+ SetSize(rect.size());
+}
+
gfx::NativeView RenderWidgetHostViewGtk::GetNativeView() {
return view_.get();
}

Powered by Google App Engine
This is Rietveld 408576698