Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(209)

Unified Diff: ui/app_list/cocoa/apps_search_box_controller.mm

Issue 25535006: Fix rebuilding the profile selector on the OSX App Launcher. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/app_list/cocoa/apps_search_box_controller.mm
diff --git a/ui/app_list/cocoa/apps_search_box_controller.mm b/ui/app_list/cocoa/apps_search_box_controller.mm
index 2c806324f8c1eabd28cb5f18256df6832f4ec61a..c31b865308c95c921ffd230dbaa036b131e4a2b7 100644
--- a/ui/app_list/cocoa/apps_search_box_controller.mm
+++ b/ui/app_list/cocoa/apps_search_box_controller.mm
@@ -159,6 +159,9 @@ void SearchBoxModelObserverBridge::TextChanged() {
if (![delegate_ appListDelegate])
return;
+ appListMenu_.reset(
+ new app_list::AppListMenu([delegate_ appListDelegate],
+ [delegate_ appListModel]->users()));
menuController_.reset([[AppListMenuController alloc]
initWithSearchBoxController:self]);
[menuButton_ setMenu:[menuController_ menu]]; // Menu will populate here.
@@ -174,12 +177,6 @@ void SearchBoxModelObserverBridge::TextChanged() {
return;
bridge_.reset(new app_list::SearchBoxModelObserverBridge(self));
- if (![delegate_ appListDelegate])
- return;
-
- appListMenu_.reset(
- new app_list::AppListMenu([delegate_ appListDelegate],
- [delegate_ appListModel]->users()));
[self rebuildMenu];
}

Powered by Google App Engine
This is Rietveld 408576698