Rework the way Widget::Init works:

views/controls/menu/menu_host_gtk.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 "views/controls/menu/menu_host_gtk.h"
 
 #include <gdk/gdk.h>
 
 #if defined(HAVE_XINPUT2) && defined(TOUCH_UI)
 #include <gdk/gdkx.h>
 #include <X11/extensions/XInput2.h>
 #endif
 
 #include "views/controls/menu/native_menu_host_delegate.h"
 
 #if defined(HAVE_XINPUT2) && defined(TOUCH_UI)
 #include "views/touchui/touch_factory.h"
 #endif
 
 namespace views {
 
 ////////////////////////////////////////////////////////////////////////////////
 // MenuHostGtk, public:
 
 MenuHostGtk::MenuHostGtk(internal::NativeMenuHostDelegate* delegate)
-    : WidgetGtk(WidgetGtk::TYPE_POPUP),
-      did_input_grab_(false),
+    : did_input_grab_(false),
       delegate_(delegate) {
 }
 
 MenuHostGtk::~MenuHostGtk() {
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // MenuHostGtk, NativeMenuHost implementation:
 
-void MenuHostGtk::InitMenuHost(gfx::NativeWindow parent,
-                               const gfx::Rect& bounds) {
-  make_transient_to_parent();
-  WidgetGtk::Init(GTK_WIDGET(parent), bounds);
-  // Make sure we get destroyed when the parent is destroyed.
-  gtk_window_set_destroy_with_parent(GTK_WINDOW(GetNativeView()), TRUE);
-  gtk_window_set_type_hint(GTK_WINDOW(GetNativeView()),
-                           GDK_WINDOW_TYPE_HINT_MENU);
-}
-
 void MenuHostGtk::StartCapturing() {
   DCHECK(!did_input_grab_);
 
   // Release the current grab.
   GtkWidget* current_grab_window = gtk_grab_get_current();
   if (current_grab_window)
     gtk_grab_remove(current_grab_window);
 
   // Make sure all app mouse/keyboard events are targeted at us only.
   SetMouseCapture();
@@ skipping 28 matching lines @@
   // need keyboard grab.
 }
 
 NativeWidget* MenuHostGtk::AsNativeWidget() {
   return this;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // MenuHostGtk, WidgetGtk overrides:
 
+void MenuHostGtk::InitNativeWidget(const Widget::CreateParams& params) {
+  make_transient_to_parent();
+  WidgetGtk::InitNativeWidget(params);
+  // Make sure we get destroyed when the parent is destroyed.
+  gtk_window_set_destroy_with_parent(GTK_WINDOW(GetNativeView()), TRUE);
+  gtk_window_set_type_hint(GTK_WINDOW(GetNativeView()),
+                           GDK_WINDOW_TYPE_HINT_MENU);
+}
+
 // TODO(beng): remove once MenuHost is-a Widget
 RootView* MenuHostGtk::CreateRootView() {
   return delegate_->CreateRootView();
 }
 
 bool MenuHostGtk::ShouldReleaseCaptureOnMouseReleased() const {
   return delegate_->ShouldReleaseCaptureOnMouseRelease();
 }
 
 void MenuHostGtk::ReleaseMouseCapture() {
@@ skipping 35 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 // NativeMenuHost, public:
 
 // static
 NativeMenuHost* NativeMenuHost::CreateNativeMenuHost(
     internal::NativeMenuHostDelegate* delegate) {
   return new MenuHostGtk(delegate);
 }
 
 }  // namespace views