Add a short term solution to fix OSX 10.11 system font rendering problems.

Add a short term solution to fix OSX 10.11 system font rendering problems.

I landed a CL (https://codereview.chromium.org/1276003003/) which fixes system
font problems on OSX 10.9 and OSX 10.10. The CL also fixed MatchNSFontFamily()
to correctly return the system font on 10.11, but this causes glyph rendering
problems. In the short term, I'm adding logic to MatchNSFontFamily() to return
nil on 10.11, which was the original behavior before I landed that CL.

BUG=521034
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200618

[erikchen, 2015-08-15 01:31:57]

https://codereview.chromium.org/1284993004/diff/20001/Source/platform/mac/Ver...
File Source/platform/mac/VersionUtilMac.mm (right):

https://codereview.chromium.org/1284993004/diff/20001/Source/platform/mac/Ver...
Source/platform/mac/VersionUtilMac.mm:30: bool IsOSYosemiteOrEarlier()
Ideally, I would make a function called IsOSElCapitan, but the AppKit version
number for El Capitan is not yet finalized. This function will be removed once I
fix the rendering problem anyways.

[erikchen, 2015-08-15 01:34:55]

On 2015/08/15 01:31:57, erikchen wrote:
>
https://codereview.chromium.org/1284993004/diff/20001/Source/platform/mac/Ver...
> File Source/platform/mac/VersionUtilMac.mm (right):
> 
>
https://codereview.chromium.org/1284993004/diff/20001/Source/platform/mac/Ver...
> Source/platform/mac/VersionUtilMac.mm:30: bool IsOSYosemiteOrEarlier()
> Ideally, I would make a function called IsOSElCapitan, but the AppKit version
> number for El Capitan is not yet finalized. This function will be removed once
I
> fix the rendering problem anyways.

I've manually confirmed that this CL produces the right behavior on 10.11. (We
don't have waterfall Webkit layout tests yet for 10.11)

[erikchen, 2015-08-15 01:35:35]

erikchen@chromium.org changed reviewers:
+ keishi@chromium.org, tkent@chromium.org

[erikchen, 2015-08-15 01:35:35]

tkent or keishi: Please review.

[erikchen, 2015-08-15 01:35:40]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-08-15 01:35:58]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1284993004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1284993004/20001

[commit-bot: I haz the power, 2015-08-15 03:10:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-15 03:10:33]

Dry run: This issue passed the CQ dry run.

[keishi, 2015-08-15 04:16:17]

lgtm

[erikchen, 2015-08-15 04:25:17]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2015-08-15 04:25:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1284993004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1284993004/20001

[commit-bot: I haz the power, 2015-08-15 04:33:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-08-15 04:33:14]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[tkent, 2015-08-17 00:11:43]

The CQ bit was checked by tkent@chromium.org

[tkent, 2015-08-17 00:11:44]

lgtm

[commit-bot: I haz the power, 2015-08-17 00:11:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1284993004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1284993004/20001

[commit-bot: I haz the power, 2015-08-17 00:58:25]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200618

[yurys, 2015-08-17 15:41:17]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1296243002/ by yurys@chromium.org.

The reason for reverting is: Newly added test has been failing since its
introduction

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=....