[Profiles] Send out less profile avatar related notifications

chrome/browser/ui/views/profiles/new_avatar_button.cc

 // Copyright 2014 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/profiles/new_avatar_button.h"
 
 #include "base/win/windows_version.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/profiles/profiles_state.h"
@@ skipping 132 matching lines @@
     UpdateAvatarButtonAndRelayoutParent();
 }
 
 void NewAvatarButton::OnProfileNameChanged(
       const base::FilePath& profile_path,
       const base::string16& old_profile_name) {
   if (browser_->profile()->GetPath() == profile_path)
     UpdateAvatarButtonAndRelayoutParent();
 }
 
-void NewAvatarButton::OnProfileAvatarChanged(
-      const base::FilePath& profile_path) {
-  if (browser_->profile()->GetPath() == profile_path)

[Mike Lerman, 2015/01/27 16:39:16]

Why remove this? Don't you want to do the same as on Mac... only execute this if
!IsNewAvatarMenu()?

[noms (inactive), 2015/01/27 17:02:20]

NewAvatarButton only exists for the new profiles UI (the other one is
AvatarMenuButton), so it doesn't care about avatar changes :)

-    UpdateAvatarButtonAndRelayoutParent();
-}
-
 void NewAvatarButton::OnProfileSupervisedUserIdChanged(
       const base::FilePath& profile_path) {
   if (browser_->profile()->GetPath() == profile_path)
     UpdateAvatarButtonAndRelayoutParent();
 }
 
 void NewAvatarButton::OnErrorChanged() {
   // If there is an error, show an warning icon.
   const SigninErrorController* error =
       profiles::GetSigninErrorController(browser_->profile());
@@ skipping 39 matching lines @@
   const int kDefaultImageTextSpacing = 5;
   SetImageLabelSpacing(use_generic_button ? 0 : kDefaultImageTextSpacing);
 
   InvalidateLayout();
 
   // Because the width of the button might have changed, the parent browser
   // frame needs to recalculate the button bounds and redraw it.
   if (parent())
     parent()->Layout();
 }