Fix racing condition that blocks profiles menu showing up

Chrome should show profiles menu when "enable-udd-profiles" is supplied
on command line. This is broken while refactoring app menu code from
ToolbarView into AppMenuModel. Previously, the profile submenu model is
kept in ToolbarView and profiles menu would show up on 2nd time app menu
showing up. After the refactoring, profile menu is moved into
AppMenuModel, which is re-created everytime before we show app menu. In
AppMenuModel, it requests profiles data from GetProfilesHelper which
will later invoke OnGetProfilesDone. However, app menu is created on the
UI thread and blocks the callback until it shows up. This makes the
profiles mneu empty.
The fix is to leverage an existing profile list in BrowserProcess. When
"enable-udd-profiles" is on, BrowserInit will get the initial profile
list and NewProfileDialog will refresh it if user creates new profile.
And AppMenuModel just use the list to populate the profile menu.

BUG=30417
TEST=Verify profiles menu exists when "enable-udd-profiles" is on command line for issue 30417.

Review URL: http://codereview.chromium.org/503062

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35166 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 9 deletions

diff --git a/chrome/browser/app_menu_model.h b/chrome/browser/app_menu_model.h
index d298a86..41343d4 100644
--- a/chrome/browser/app_menu_model.h
+++ b/chrome/browser/app_menu_model.h
@@ -14,26 +14,19 @@ class Browser;
 
 // A menu model that builds the contents of the app menu. This menu has only
 // one level (no submenus).
-class AppMenuModel : public menus::SimpleMenuModel,
-                     public GetProfilesHelper::Delegate {
+class AppMenuModel : public menus::SimpleMenuModel {
  public:
   explicit AppMenuModel(menus::SimpleMenuModel::Delegate* delegate,
                         Browser* browser);
   virtual ~AppMenuModel();
 
-  // Overridden from GetProfilesHelper::Delegate
-  virtual void OnGetProfilesDone(
-      const std::vector<std::wstring>& profiles);
-
  private:
   void Build();
+  void BuildProfileSubMenu();
 
   // Contents of the profiles menu to populate with profile names.
   scoped_ptr<menus::SimpleMenuModel> profiles_menu_contents_;
 
-  // Helper class to enumerate profiles information on the file thread.
-  scoped_refptr<GetProfilesHelper> profiles_helper_;
-
   Browser* browser_;  // weak
 
   DISALLOW_COPY_AND_ASSIGN(AppMenuModel);