Couple more wrench menu issues. The patch I sent around a couple of

views/controls/menu/menu_controller.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_controller.h"
 
 #include "base/i18n/case_conversion.h"
 #include "base/i18n/rtl.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
@@ skipping 1307 matching lines @@
   DCHECK(item);
   if (item->GetSubmenu()->IsShowing()) {
     return;
   }
 
   OpenMenuImpl(item, true);
   did_capture_ = true;
 }
 
 void MenuController::OpenMenuImpl(MenuItemView* item, bool show) {
-  if (show)
+  if (show) {
+    int old_count = item->GetSubmenu()->child_count();
     item->GetDelegate()->WillShowMenu(item);
+    if (old_count != item->GetSubmenu()->child_count()) {
+      // If the number of children changed then we may need to add empty items.
+      item->AddEmptyMenus();
+    }
+  }
   bool prefer_leading =
       state_.open_leading.empty() ? true : state_.open_leading.back();
   bool resulting_direction;
   gfx::Rect bounds =
       CalculateMenuBounds(item, prefer_leading, &resulting_direction);
   state_.open_leading.push_back(resulting_direction);
   bool do_capture = (!did_capture_ && blocking_run_);
   showing_submenu_ = true;
   if (show)
     item->GetSubmenu()->ShowAt(owner_, bounds, do_capture);
@@ skipping 522 matching lines @@
     return;
 
   // Reset the active_mouse_view_ before sending mouse capture lost. That way if
   // it calls back to us, we aren't in a weird state.
   View* active_view = active_mouse_view_;
   active_mouse_view_ = NULL;
   active_view->OnMouseCaptureLost();
 }
 
 }  // namespace views