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

Unified Diff: chrome/browser/ui/gtk/bubble/bubble_gtk.cc

Issue 9167002: Revert 116956 - GTK: Seal up GSEALs, focusing on GtkSelectionData. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 11 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
« no previous file with comments | « no previous file | chrome/browser/ui/gtk/certificate_viewer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/gtk/bubble/bubble_gtk.cc
===================================================================
--- chrome/browser/ui/gtk/bubble/bubble_gtk.cc (revision 116976)
+++ chrome/browser/ui/gtk/bubble/bubble_gtk.cc (working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
@@ -307,10 +307,8 @@
mask_region_ = gdk_region_polygon(&points[0],
points.size(),
GDK_EVEN_ODD_RULE);
-
- GdkWindow* gdk_window = gtk_widget_get_window(window_);
- gdk_window_shape_combine_region(gdk_window, NULL, 0, 0);
- gdk_window_shape_combine_region(gdk_window, mask_region_, 0, 0);
+ gdk_window_shape_combine_region(window_->window, NULL, 0, 0);
+ gdk_window_shape_combine_region(window_->window, mask_region_, 0, 0);
}
void BubbleGtk::MoveWindow() {
@@ -375,12 +373,10 @@
}
void BubbleGtk::GrabPointerAndKeyboard() {
- GdkWindow* gdk_window = gtk_widget_get_window(window_);
-
// Install X pointer and keyboard grabs to make sure that we have the focus
// and get all mouse and keyboard events until we're closed.
GdkGrabStatus pointer_grab_status =
- gdk_pointer_grab(gdk_window,
+ gdk_pointer_grab(window_->window,
TRUE, // owner_events
GDK_BUTTON_PRESS_MASK, // event_mask
NULL, // confine_to
@@ -393,7 +389,7 @@
<< pointer_grab_status << ")";
}
GdkGrabStatus keyboard_grab_status =
- gdk_keyboard_grab(gdk_window,
+ gdk_keyboard_grab(window_->window,
FALSE, // owner_events
GDK_CURRENT_TIME);
if (keyboard_grab_status != GDK_GRAB_SUCCESS) {
@@ -460,7 +456,6 @@
}
gboolean BubbleGtk::OnExpose(GtkWidget* widget, GdkEventExpose* expose) {
- // TODO(erg): This whole method will need to be rewritten in cairo.
GdkDrawable* drawable = GDK_DRAWABLE(window_->window);
GdkGC* gc = gdk_gc_new(drawable);
gdk_gc_set_rgb_fg_color(gc, &kFrameColor);
« no previous file with comments | « no previous file | chrome/browser/ui/gtk/certificate_viewer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698