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

Unified Diff: chrome/browser/ui/views/frame/avatar_button_manager.cc

Issue 2851543002: Update avatar button to MD (part 1) (Closed)
Patch Set: Merged ThemedAvatarButton and Win10NativeAvatarButton into the base AvatarButton class Created 3 years, 7 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/avatar_button_manager.cc
diff --git a/chrome/browser/ui/views/frame/avatar_button_manager.cc b/chrome/browser/ui/views/frame/avatar_button_manager.cc
index 8101bf8b0e16bf11027d8b71cb84f1e4d0e38a71..442243c54e280da56678a7099ce8ec004ddd7ba6 100644
--- a/chrome/browser/ui/views/frame/avatar_button_manager.cc
+++ b/chrome/browser/ui/views/frame/avatar_button_manager.cc
@@ -9,7 +9,7 @@
#include "chrome/browser/ui/view_ids.h"
#include "chrome/browser/ui/views/frame/browser_frame.h"
#include "chrome/browser/ui/views/frame/browser_view.h"
-#include "chrome/browser/ui/views/profiles/new_avatar_button.h"
+#include "chrome/browser/ui/views/profiles/avatar_button.h"
AvatarButtonManager::AvatarButtonManager(BrowserNonClientFrameView* frame_view)
: frame_view_(frame_view), view_(nullptr) {}
@@ -31,7 +31,7 @@ void AvatarButtonManager::Update(AvatarButtonStyle style) {
// Desktop guest shows the avatar button.
browser_view->IsIncognito()) {
if (!view_) {
- view_ = new NewAvatarButton(this, style, profile);
+ view_ = new AvatarButton(this, style, profile);
view_->set_id(VIEW_ID_AVATAR_BUTTON);
frame_view_->AddChildView(view_);
frame->GetRootView()->Layout();
@@ -43,18 +43,8 @@ void AvatarButtonManager::Update(AvatarButtonStyle style) {
}
}
-void AvatarButtonManager::ButtonPreferredSizeChanged() {
- // Perform a re-layout if the avatar button has changed, since that can affect
- // the size of the tabs.
- if (!view_ || !frame_view_->browser_view()->initialized())
- return; // Ignore the update during view creation.
-
- frame_view_->InvalidateLayout();
- frame_view_->frame()->GetRootView()->Layout();
-}
-
void AvatarButtonManager::ButtonPressed(views::Button* sender,
- const ui::Event& event) {
+ const ui::Event& event) {
DCHECK_EQ(view_, sender);
BrowserWindow::AvatarBubbleMode mode =
BrowserWindow::AVATAR_BUBBLE_MODE_DEFAULT;

Powered by Google App Engine
This is Rietveld 408576698