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

Issue 2468373003: [Mac] Take Sierra RTL into account when laying out window buttons (Closed)

Created:
4 years, 1 month ago by lgrey
Modified:
4 years, 1 month ago
CC:
chromium-reviews, mac-reviews_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Mac] Take Sierra RTL into account when laying out window buttons Sierra reverses the window buttons when launching programs in RTL mode. We calculate the size between window buttons based on the gap between the close and miniaturize buttons, but when the buttons are reversed this is incorrect. The result is a large gap between buttons, pushing one offscreen (see image in bug.) BUG=662097 Committed: https://crrev.com/fa5cecc6e5507dc0497b09a3295c9c467e0c64d5 Cr-Commit-Position: refs/heads/master@{#429684}

Patch Set 1 #

Patch Set 2 : Add expectation that close button is left of miniaturize button in original test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+127 lines, -0 lines) Patch
M chrome/browser/ui/cocoa/framed_browser_window.mm View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/ui/cocoa/framed_browser_window_unittest.mm View 1 2 chunks +123 lines, -0 lines 0 comments Download

Messages

Total messages: 11 (4 generated)
lgrey
PTAL :) Elly for RTL, Mark for OWNERS
4 years, 1 month ago (2016-11-03 18:55:51 UTC) #2
Elly Fong-Jones
lgtm Nice! I love the unit tests.
4 years, 1 month ago (2016-11-03 18:59:41 UTC) #3
Mark Mentovai
LGTM
4 years, 1 month ago (2016-11-03 19:38:12 UTC) #4
lgrey
Thanks!
4 years, 1 month ago (2016-11-03 19:42:41 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2468373003/20001
4 years, 1 month ago (2016-11-03 19:44:02 UTC) #8
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 1 month ago (2016-11-03 20:16:11 UTC) #9
commit-bot: I haz the power
4 years, 1 month ago (2016-11-03 20:19:28 UTC) #11
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/fa5cecc6e5507dc0497b09a3295c9c467e0c64d5
Cr-Commit-Position: refs/heads/master@{#429684}

Powered by Google App Engine
This is Rietveld 408576698