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

Unified Diff: chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc

Issue 10918275: [gtk] website settings bubble (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: slightly simpler drawing Created 8 years, 3 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/ui/gtk/extensions/extension_popup_gtk.cc
diff --git a/chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc b/chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc
index a7f5509dd7835087479f3d83662027027db5b282..8ec192ae230a532df69df2b9eecd3d6c10a87e3f 100644
--- a/chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc
+++ b/chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc
@@ -175,6 +175,12 @@ void ExtensionPopupGtk::ShowPopup() {
current_extension_popup_->DestroyPopup();
current_extension_popup_ = this;
+ GtkWidget* border_box = gtk_alignment_new(0, 0, 1.0, 1.0);
+ // This border is necessary so the bubble's corners do not get cut off by the
+ // render view.
+ gtk_container_set_border_width(GTK_CONTAINER(border_box), 2);
+ gtk_container_add(GTK_CONTAINER(border_box), host_->view()->native_view());
+
// We'll be in the upper-right corner of the window for LTR languages, so we
// want to put the arrow at the upper-right corner of the bubble to match the
// page and app menus.
@@ -184,7 +190,7 @@ void ExtensionPopupGtk::ShowPopup() {
BubbleGtk::ARROW_LOCATION_TOP_LEFT;
bubble_ = BubbleGtk::Show(anchor_,
NULL,
- host_->view()->native_view(),
+ border_box,
arrow_location,
being_inspected_ ? 0 :
BubbleGtk::POPUP_WINDOW | BubbleGtk::GRAB_INPUT,
« no previous file with comments | « chrome/browser/ui/gtk/extensions/extension_installed_bubble_gtk.cc ('k') | chrome/browser/ui/gtk/first_run_bubble.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698