RESOLVED FIXED 146894
REGRESSION (r177876): store.apple.com profile and cart icons are missing
https://bugs.webkit.org/show_bug.cgi?id=146894
Summary REGRESSION (r177876): store.apple.com profile and cart icons are missing
Antti Koivisto
Reported 2015-07-12 13:49:56 PDT
Two competing @font-face rules with only one having the correct glyphs.
Attachments
patch (36.04 KB, patch)
2015-07-12 15:02 PDT, Antti Koivisto
mmaxfield: review+
Antti Koivisto
Comment 1 2015-07-12 13:50:17 PDT
Antti Koivisto
Comment 2 2015-07-12 15:02:26 PDT
Alexey Proskuryakov
Comment 3 2015-07-12 15:34:11 PDT
Comment on attachment 256683 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=256683&action=review > LayoutTests/fast/css/font-face-multiple-missing-glyphs-expected.html:7 > + font-family:AppleIcons; Is this available under an appropriate license?
Antti Koivisto
Comment 4 2015-07-12 15:41:16 PDT
> Is this available under an appropriate license? No idea.
Myles C. Maxfield
Comment 5 2015-07-13 13:48:15 PDT
Comment on attachment 256683 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=256683&action=review r=me > Source/WebCore/platform/graphics/FontCascadeFonts.cpp:339 > + ++fallbackIndex; I don't understand. It doesn't look like there is a "continue" anywhere in the loop which will cause this to be skipped. Why did you move this to the bottom of the loop? >> LayoutTests/fast/css/font-face-multiple-missing-glyphs-expected.html:7 >> + font-family:AppleIcons; > > Is this available under an appropriate license? You can author your own font in SVG for this test. Should be straightforward.
Myles C. Maxfield
Comment 6 2015-07-14 12:52:00 PDT
WebKit Commit Bot
Comment 7 2015-07-14 14:16:04 PDT
Re-opened since this is blocked by bug 146943
Mark Lam
Comment 8 2015-07-14 14:23:35 PDT
Antti Koivisto
Comment 9 2015-07-14 16:18:58 PDT
Thanks for updating the test case but you probably didn't want to include the SimpleLineLayout.cpp change.
Myles C. Maxfield
Comment 10 2015-07-14 16:32:08 PDT
(In reply to comment #9) > Thanks for updating the test case but you probably didn't want to include > the SimpleLineLayout.cpp change. hahah .... whoops. :(
Myles C. Maxfield
Comment 11 2015-07-14 17:24:53 PDT
Note You need to log in before you can comment on or make changes to this bug.