Fixes possible crash in showing bookmark menu. The problem occurred...

views/controls/menu/menu_item_view.h

 // Copyright (c) 2006-2008 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.
 
 #ifndef VIEWS_CONTROLS_MENU_MENU_ITEM_VIEW_H_
 #define VIEWS_CONTROLS_MENU_MENU_ITEM_VIEW_H_
 
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "views/view.h"
 
@@ skipping 200 matching lines @@
   // Returns the mnemonic for this MenuItemView, or 0 if this MenuItemView
   // doesn't have a mnemonic.
   wchar_t GetMnemonic();
 
   // Do we have icons? This only has effect on the top menu. Turning this on
   // makes the menus slightly wider and taller.
   void set_has_icons(bool has_icons) {
     has_icons_ = has_icons;
   }
 
+  // Returns the descendant with the specified command.
+  MenuItemView* GetMenuItemByID(int id);
+
  protected:
   // Creates a MenuItemView. This is used by the various AddXXX methods.
   MenuItemView(MenuItemView* parent, int command, Type type);
 
  private:
   // Calculates all sizes that we can from the OS.
   //
   // This is invoked prior to Running a menu.
   static void UpdateMenuPartSizes(bool has_icons);
 
   // Called by the two constructors to initialize this menu item.
   void Init(MenuItemView* parent,
             int command,
             MenuItemView::Type type,
             MenuDelegate* delegate);
 
   // All the AddXXX methods funnel into this.
   MenuItemView* AppendMenuItemInternal(int item_id,
                                        const std::wstring& label,
                                        const SkBitmap& icon,
                                        Type type);
 
-  // Returns the descendant with the specified command.
-  MenuItemView* GetDescendantByID(int id);
-
   // Invoked by the MenuController when the menu closes as the result of
   // drag and drop run.
   void DropMenuClosed(bool notify_delegate);
 
   // The RunXXX methods call into this to set up the necessary state before
   // running.
   void PrepareForRun(bool has_mnemonics);
 
   // Returns the flags passed to DrawStringInt.
   int GetDrawStringFlags();
@@ skipping 69 matching lines @@
 
   // Preferred height of menu items. Reset every time a menu is run.
   static int pref_menu_height_;
 
   DISALLOW_COPY_AND_ASSIGN(MenuItemView);
 };
 
 }  // namespace views
 
 #endif  // VIEWS_CONTROLS_MENU_MENU_ITEM_VIEW_H_