alternate ntp: add "Recent Tabs" submenu to wrench menu

chrome/browser/ui/views/toolbar_view.cc

Index: chrome/browser/ui/views/toolbar_view.cc
diff --git a/chrome/browser/ui/views/toolbar_view.cc b/chrome/browser/ui/views/toolbar_view.cc
index 7f91d3f070787f22b800dcf15da894ce5ab21c7f..7ab1c1917f416a7b04bc54a78a7acb118f5efbae 100644
--- a/chrome/browser/ui/views/toolbar_view.cc
+++ b/chrome/browser/ui/views/toolbar_view.cc
@@ -422,8 +422,8 @@ void ToolbarView::OnMenuButtonClicked(views::View* source,
                                       const gfx::Point& point) {
   DCHECK_EQ(VIEW_ID_APP_MENU, source->id());
 
-  wrench_menu_.reset(new WrenchMenu(browser_));
   wrench_menu_model_.reset(new WrenchMenuModel(this, browser_));

[sky, 2012/11/07 00:10:47]

This order goes against your comment in the header.

[kuan, 2012/11/07 02:29:46]

my comment in header said wrench_menu_model_ shld be listed later so that it's
destructed earlier.  so here in construction, shldn't it be constructed earlier?

[kuan, 2012/11/07 16:48:52]

now, i understand what u mean; u're right, i shld construct it later.  i forgot
that scoped_ptr::reset will destruct the previous pointer b4 constructing new
one, so i shld reset model first.

+  wrench_menu_.reset(new WrenchMenu(browser_));
   wrench_menu_->Init(wrench_menu_model_.get());
 
   FOR_EACH_OBSERVER(views::MenuListener, menu_listeners_, OnMenuOpened());