Do not create display_change_listener for parent=NULL.

ui/views/controls/menu/menu_runner.cc

 // Copyright (c) 2012 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 "ui/views/controls/menu/menu_runner.h"
 
 #include <set>
 
 #include "ui/views/controls/button/menu_button.h"
 #include "ui/views/controls/menu/menu_controller.h"
@@ skipping 266 matching lines @@
 }
 
 MenuRunner::RunResult MenuRunner::RunMenuAt(Widget* parent,
                                             MenuButton* button,
                                             const gfx::Rect& bounds,
                                             MenuItemView::AnchorPosition anchor,
                                             int32 types) {
   // The parent of the nested menu will have created a DisplayChangeListener, so
   // we avoid creating a DisplayChangeListener if nested. Drop menus are
   // transient, so we don't cancel in that case.
-  if ((types & (IS_NESTED | FOR_DROP)) == 0) {
+  if ((types & (IS_NESTED | FOR_DROP)) == 0 && parent != NULL) {

[Ben Goodger (Google), 2013/03/11 16:10:07]

nit && parent

[Jun Mukai, 2013/03/11 16:42:29]

Done.

     display_change_listener_.reset(
         internal::DisplayChangeListener::Create(parent, this));
   }
   if ((types & MenuRunner::CONTEXT_MENU) &&
       parent &&
       parent->GetCurrentEvent() &&
       !MenuItemView::IsBubble(anchor))
     anchor = parent->GetCurrentEvent()->IsGestureEvent() ?
         MenuItemView::BOTTOMCENTER : MenuItemView::TOPLEFT;
 
   return holder_->RunMenuAt(parent, button, bounds, anchor, types);
 }
 
 bool MenuRunner::IsRunning() const {
   return holder_->running();
 }
 
 void MenuRunner::Cancel() {
   holder_->Cancel();
 }
 
 }  // namespace views