RESOLVED FIXED 60322
Implement whitelist for registerProtocolHandler 
https://bugs.webkit.org/show_bug.cgi?id=60322
Summary Implement whitelist for registerProtocolHandler
James Kozianski
Reported 2011-05-05 17:12:20 PDT
Implement whitelist for registerProtocolHandler
Attachments
Patch (4.46 KB, patch)
2011-05-05 17:13 PDT, James Kozianski 		eric: review+
abarth: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
James Kozianski
Comment 1 2011-05-05 17:13:56 PDT
Created attachment 92507 [details] Patch
Eric Seidel (no email)
Comment 2 2011-05-11 19:55:36 PDT
Comment on attachment 92507 [details] Patch Seems reasonable.
Adam Barth
Comment 3 2011-05-11 20:11:00 PDT
Comment on attachment 92507 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=92507&action=review > Source/WebCore/page/Navigator.cpp:236 > + if (scheme.startsWith("web+")) { Is there a spec for this scheme prefix? This feels like the kind of thing that would make IETFers go bananas.
Adam Barth
Comment 4 2011-05-11 20:13:08 PDT
I see. It was hixie's idea.
James Kozianski
Comment 5 2011-05-26 18:17:45 PDT
Committed r87459: <http://trac.webkit.org/changeset/87459>
rektide
Comment 6 2011-06-09 11:48:18 PDT
Why was this patch done? Is it part of any spec? It mutes the ability of registerProtocolHandler to extend the web, and strikes me as a drastic overreaction to the need for a protective black-list. Mega :(
Note You need to log in before you can comment on or make changes to this bug.