Upstream version 5.34.104.0

[platform/framework/web/crosswalk.git] / src / chrome / browser / ui / cocoa / browser_window_controller_private.mm

diff --git a/src/chrome/browser/ui/cocoa/browser_window_controller_private.mm b/src/chrome/browser/ui/cocoa/browser_window_controller_private.mm
index af920e6..feced43 100644 (file)
--- a/src/chrome/browser/ui/cocoa/browser_window_controller_private.mm
+++ b/src/chrome/browser/ui/cocoa/browser_window_controller_private.mm
@@ -59,7 +59,7 @@ const CGFloat kAvatarTabStripShrink = 18;
 
 // Width of the full screen icon. Used to position the AvatarButton to the
 // left of the icon.
-const CGFloat kFullscreenIconWidth = 30;
+const CGFloat kFullscreenIconWidth = 32;
 
 // Insets for the location bar, used when the full toolbar is hidden.
 // TODO(viettrungluu): We can argue about the "correct" insetting; I like the
@@ -319,26 +319,23 @@ willPositionSheet:(NSWindow*)sheet
 
   // Lay out the icognito/avatar badge because calculating the indentation on
   // the right depends on it.
+  NSView* avatarButton = [avatarButtonController_ view];
   if ([self shouldShowAvatar]) {
-    NSView* avatarButton = [avatarButtonController_ view];
     CGFloat badgeXOffset = -kAvatarRightOffset;
     CGFloat badgeYOffset = 0;
-    CGFloat buttonHeight;
+    CGFloat buttonHeight = NSHeight([avatarButton frame]);
 
     if ([self shouldUseNewAvatarButton]) {
       // The fullscreen icon is displayed to the right of the avatar button.
       if (![self isFullscreen])
         badgeXOffset -= kFullscreenIconWidth;
-
       // Center the button vertically on the tabstrip.
-      buttonHeight = NSHeight([avatarButton frame]);
       badgeYOffset = (tabStripHeight - buttonHeight) / 2;
     } else {
-      buttonHeight = static_cast<CGFloat>(profiles::kAvatarIconHeight);
       // Actually place the badge *above* |maxY|, by +2 to miss the divider.
       badgeYOffset = 2 * [[avatarButton superview] cr_lineWidth];
-
     }
+
     [avatarButton setFrameSize:NSMakeSize(NSWidth([avatarButton frame]),
         std::min(buttonHeight, tabStripHeight))];
     NSPoint origin =
@@ -358,18 +355,14 @@ willPositionSheet:(NSWindow*)sheet
     FramedBrowserWindow* window =
         static_cast<FramedBrowserWindow*>([self window]);
     rightIndent += -[window fullScreenButtonOriginAdjustment].x;
+
+    // The new avatar is wider than the default indentation, so we need to
+    // account for its width.
+    if ([self shouldUseNewAvatarButton])
+      rightIndent += NSWidth([avatarButton frame]) + kAvatarTabStripShrink;
   } else if ([self shouldShowAvatar]) {
-    rightIndent += kAvatarTabStripShrink;
-    if ([self shouldUseNewAvatarButton]) {
-      rightIndent += NSWidth([[avatarButtonController_ view] frame])
-        + kAvatarTabStripShrink;
-    } else {
-      NSButton* labelButton =
-          [static_cast<AvatarIconController*>(avatarButtonController_)
-              labelButtonView];
-      if (labelButton)
-        rightIndent += NSWidth([labelButton frame]) + kAvatarRightOffset;
-    }
+    rightIndent += kAvatarTabStripShrink +
+        NSWidth([avatarButton frame]) + kAvatarRightOffset;
   }
   [tabStripController_ setRightIndentForControls:rightIndent];