Bug 82810 - [Mac] WebKit doesn’t work correctly with composite font references
Summary: [Mac] WebKit doesn’t work correctly with composite font references
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac Unspecified
: P2 Normal
Assignee: mitz
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2012-03-30 17:43 PDT by mitz
Modified: 2012-04-01 11:21 PDT (History)
4 users (show)

See Also:


Attachments
Make WebKit work with composite font references (132.55 KB, patch)
2012-03-30 17:43 PDT, mitz
no flags Details | Formatted Diff | Diff
Make WebKit work with composite font references (132.62 KB, patch)
2012-03-30 17:54 PDT, mitz
no flags Details | Formatted Diff | Diff
Make WebKit work with composite font references (132.71 KB, patch)
2012-03-30 18:17 PDT, mitz
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-02 (7.48 MB, application/zip)
2012-03-30 19:38 PDT, WebKit Review Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2012-03-30 17:43:24 PDT
Created attachment 134919 [details]
Make WebKit work with composite font references

<rdar://problem/10717370>

When an NSFont is composed of multiple fonts, rather than representing a single CGFont, WebKit doesn’t render it correctly. Attaching a patch from another contributor, which addresses this issue.
Comment 1 mitz 2012-03-30 17:54:32 PDT
Created attachment 134921 [details]
Make WebKit work with composite font references
Comment 2 WebKit Review Bot 2012-03-30 17:58:03 PDT
Attachment 134921 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1
LayoutTests/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Tools/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Source/WebCore/platform/graphics/mac/GlyphPageTreeNodeMac.cpp:139:  An else should appear on the same line as the preceding }  [whitespace/newline] [4]
Source/WebCore/ChangeLog:28:  Line contains tab character.  [whitespace/tab] [5]
Source/WebCore/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Total errors found: 5 in 22 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 mitz 2012-03-30 18:17:51 PDT
Created attachment 134923 [details]
Make WebKit work with composite font references
Comment 4 WebKit Review Bot 2012-03-30 18:20:28 PDT
Attachment 134923 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1
Source/WebCore/ChangeLog:29:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 1 in 22 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 WebKit Review Bot 2012-03-30 19:38:09 PDT
Comment on attachment 134923 [details]
Make WebKit work with composite font references

Attachment 134923 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/12267573

New failing tests:
fast/text/international/text-spliced-font.html
Comment 6 WebKit Review Bot 2012-03-30 19:38:15 PDT
Created attachment 134933 [details]
Archive of layout-test-results from ec2-cr-linux-02

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-02  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 7 mitz 2012-03-30 19:43:32 PDT
(In reply to comment #5)
> New failing tests:
> fast/text/international/text-spliced-font.html

This is a new test from this patch.
Comment 8 mitz 2012-03-30 20:14:22 PDT
Fixed in <http://trac.webkit.org/r112767>.
Comment 9 Dirk Pranke 2012-03-30 20:38:03 PDT
Looks like this broke the Chromium Mac build ... I expect it will probably break the Apple Mac builds shortly.

http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac%20Builder/builds/21403/steps/compile/logs/stdio#error1
Comment 10 mitz 2012-03-30 20:41:20 PDT
(In reply to comment #9)
> Looks like this broke the Chromium Mac build ... I expect it will probably break the Apple Mac builds shortly.
> 
> http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac%20Builder/builds/21403/steps/compile/logs/stdio#error1

I am looking into this now.
Comment 11 mitz 2012-03-30 20:44:45 PDT
Tried to fix the Chromium Mac build in <http://trac.webkit.org/r112770>.
Comment 12 Simon Fraser (smfr) 2012-03-30 20:55:48 PDT
This caused a lot of new test failures:
http://build.webkit.org/results/Lion%20Release%20(Tests)/r112767%20(7070)/results.html
Comment 13 mitz 2012-03-30 20:58:01 PDT
(In reply to comment #12)
> This caused a lot of new test failures:
> http://build.webkit.org/results/Lion%20Release%20(Tests)/r112767%20(7070)/results.html

I am going to revert this change ASAP.
Comment 14 mitz 2012-03-30 21:02:17 PDT
Reverted r112770 and r112767 in <http://trac.webkit.org/r112771>.
Comment 15 mitz 2012-04-01 11:21:51 PDT
Fixed in <http://trac.webkit.org/r112816>.