[Mac] Redesign of the avatar menu button

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

Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index a1dc1b6fe91841581f5784cea1ac4d3b323a895c..579f17664a160cea54cb6aba6b98d93b64cf7f07 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -561,8 +561,7 @@ bool BrowserView::IsGuestSession() const {
 }
 
 bool BrowserView::IsRegularOrGuestSession() const {
-  Profile* profile = browser_->profile();
-  return (profile->IsGuestSession() || !profile->IsOffTheRecord());
+  return profiles::IsRegularOrGuestSession(browser_.get());

[Nico, 2014/01/07 23:20:33]

is this change intentionally in this cl?

[noms (inactive), 2014/01/08 15:05:31]

Yes, but I can move it in a separate CL if you want. I had to make the same
checks in the Mac code, so I realized it would be easier if I made a utility
function in profile_state (and then use it correctly here, rather than duplicate
the code again). Should I move it out?

On 2014/01/07 23:20:33, Nico wrote:

 }
 
 int BrowserView::GetOTRIconResourceID() const {