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

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

Issue 2893343002: Resize avatar button as appropriate for opaque browser frame. (Closed)
Patch Set: comment 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
« no previous file with comments | « chrome/browser/ui/views/frame/glass_browser_frame_view.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 520c6058f30a2326ffae9b9421a32b42489a2ae4..61c9180850178aae9b2d97d33b0b8d8e96bd4e82 100644
--- a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
+++ b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
@@ -432,21 +432,6 @@ void GlassBrowserFrameView::Layout() {
LayoutClientView();
}
-void GlassBrowserFrameView::ChildPreferredSizeChanged(views::View* child) {
- if (child == GetProfileSwitcherView()) {
- // Need to layout the root view here, too, as the avatar button may change
- // between the text and the icon when a profile is added or removed, which
- // changes its width. This may cause it to start or stop overlapping the
- // the tabstrip horizontally, which in turn causes it to change height, as
- // calculated in LayoutProfileSwitcher(). Calling LayoutProfileSwitcher()
- // is not enough here - it does not re-draw the line below the tabstrip
- // properly when a profile is added or removed. Even adding
- // browser_view()->tabstrip()->Layout() and SchedulePaint() is not enough.
- // TODO(bsep): Figure out the most efficient way to do this.
- frame()->GetRootView()->Layout();
- }
-}
-
///////////////////////////////////////////////////////////////////////////////
// GlassBrowserFrameView, protected:
« no previous file with comments | « chrome/browser/ui/views/frame/glass_browser_frame_view.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698