[Webkit-unassigned] [Bug 136938] XMLHttpRequest should have a responseURL attribute (added in recent XHR specification drafts)
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Oct 20 09:47:31 PDT 2014
https://bugs.webkit.org/show_bug.cgi?id=136938
Alexey Proskuryakov <ap at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #240115|review?, commit-queue? |review-, commit-queue-
Flags| |
--- Comment #35 from Alexey Proskuryakov <ap at webkit.org> ---
Comment on attachment 240115
--> https://bugs.webkit.org/attachment.cgi?id=240115
Patch-Updated-Review6
View in context: https://bugs.webkit.org/attachment.cgi?id=240115&action=review
http/tests/xmlhttprequest/xmlhttprequest-responseURL.html fails on EWS.
> Source/WebCore/platform/network/ResourceResponseBase.cpp:153
> + m_url.removeFragmentIdentifier();
By doing it at this level, you are effectively saying that response URL can never have a fragment identifier. How did you determine that this is correct? This change goes way beyond the purpose of this patch.
I do not have any easy way to determine whether this change is OK. It looks like it may not break anything at the moment, but only because of bugs in other places, and fragment actually needs to be respected - see bug 24175.
> Source/WebCore/xml/XMLHttpRequest.cpp:340
> + return m_response.url().string();
The right place to remove fragment is in this accessor.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20141020/145a5db9/attachment-0002.html>
More information about the webkit-unassigned
mailing list