Bug 168682

Summary:

[Cocoa] Remove Yosemite-specific font lookup code

Product:

WebKit

Reporter:

Myles C. Maxfield <mmaxfield>

Component:

New Bugs

Assignee:

Myles C. Maxfield <mmaxfield>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

benjamin, buildbot, cdumez, cmarcelo, commit-queue, dbates, dino, jonlee, rniwa, simon.fraser, thorton

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	zalan: review+, buildbot: commit-queue-
Archive of layout-test-results from ews106 for mac-elcapitan-wk2	none
Archive of layout-test-results from ews112 for mac-elcapitan	none
Archive of layout-test-results from ews102 for mac-elcapitan	none
Testing EWS	none
Patch	none
Patch for committing	none

Myles C. Maxfield

Reported 2017-02-21 15:31:18 PST

[Cocoa] Remove Yosemite-specific font lookup code

Attachments
Patch (15.79 KB, patch) 2017-02-21 15:33 PST, Myles C. Maxfield	zalan: review+ buildbot: commit-queue-	Details Formatted Diff Diff
Archive of layout-test-results from ews106 for mac-elcapitan-wk2 (1.02 MB, application/zip) 2017-02-21 16:58 PST, Build Bot	no flags	Details
Archive of layout-test-results from ews112 for mac-elcapitan (1.73 MB, application/zip) 2017-02-21 17:04 PST, Build Bot	no flags	Details
Archive of layout-test-results from ews102 for mac-elcapitan (1.68 MB, application/zip) 2017-02-22 05:47 PST, Build Bot	no flags	Details
Testing EWS (1.08 KB, patch) 2017-02-22 16:20 PST, Myles C. Maxfield	no flags	Details Formatted Diff Diff
Patch (26.33 KB, patch) 2017-02-22 16:46 PST, Myles C. Maxfield	no flags	Details Formatted Diff Diff
Patch for committing (25.73 KB, patch) 2017-02-22 18:33 PST, Myles C. Maxfield	no flags	Details Formatted Diff Diff
Show Obsolete (6) View All Add attachment proposed patch, testcase, etc.

Myles C. Maxfield

Comment 1 2017-02-21 15:33:09 PST

Created attachment 302321 [details] Patch

Build Bot

Comment 2 2017-02-21 16:58:23 PST

Comment on attachment 302321 [details] Patch Attachment 302321 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/3168827 New failing tests: fast/text/font-weights.html fast/text/font-weights-zh.html

Build Bot

Comment 3 2017-02-21 16:58:27 PST

Created attachment 302342 [details] Archive of layout-test-results from ews106 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews106 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6

Build Bot

Comment 4 2017-02-21 17:04:35 PST

Comment on attachment 302321 [details] Patch Attachment 302321 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/3168842 New failing tests: fast/text/font-weights.html fast/text/font-weights-zh.html

Build Bot

Comment 5 2017-02-21 17:04:40 PST

Created attachment 302343 [details] Archive of layout-test-results from ews112 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews112 Port: mac-elcapitan Platform: Mac OS X 10.11.6

Build Bot

Comment 6 2017-02-22 05:47:08 PST

Comment on attachment 302321 [details] Patch Attachment 302321 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/3172260 New failing tests: fast/text/font-weights.html fast/text/font-weights-zh.html

Build Bot

Comment 7 2017-02-22 05:47:12 PST

Created attachment 302391 [details] Archive of layout-test-results from ews102 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-elcapitan Platform: Mac OS X 10.11.6

Myles C. Maxfield

Comment 8 2017-02-22 16:20:31 PST

Created attachment 302460 [details] Testing EWS

Myles C. Maxfield

Comment 9 2017-02-22 16:46:00 PST

Created attachment 302465 [details] Patch

Myles C. Maxfield

Comment 10 2017-02-22 18:33:38 PST

Created attachment 302476 [details] Patch for committing

Myles C. Maxfield

Comment 11 2017-02-22 22:13:00 PST

Committed r212876: <http://trac.webkit.org/changeset/212876>

Note You need to log in before you can comment on or make changes to this bug.