Show avatar icon on V2 app's frame

chrome/browser/ui/views/frame/browser_non_client_frame_view.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/ui/views/frame/browser_non_client_frame_view.h"
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/avatar_menu.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_info_cache.h"
@@ skipping 50 matching lines @@
       Profile* profile = browser_view_->browser()->profile();
       if (profile->IsManaged() && !avatar_label_) {
         avatar_label_ = new AvatarLabel(browser_view_);
         avatar_label_->set_id(VIEW_ID_AVATAR_LABEL);
         AddChildView(avatar_label_);
       }
       avatar_button_ = new AvatarMenuButton(
           browser_view_->browser(), !browser_view_->IsRegularOrGuestSession());
       avatar_button_->set_id(VIEW_ID_AVATAR_BUTTON);
       AddChildView(avatar_button_);
+      InvalidateLayout();

[oshima, 2014/03/14 20:40:44]

This is necessary because adding child does not invalidate layout, which was
causing an issue when re-showing avatar.

[Mr4D (OOO till 08-26), 2014/03/14 22:01:30]

Thanks for the explanation! It might be good to add a comment here towards that?

[oshima, 2014/03/15 01:32:32]

Done.

       frame_->GetRootView()->Layout();
     }
   } else if (avatar_button_) {
     // The avatar label can just be there if there is also an avatar button.
     if (avatar_label_) {
       RemoveChildView(avatar_label_);
       delete avatar_label_;
       avatar_label_ = NULL;
     }
     RemoveChildView(avatar_button_);
@@ skipping 73 matching lines @@
       new_avatar_button_->set_id(VIEW_ID_NEW_AVATAR_BUTTON);
       AddChildView(new_avatar_button_);
       frame_->GetRootView()->Layout();
     }
   } else if (new_avatar_button_) {
     delete new_avatar_button_;
     new_avatar_button_ = NULL;
     frame_->GetRootView()->Layout();
   }
 }