Commit ec030bde authored by Sidney San Martín's avatar Sidney San Martín Committed by Commit Bot

Require 10.12 for ShouldUseFullSizeContentView().

Bug: 742472
Change-Id: I82a02072b308d7f7753b9a5274465cdba7c64bbd
Reviewed-on: https://chromium-review.googlesource.com/571420Reviewed-by: default avatarElly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Sidney San Martin <sdy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#486838}
parent cd38ef1c
......@@ -24,9 +24,10 @@ bool ShouldUseFullSizeContentView() {
// stopped the window buttons from being covered by the content view. This
// may break in a future macOS release. NSFullSizeContentViewWindowMask is a
// new (10.10+), supported way to make the content view the full size of the
// window without covering the controls.
// window without covering the controls. It's only enabled in 10.12+ due to a
// performance regression seen in 10.11 (https://crbug.com/742472).
return base::FeatureList::IsEnabled(features::kMacFullSizeContentView) &&
base::mac::IsAtLeastOS10_11();
base::mac::IsAtLeastOS10_12();
}
} // namespace chrome
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment