GTK Themes: Prevent colors from "leaking through" by using an event box.

chrome/browser/gtk/browser_toolbar_gtk.cc

 // Copyright (c) 2009 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.
 
 #include "chrome/browser/gtk/browser_toolbar_gtk.h"
 
 #include <gdk/gdkkeysyms.h>
 #include <X11/XF86keysym.h>
 
 #include "app/l10n_util.h"
@@ skipping 78 matching lines @@
   g_object_unref(accel_group_);
 }
 
 void BrowserToolbarGtk::Init(Profile* profile,
                              GtkWindow* top_level_window) {
   // Make sure to tell the location bar the profile before calling its Init.
   SetProfile(profile);
 
   show_home_button_.Init(prefs::kShowHomeButton, profile->GetPrefs(), this);
 
+  event_box_ = gtk_event_box_new();
+
   toolbar_ = gtk_hbox_new(FALSE, kToolbarWidgetSpacing);
+  gtk_container_add(GTK_CONTAINER(event_box_), toolbar_);
   gtk_container_set_border_width(GTK_CONTAINER(toolbar_), 4);
   // Demand we're always at least kToolbarHeight tall.
   // -1 for width means "let GTK do its normal sizing".
   gtk_widget_set_size_request(toolbar_, -1, kToolbarHeight);
   g_signal_connect(toolbar_, "expose-event",
                    G_CALLBACK(&OnToolbarExpose), this);
 
   // A GtkAccelGroup is not InitiallyUnowned, meaning we get a real reference
   // count starting at one.  We don't want the lifetime to be managed by the
   // top level window, since the lifetime should be tied to the C++ object.
@@ skipping 59 matching lines @@
                    G_CALLBACK(OnPageAppMenuMouseMotion), this);
   g_signal_connect(app_menu_->widget(), "move-current",
                    G_CALLBACK(OnPageAppMenuMoveCurrent), this);
   gtk_box_pack_start(GTK_BOX(menus_hbox_), chrome_menu, FALSE, FALSE, 0);
 
   gtk_box_pack_start(GTK_BOX(toolbar_), menus_hbox_, FALSE, FALSE, 0);
 
   // Force all the CustomDrawButtons to load the correct rendering style.
   UserChangedTheme();
 
-  gtk_widget_show_all(toolbar_);
+  gtk_widget_show_all(event_box_);
 
   if (show_home_button_.GetValue()) {
     gtk_widget_show(home_->widget());
   } else {
     gtk_widget_hide(home_->widget());
   }
 }
 
 void BrowserToolbarGtk::AddToolbarToBox(GtkWidget* box) {
-  gtk_box_pack_start(GTK_BOX(box), toolbar_, FALSE, FALSE, 0);
+  gtk_box_pack_start(GTK_BOX(box), event_box_, FALSE, FALSE, 0);
 }
 
 void BrowserToolbarGtk::Show() {
   gtk_widget_show(toolbar_);
 }
 
 void BrowserToolbarGtk::Hide() {
   gtk_widget_hide(toolbar_);
 }
 
@@ skipping 323 matching lines @@
       break;
   }
 }
 
 void BrowserToolbarGtk::InitNineBox() {
   // TODO(estade): use |profile_|?
   background_ninebox_.reset(new NineBox(
       browser_->profile()->GetThemeProvider(),
       0, IDR_THEME_TOOLBAR, 0, 0, 0, 0, 0, 0, 0));
 }