[cros] Use a distinct icon for the guest session.

chrome/browser/ui/views/frame/browser_non_client_frame_view.cc

 // Copyright (c) 2011 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/ui/views/frame/browser_non_client_frame_view.h"
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
 #include "chrome/browser/profiles/profile_manager.h"
@@ skipping 20 matching lines @@
     }
   } else if (avatar_button_.get()) {
     RemoveChildView(avatar_button_.get());
     avatar_button_.reset();
     frame_->GetRootView()->Layout();
   }
 
   if (!avatar_button_.get())
     return;
 
-  if (browser_view_->IsOffTheRecord()) {
+  if (browser_view_->IsGuestSession()) {
+    avatar_button_->SetAvatarIcon(
+        gfx::Image(new SkBitmap(browser_view_->GetGuestAvatarIcon())), false);
+  } else if (browser_view_->IsOffTheRecord()) {
     avatar_button_->SetAvatarIcon(
         gfx::Image(new SkBitmap(browser_view_->GetOTRAvatarIcon())), false);

[whywhat, 2011/12/08 17:21:01]

Maybe we should rather change GetOTRAvatarIcon that would return different icons
depending on what session we're in?

[Ivan Korotkov, 2011/12/09 09:38:22]

As discussed offline, better leave it as is.

   } else {
     ProfileInfoCache& cache =
         g_browser_process->profile_manager()->GetProfileInfoCache();
     Profile* profile = browser_view_->browser()->profile();
     size_t index = cache.GetIndexOfProfileWithPath(profile->GetPath());
     if (index != std::string::npos) {
       bool is_gaia_picture =
           cache.IsUsingGAIAPictureOfProfileAtIndex(index) &&
           cache.GetGAIAPictureOfProfileAtIndex(index);
       avatar_button_->SetAvatarIcon(
           cache.GetAvatarIconOfProfileAtIndex(index), is_gaia_picture);
       avatar_button_->SetText(cache.GetNameOfProfileAtIndex(index));
     }
   }
 }