Bug 161996

Summary: Add needsPlainTextQuirk and send it to the UIProcess
Product: WebKit Reporter: Beth Dakin <bdakin>
Component: WebKit2Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, bdakin, sam, thorton, wenson_hsieh
Priority: P2    
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch andersca: review+

Description Beth Dakin 2016-09-14 15:37:56 PDT 
Add needsPlainTextQuirk and send it to the UIProcess

rdar://problem/26013388
Comment 1 Beth Dakin 2016-09-14 15:41:31 PDT 
Created attachment 288875 [details]
Patch
Comment 2 Simon Fraser (smfr) 2016-09-14 16:02:26 PDT 
Comment on attachment 288875 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=288875&action=review

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:4820
> +    String fragmentIdentifier = url.fragmentIdentifier();

Why compute this up front, when you won't use it for twitter, onedrive etc?
Comment 3 Beth Dakin 2016-09-14 16:08:36 PDT 
Thanks Anders! https://trac.webkit.org/changeset/205934
Comment 4 Beth Dakin 2016-09-14 16:17:27 PDT 
(In reply to comment #2)
> Comment on attachment 288875 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=288875&action=review
> 
> > Source/WebKit2/WebProcess/WebPage/WebPage.cpp:4820
> > +    String fragmentIdentifier = url.fragmentIdentifier();
> 
> Why compute this up front, when you won't use it for twitter, onedrive etc?

https://trac.webkit.org/changeset/205935 Thanks Simon!