GTK: Even more cleanup to access allocation through the accessor.

chrome/browser/ui/gtk/menu_bar_helper.cc

 // 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.
 
 #include "chrome/browser/ui/gtk/menu_bar_helper.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "chrome/browser/ui/gtk/gtk_util.h"
 #include "ui/base/gtk/gtk_signal_registrar.h"
 
 namespace {
 
 // Recursively find all the GtkMenus that are attached to menu item |child|
 // and add them to |data|, which is a vector of GtkWidgets.
 void PopulateSubmenus(GtkWidget* child, gpointer data) {
   std::vector<GtkWidget*>* submenus =
       static_cast<std::vector<GtkWidget*>*>(data);
   GtkWidget* submenu = gtk_menu_item_get_submenu(GTK_MENU_ITEM(child));
   if (submenu) {
     submenus->push_back(submenu);
     gtk_container_foreach(GTK_CONTAINER(submenu), PopulateSubmenus, submenus);
   }
 }
 
 // Is the cursor over |menu| or one of its parent menus?
 bool MotionIsOverMenu(GtkWidget* menu, GdkEventMotion* motion) {
+  GtkAllocation allocation;
+  gtk_widget_get_allocation(menu, &allocation);
+
   if (motion->x >= 0 && motion->y >= 0 &&
-      motion->x < menu->allocation.width &&
-      motion->y < menu->allocation.height) {
+      motion->x < allocation.width &&
+      motion->y < allocation.height) {
     return true;
   }
 
   while (menu) {
     GtkWidget* menu_item = gtk_menu_get_attach_widget(GTK_MENU(menu));
     if (!menu_item)
       return false;
     GtkWidget* parent = gtk_widget_get_parent(menu_item);
 
     if (gtk_util::WidgetContainsCursor(parent))
@@ skipping 82 matching lines @@
       gint last_y = y;
       if (!gtk_widget_translate_coordinates(
           last_button, button, last_x, last_y, &x, &y)) {
         // |button| may not be realized.
         continue;
       }
     }
 
     last_button = button;
 
-    if (x >= 0 && y >= 0 && x < button->allocation.width &&
-        y < button->allocation.height) {
+    GtkAllocation allocation;
+    gtk_widget_get_allocation(button, &allocation);
+
+    if (x >= 0 && y >= 0 && x < allocation.width && y < allocation.height) {
       if (button != button_showing_menu_)
         delegate_->PopupForButton(button);
       return TRUE;
     }
   }
 
   return FALSE;
 }
 
 void MenuBarHelper::OnMenuHiddenOrDestroyed(GtkWidget* menu) {
@@ skipping 28 matching lines @@
       break;
     }
     default:
       return;
   }
 
   // This signal doesn't have a return value; we have to manually stop its
   // propagation.
   g_signal_stop_emission_by_name(menu, "move-current");
 }