[webkit-reviews] review denied: [Bug 24853] Create a SecurityOriginDelegate which is called inn SecurityOrigin::canRequest() and ::canAccess() : [Attachment 28979] Create SecurityOriginDelegate that is called from SecurityOrigin::canAccess() & canRequest()

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has denied Rafael Weinstein
<rafaelw at chromium.org>'s request for review:
Bug 24853: Create a SecurityOriginDelegate which is called inn
SecurityOrigin::canRequest() and ::canAccess()
https://bugs.webkit.org/show_bug.cgi?id=24853

Attachment 28979: Create SecurityOriginDelegate that is called from
SecurityOrigin::canAccess() & canRequest()
https://bugs.webkit.org/attachment.cgi?id=28979&action=review

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
> This is necessary for the chromium extensions mechanism to have static,
> installed pages that have been granted permissions to make out-bound XHR
> requests.

Perhaps you could use a custom URL scheme for such static pages? These are
registered via FrameLoader::registerURLSchemeAsLocal() (it needs to be invoked
before secondary threads are spawned).

r-, as this feature doesn't seem to warrant introducing callbacks that work
from multiple threads.