Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4944)

Unified Diff: chrome/browser/ui/views/frame/browser_non_client_frame_view.h

Issue 895803003: [Profiles] Remove the NotificationService from the ProfileInfoCache (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/frame/browser_non_client_frame_view.h
diff --git a/chrome/browser/ui/views/frame/browser_non_client_frame_view.h b/chrome/browser/ui/views/frame/browser_non_client_frame_view.h
index 43c820aeb4812fd0fac85db9a1f124b673d9f17a..055efb272850b53379404035c4a9ccfb95921507 100644
--- a/chrome/browser/ui/views/frame/browser_non_client_frame_view.h
+++ b/chrome/browser/ui/views/frame/browser_non_client_frame_view.h
@@ -66,9 +66,19 @@ class BrowserNonClientFrameView : public views::NonClientFrameView,
BrowserView* browser_view() const { return browser_view_; }
BrowserFrame* frame() const { return frame_; }
- // Updates the title and icon of the avatar button.
+ // Updates the avatar button using the old or new UI based on the BrowserView
+ // type, and the presence of the --enable-new-avatar-menu flag. Calls either
+ // UpdateAvatarInfo() or UpdateNewStyleAvatar() accordingly.
+ void UpdateAvatar();
+
+ // Updates the title and icon of the old avatar button.
void UpdateAvatarInfo();
+ // Updates the avatar button displayed in the caption area by calling
+ // UpdateNewStyleAvatarInfo() with an implementation specific |listener|
+ // and button |style|.
+ virtual void UpdateNewStyleAvatar() = 0;
+
// Updates the title of the avatar button displayed in the caption area.
// The button uses |style| to match the browser window style and notifies
// |listener| when it is clicked.
@@ -76,16 +86,16 @@ class BrowserNonClientFrameView : public views::NonClientFrameView,
const NewAvatarButton::AvatarButtonStyle style);
private:
- // Draws a taskbar icon if avatar are enabled, erases it otherwise. If
- // |taskbar_badge_avatar| is null, then |avatar| is used.
- void DrawTaskbarDecoration(const gfx::Image& avatar,
- const gfx::Image& taskbar_badge_avatar);
-
// Overriden from ProfileInfoCacheObserver.
void OnProfileAdded(const base::FilePath& profile_path) override;
void OnProfileWasRemoved(const base::FilePath& profile_path,
const base::string16& profile_name) override;
void OnProfileAvatarChanged(const base::FilePath& profile_path) override;
+ void OnProfileNameChanged(const base::FilePath& profile_path,
+ const base::string16& old_profile_name) override;
+
+ // Draws a taskbar icon if avatars are enabled, erases it otherwise.
+ void UpdateTaskbarDecoration();
// The frame that hosts this view.
BrowserFrame* frame_;
« no previous file with comments | « chrome/browser/profiles/profile_manager.cc ('k') | chrome/browser/ui/views/frame/browser_non_client_frame_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698