[Webkit-unassigned] [Bug 41210] Cross Origin XMLHttpRequest can not expose headers indicated in Access-Control-Expose-Headers HTTP Response Header
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jan 16 09:06:06 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=41210
--- Comment #27 from Joe Thomas <joethomas at motorola.com> 2012-01-16 09:06:06 PST ---
(In reply to comment #20)
> (From update of attachment 122549 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=122549&action=review
>
> THis is pretty close. I have a few minor nitpicks, and I think that you should make another try converting this to a "shouldBe" test.
>
Done
> > Source/WebCore/loader/CrossOriginAccessControl.cpp:173
> > +void parseAccessControlExposeHeadersAllowList(const String& headerValue, HashSet<String, CaseFoldingHash>& headersSet)
>
> Is "headers set" good English grammar? FWIW, existing code says "header set".
>
Changed.
> > Source/WebCore/loader/CrossOriginAccessControl.cpp:177
> > + for (unsigned int headerCount = 0; headerCount < headers.size(); headerCount++) {
>
> WebKit coding style is to use "unsigned", not "unsigned int".
>
Done
> > Source/WebCore/loader/CrossOriginAccessControl.h:50
> > +void parseAccessControlExposeHeadersAllowList(const String& headerValue, HashSet<String, CaseFoldingHash>& headersSet);
>
> I'd suggest moving HTTPHeaderSet definition to the header, and using it for consistency.
Done
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list