| Index: chrome/browser/ui/views/frame/glass_browser_frame_view.cc
|
| diff --git a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
|
| index 4eaaf0647767725b64d2b1ea8db8a70f7f093b33..61f03855ea12bc38c7804625b12d26a9990cafb9 100644
|
| --- a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
|
| +++ b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
|
| @@ -8,7 +8,6 @@
|
| #include "base/strings/utf_string_conversions.h"
|
| #include "chrome/app/chrome_command_ids.h"
|
| #include "chrome/app/chrome_dll_resource.h"
|
| -#include "chrome/browser/chrome_notification_types.h"
|
| #include "chrome/browser/profiles/profile.h"
|
| #include "chrome/browser/signin/signin_header_helper.h"
|
| #include "chrome/browser/themes/theme_properties.h"
|
| @@ -19,7 +18,6 @@
|
| #include "chrome/browser/ui/views/tabs/tab_strip.h"
|
| #include "chrome/browser/ui/views/toolbar/toolbar_view.h"
|
| #include "components/signin/core/common/profile_management_switches.h"
|
| -#include "content/public/browser/notification_service.h"
|
| #include "grit/theme_resources.h"
|
| #include "skia/ext/image_operations.h"
|
| #include "ui/base/resource/resource_bundle_win.h"
|
| @@ -97,15 +95,7 @@ GlassBrowserFrameView::GlassBrowserFrameView(BrowserFrame* frame,
|
| if (browser_view->ShouldShowWindowIcon())
|
| InitThrobberIcons();
|
|
|
| - if (browser_view->IsRegularOrGuestSession() && switches::IsNewAvatarMenu())
|
| - UpdateNewStyleAvatarInfo(this, NewAvatarButton::NATIVE_BUTTON);
|
| - else
|
| - UpdateAvatarInfo();
|
| -
|
| - if (!browser_view->IsOffTheRecord()) {
|
| - registrar_.Add(this, chrome::NOTIFICATION_PROFILE_CACHED_INFO_CHANGED,
|
| - content::NotificationService::AllSources());
|
| - }
|
| + UpdateAvatar(this, NewAvatarButton::NATIVE_BUTTON);
|
| }
|
|
|
| GlassBrowserFrameView::~GlassBrowserFrameView() {
|
| @@ -594,23 +584,27 @@ void GlassBrowserFrameView::DisplayNextThrobberFrame() {
|
| reinterpret_cast<LPARAM>(throbber_icons_[throbber_frame_]));
|
| }
|
|
|
| -void GlassBrowserFrameView::Observe(
|
| - int type,
|
| - const content::NotificationSource& source,
|
| - const content::NotificationDetails& details) {
|
| - switch (type) {
|
| - case chrome::NOTIFICATION_PROFILE_CACHED_INFO_CHANGED:
|
| - if (browser_view()->IsRegularOrGuestSession() &&
|
| - switches::IsNewAvatarMenu()) {
|
| - UpdateNewStyleAvatarInfo(this, NewAvatarButton::NATIVE_BUTTON);
|
| - } else {
|
| - UpdateAvatarInfo();
|
| - }
|
| - break;
|
| - default:
|
| - NOTREACHED() << "Got a notification we didn't register for!";
|
| - break;
|
| - }
|
| +void GlassBrowserFrameView::OnProfileAdded(const base::FilePath& profile_path) {
|
| + UpdateAvatar(this, NewAvatarButton::NATIVE_BUTTON);
|
| +}
|
| +
|
| +void GlassBrowserFrameView::OnProfileWasRemoved(
|
| + const base::FilePath& profile_path,
|
| + const base::string16& profile_name) {
|
| + UpdateAvatar(this, NewAvatarButton::NATIVE_BUTTON);
|
| +}
|
| +
|
| +void GlassBrowserFrameView::OnProfileNameChanged(
|
| + const base::FilePath& profile_path,
|
| + const base::string16& old_profile_name) {
|
| + UpdateAvatar(this, NewAvatarButton::NATIVE_BUTTON);
|
| +}
|
| +
|
| +void GlassBrowserFrameView::OnProfileAvatarChanged(
|
| + const base::FilePath& profile_path) {
|
| + // Profile avatars are only displayed in the old UI or incognito.
|
| + if (browser_view()->IsOffTheRecord() || !switches::IsNewAvatarMenu())
|
| + UpdateAvatarInfo();
|
| }
|
|
|
| // static
|
|
|