[win] Initialize the app list menu lazily, and allow invalidation.

ui/app_list/views/app_list_main_view.cc

Index: ui/app_list/views/app_list_main_view.cc
diff --git a/ui/app_list/views/app_list_main_view.cc b/ui/app_list/views/app_list_main_view.cc
index d3791ad9c9a5a556b9046efcb325ea13c1cc1be2..1a8fecdb75bedaeb7c2c13d3e70307070176fac7 100644
--- a/ui/app_list/views/app_list_main_view.cc
+++ b/ui/app_list/views/app_list_main_view.cc
@@ -90,7 +90,7 @@ AppListMainView::AppListMainView(AppListViewDelegate* delegate,
                                         kInnerPadding,
                                         kInnerPadding));
 
-  search_box_view_ = new SearchBoxView(this, delegate);
+  search_box_view_ = new SearchBoxView(this);

[benwells, 2013/07/23 01:10:43]

Should you update the menu here?

[tapted, 2013/07/23 01:16:47]

I was doing this in patchset 1, but realised it would always be reset again,
because AppListView::InitAsBubble(..) calls OnSigninStatusChanged()

[benwells, 2013/07/23 01:23:26]

I'd prefer not to have distant parts of the code rely on each other like that,
it's the kind of thing that can change easily and cause unexpected problems.

   AddChildView(search_box_view_);
 
   contents_view_ = new ContentsView(this, pagination_model);