Restructure the wrench menu into an action based order

chrome/browser/ui/toolbar/wrench_menu_model.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_TOOLBAR_WRENCH_MENU_MODEL_H_
 #define CHROME_BROWSER_UI_TOOLBAR_WRENCH_MENU_MODEL_H_
 
 #include "base/files/file_path.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/time/time.h"
@@ skipping 172 matching lines @@
 
  private:
   class HelpMenuModel;
   // Testing constructor used for mocking.
   friend class ::MockWrenchMenuModel;
 
   WrenchMenuModel();
 
   void Build();
 
-  void AddGlobalErrorMenuItems();
+  bool AddGlobalErrorMenuItems();

[Peter Kasting, 2015/03/30 19:37:02]

Nit: Please add a comment explaining what this function does, in particular the
return value.

[edwardjung, 2015/03/31 11:44:05]

Done.

 
   // Appends everything needed for the clipboard menu: a menu break, the
   // clipboard menu content and the finalizing menu break.
   void CreateCutCopyPasteMenu();
 
   // Add a menu item for the extension icons.
   void CreateExtensionToolbarOverflowMenu();
 
   // Appends everything needed for the zoom menu: a menu break, then the zoom
   // menu content and then another menu break.
@@ skipping 46 matching lines @@
   Browser* browser_;  // weak
   TabStripModel* tab_strip_model_; // weak
 
   scoped_ptr<content::HostZoomMap::Subscription> browser_zoom_subscription_;
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(WrenchMenuModel);
 };
 
 #endif  // CHROME_BROWSER_UI_TOOLBAR_WRENCH_MENU_MODEL_H_