superreview requested: [Bug 476567] Clear Access-Control preflight cache when spec says to : [Attachment 360222] Patch to fix

[bugzill...@mozilla.org]

Jonas Sicking (:sicking) <jo...@sicking.cc> has asked Johnny Stenback (:jst,
j...@mozilla.com) <j...@mozilla.org> for superreview:
Bug 476567: Clear Access-Control preflight cache when spec says to
https://bugzilla.mozilla.org/show_bug.cgi?id=476567

Attachment 360222: Patch to fix
https://bugzilla.mozilla.org/attachment.cgi?id=360222&action=edit

------- Additional Comments from Jonas Sicking (:sicking) <jo...@sicking.cc>
The AC spec says that if a request fails, the spec says to clear all cache
entries related to that origin and target uri.

http://dev.w3.org/2006/waf/access-control/#preflight-result-cache-remove

The attached patch makes us do this.

I also made the cache tests use the new getState/setState mechanism that we
expose to sjs files. The old crazy header-name tricks were a pain to deal with
so this works better.

[bugzill...@mozilla.org]

Johnny Stenback (:jst, j...@mozilla.com) <j...@mozilla.org> has granted Jonas
Sicking (:sicking) <jo...@sicking.cc>'s request for superreview: