Summary: | Restrict HTTP/0.9 responses to default ports and cancel HTTP/0.9 resource loads if the document was loaded with another HTTP protocol | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | John Wilander <wilander> | ||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | achristensen, bfulgham, dveditz, jww, webkit-bug-importer, wilander | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 164530 | ||||||||
Attachments: |
|
Description
John Wilander
2016-06-09 15:54:27 PDT
Created attachment 280961 [details]
Patch
Comment on attachment 280961 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=280961&action=review Looks good! r=me (assuming all tests continue to pass). > Source/WebCore/ChangeLog:12 > + HTTP/0.9 header tests for positive and negative cases. Could you please attach the Python script and instructions to the Bugzilla bug so others (e.g., GTK people) could do testing later if they need to? > Source/WebCore/loader/ResourceLoader.cpp:435 > + auto url = r.url(); This should really be "m_response.url()" for consistency. Committed r201895: <http://trac.webkit.org/changeset/201895> Created attachment 280969 [details]
Manual test cases for main document and resource loads
I made the Python test as a stand-alone file. Instructions as comments in the top of the file.
Adding Dan Veditz from Mozilla and Joel Weinberger from Google to the CC list so as to facilitate coordination. |