Bug 194394

Summary: Disable the WP emoji polyfill
Product: WebKit Reporter: Jon Davis <jond>
Component: WebKit WebsiteAssignee: Jon Davis <jond>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, hi, jond, nvasilyev, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Jon Davis
Reported 2019-02-07 08:53:07 PST
The WP emoji polyfill is an additional unnecessary script load and should be disabled.
Attachments
Patch (1.83 KB, patch)
2019-02-07 08:54 PST, Jon Davis 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jon Davis
Comment 1 2019-02-07 08:54:51 PST
Created attachment 361401 [details] Patch
Nikita Vasilyev
Comment 2 2019-02-07 13:33:22 PST
Looks good, I'd r+! Emojis are supported on all platforms we care about.
Devin Rousso
Comment 3 2019-02-07 16:39:06 PST
Comment on attachment 361401 [details] Patch rs=me, finally no more canvas in Web Inspector's Canvas tab 😁
WebKit Commit Bot
Comment 4 2019-02-07 16:55:14 PST
Comment on attachment 361401 [details] Patch Clearing flags on attachment: 361401 Committed r241174: <https://trac.webkit.org/changeset/241174>
WebKit Commit Bot
Comment 5 2019-02-07 16:55:15 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2019-02-07 16:56:29 PST
<rdar://problem/47903164>
Note You need to log in before you can comment on or make changes to this bug.