[Mac] Take Sierra RTL into account when laying out window buttons

chrome/browser/ui/cocoa/framed_browser_window_unittest.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/debug/debugger.h"
 #include "base/mac/scoped_nsobject.h"
 #include "chrome/app/chrome_command_ids.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #import "chrome/browser/ui/cocoa/cocoa_test_helper.h"
 #import "chrome/browser/ui/cocoa/framed_browser_window.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #import "testing/gtest_mac.h"
 #include "testing/platform_test.h"
 #import "third_party/ocmock/OCMock/OCMock.h"
 
+namespace {
+NSString* const kAppleTextDirectionDefaultsKey = @"AppleTextDirection";
+NSString* const kForceRTLWritingDirectionDefaultsKey =
+    @"NSForceRightToLeftWritingDirection";
+}  // namespace
+
 class FramedBrowserWindowTest : public CocoaTest {
  public:
   void SetUp() override {
     CocoaTest::SetUp();
     // Create a window.
     window_ = [[FramedBrowserWindow alloc]
                initWithContentRect:NSMakeRect(0, 0, 800, 600)
                        hasTabStrip:YES];
     if (base::debug::BeingDebugged()) {
       [window_ orderFront:nil];
@@ skipping 106 matching lines @@
                 kFramedWindowButtonsWithoutTabStripOffsetFromTop);
   EXPECT_EQ(NSMinX(closeBoxFrame),
             kFramedWindowButtonsWithoutTabStripOffsetFromLeft);
 
   NSView* miniaturizeControl =
       [window_ standardWindowButton:NSWindowMiniaturizeButton];
   EXPECT_TRUE(miniaturizeControl);
   NSRect miniaturizeFrame =
       [miniaturizeControl convertRect:[miniaturizeControl bounds]
                                toView:nil];
+  EXPECT_LT(NSMaxX(closeBoxFrame), NSMinX(miniaturizeFrame));
   EXPECT_EQ(NSMaxY(miniaturizeFrame),
             NSMaxY(windowBounds) -
                 kFramedWindowButtonsWithoutTabStripOffsetFromTop);
+  EXPECT_EQ(NSMinX(miniaturizeFrame),
+            NSMaxX(closeBoxFrame) + [window_ windowButtonsInterButtonSpacing]);
+  [window_ setWindowController:nil];
+
+  // Then with a tabstrip.
+  [window_ close];
+  window_ = [[FramedBrowserWindow alloc]
+             initWithContentRect:NSMakeRect(0, 0, 800, 600)
+                     hasTabStrip:YES];
+  // Update window layout according to existing layout constraints.
+  [window_ layoutIfNeeded];
+  controller = [OCMockObject mockForClass:[BrowserWindowController class]];
+  [[[controller stub] andReturnValue:OCMOCK_VALUE(yes)]
+      isKindOfClass:[BrowserWindowController class]];
+  [[[controller expect] andReturnValue:OCMOCK_VALUE(yes)] hasTabStrip];
+  [[[controller expect] andReturnValue:OCMOCK_VALUE(no)] hasTitleBar];
+  [[[controller expect] andReturnValue:OCMOCK_VALUE(yes)] isTabbedWindow];
+  [window_ setWindowController:controller];
+
+  closeBoxControl = [window_ standardWindowButton:NSWindowCloseButton];
+  EXPECT_TRUE(closeBoxControl);
+  closeBoxFrame = [closeBoxControl convertRect:[closeBoxControl bounds]
+                                        toView:nil];
+  windowBounds = [window_ frame];
+  windowBounds = [[window_ contentView] convertRect:windowBounds fromView:nil];
+  windowBounds.origin = NSZeroPoint;
+  EXPECT_EQ(NSMaxY(closeBoxFrame),
+            NSMaxY(windowBounds) -
+                kFramedWindowButtonsWithTabStripOffsetFromTop);
+  EXPECT_EQ(NSMinX(closeBoxFrame),
+            kFramedWindowButtonsWithTabStripOffsetFromLeft);
+
+  miniaturizeControl = [window_ standardWindowButton:NSWindowMiniaturizeButton];
+  EXPECT_TRUE(miniaturizeControl);
+  miniaturizeFrame = [miniaturizeControl convertRect:[miniaturizeControl bounds]
+                                              toView:nil];
+  EXPECT_EQ(NSMaxY(miniaturizeFrame),
+            NSMaxY(windowBounds) -
+                kFramedWindowButtonsWithTabStripOffsetFromTop);
+  EXPECT_EQ(NSMinX(miniaturizeFrame),
+            NSMaxX(closeBoxFrame) + [window_ windowButtonsInterButtonSpacing]);
+  [window_ setWindowController:nil];
+}
+
+class FramedBrowserWindowRTLTest : public FramedBrowserWindowTest {
+  void SetUp() override {
+    NSUserDefaults* defaults = [NSUserDefaults standardUserDefaults];
+    originalAppleTextDirection_ =
+        [defaults boolForKey:kAppleTextDirectionDefaultsKey];
+    originalRTLWritingDirection_ =
+        [defaults boolForKey:kForceRTLWritingDirectionDefaultsKey];
+    [defaults setBool:YES forKey:kAppleTextDirectionDefaultsKey];
+    [defaults setBool:YES forKey:kForceRTLWritingDirectionDefaultsKey];
+    FramedBrowserWindowTest::SetUp();
+  }
+
+  void TearDown() override {
+    FramedBrowserWindowTest::TearDown();
+    NSUserDefaults* defaults = [NSUserDefaults standardUserDefaults];
+    [defaults setBool:originalAppleTextDirection_
+               forKey:kAppleTextDirectionDefaultsKey];
+    [defaults setBool:originalRTLWritingDirection_
+               forKey:kForceRTLWritingDirectionDefaultsKey];
+  }
+
+ private:
+  BOOL originalAppleTextDirection_;
+  BOOL originalRTLWritingDirection_;
+};
+
+// Test to make sure that our window widgets are in the right place.
+// Currently, this is the same exact test as above, since RTL button
+// layout is behind the ExperimentalMacRTL flag. However, this ensures
+// that our calculations are correct for Sierra RTL, which lays out
+// the window buttons in reverse by default. See crbug/662079.
+TEST_F(FramedBrowserWindowRTLTest, WindowWidgetLocation) {
+  BOOL yes = YES;
+  BOOL no = NO;
+
+  // First without a tabstrip.
+  [window_ close];
+  window_ = [[FramedBrowserWindow alloc]
+      initWithContentRect:NSMakeRect(0, 0, 800, 600)
+              hasTabStrip:NO];
+  // Update window layout according to existing layout constraints.
+  [window_ layoutIfNeeded];
+  id controller = [OCMockObject mockForClass:[BrowserWindowController class]];
+  [[[controller stub] andReturnValue:OCMOCK_VALUE(yes)]
+      isKindOfClass:[BrowserWindowController class]];
+  [[[controller expect] andReturnValue:OCMOCK_VALUE(no)] hasTabStrip];
+  [[[controller expect] andReturnValue:OCMOCK_VALUE(yes)] hasTitleBar];
+  [[[controller expect] andReturnValue:OCMOCK_VALUE(no)] isTabbedWindow];
+  [window_ setWindowController:controller];
+
+  NSView* closeBoxControl = [window_ standardWindowButton:NSWindowCloseButton];
+  EXPECT_TRUE(closeBoxControl);
+  NSRect closeBoxFrame =
+      [closeBoxControl convertRect:[closeBoxControl bounds] toView:nil];
+  NSRect windowBounds = [window_ frame];
+  windowBounds = [[window_ contentView] convertRect:windowBounds fromView:nil];
+  windowBounds.origin = NSZeroPoint;
+  EXPECT_EQ(
+      NSMaxY(closeBoxFrame),
+      NSMaxY(windowBounds) - kFramedWindowButtonsWithoutTabStripOffsetFromTop);
+  EXPECT_EQ(NSMinX(closeBoxFrame),
+            kFramedWindowButtonsWithoutTabStripOffsetFromLeft);
+
+  NSView* miniaturizeControl =
+      [window_ standardWindowButton:NSWindowMiniaturizeButton];
+  EXPECT_TRUE(miniaturizeControl);
+  NSRect miniaturizeFrame =
+      [miniaturizeControl convertRect:[miniaturizeControl bounds] toView:nil];
+  EXPECT_LT(NSMaxX(closeBoxFrame), NSMinX(miniaturizeFrame));
+  EXPECT_EQ(NSMaxY(miniaturizeFrame),
+            NSMaxY(windowBounds) -
+                kFramedWindowButtonsWithoutTabStripOffsetFromTop);
   EXPECT_EQ(NSMinX(miniaturizeFrame),
             NSMaxX(closeBoxFrame) + [window_ windowButtonsInterButtonSpacing]);
   [window_ setWindowController:nil];
 
   // Then with a tabstrip.
   [window_ close];
   window_ = [[FramedBrowserWindow alloc]
              initWithContentRect:NSMakeRect(0, 0, 800, 600)
                      hasTabStrip:YES];
   // Update window layout according to existing layout constraints.
@@ skipping 23 matching lines @@
   EXPECT_TRUE(miniaturizeControl);
   miniaturizeFrame = [miniaturizeControl convertRect:[miniaturizeControl bounds]
                                               toView:nil];
   EXPECT_EQ(NSMaxY(miniaturizeFrame),
             NSMaxY(windowBounds) -
                 kFramedWindowButtonsWithTabStripOffsetFromTop);
   EXPECT_EQ(NSMinX(miniaturizeFrame),
             NSMaxX(closeBoxFrame) + [window_ windowButtonsInterButtonSpacing]);
   [window_ setWindowController:nil];
 }