Part of multiprofile implementation.

chrome/browser/profiles/avatar_menu_model.cc

 // 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.
 
 #include "chrome/browser/profiles/avatar_menu_model.h"
 
 #include "base/bind.h"
 #include "base/metrics/field_trial.h"
 #include "base/stl_util.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 174 matching lines @@
                               const content::NotificationSource& source,
                               const content::NotificationDetails& details) {
   DCHECK_EQ(chrome::NOTIFICATION_PROFILE_CACHED_INFO_CHANGED, type);
   RebuildMenu();
   if (observer_)
     observer_->OnAvatarMenuModelChanged(this);
 }
 
 // static
 bool AvatarMenuModel::ShouldShowAvatarMenu() {
 #if defined(OS_CHROMEOS)

[sail, 2013/04/04 14:38:37]

Could you just delete this part entirely?

[Nikita (slow), 2013/04/04 14:50:05]

This part should not be deleted as eventually this code will be present in the
final version i.e. Chrome OS has separate multi-profiles UI.

The reason why this code is commented out is to have a short-term prototype that
works with --multi-profiles switch. 

[sail, 2013/04/04 15:09:34]

Makes sense. Could you add a comment / TODO then?

[dzhioev (left Google), 2013/04/04 15:54:41]

I've uncommented it back as long as we have no Chrome OS specific
implementation.

   // On Chrome OS we use different UI for multi-profiles.
-  return false;
+  // return false;
 #endif
 
   if (base::FieldTrialList::FindFullName(kShowProfileSwitcherFieldTrialName) ==
       kAlwaysShowSwitcherGroupName) {
     // We should only be in this group when multi-profiles is enabled.
     DCHECK(ProfileManager::IsMultipleProfilesEnabled());
     return true;
   }
   return ProfileManager::IsMultipleProfilesEnabled() &&
       g_browser_process->profile_manager()->GetNumberOfProfiles() > 1;
@@ skipping 22 matching lines @@
       base::FilePath path = profile_info_->GetPathOfProfileAtIndex(i);
       item->active = browser_->profile()->GetPath() == path;
     }
     items_.push_back(item);
   }
 }
 
 void AvatarMenuModel::ClearMenu() {
   STLDeleteElements(&items_);
 }