RESOLVED FIXED 90422
[BlackBerry] Enable registerProtocolHandler for Blackberry 
https://bugs.webkit.org/show_bug.cgi?id=90422
Summary [BlackBerry] Enable registerProtocolHandler for Blackberry
George Staikos
Reported 2012-07-02 18:38:57 PDT
Enable protocol handlers for the BlackBerry port. This will be a three-stage set of patches.
Attachments
Patch part 1/3 - Enable RegisterProtocolHandler and stub out (4.83 KB, patch)
2012-07-02 18:46 PDT, George Staikos 		no flags
DetailsFormatted DiffDiff
Part 2/3 - Enable custom scheme handlers, stub, and remove the whitelist for BlackBerry (4.83 KB, text/plain)
2012-07-02 20:51 PDT, George Staikos 		no flags
Details
Part 2/3 - Enable custom scheme handlers, stub, and remove the whitelist for BlackBerry (6.40 KB, patch)
2012-07-02 20:53 PDT, George Staikos 		no flags
DetailsFormatted DiffDiff
Patch (5.46 KB, patch)
2012-07-16 03:02 PDT, Chris.Guan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
George Staikos
Comment 1 2012-07-02 18:46:09 PDT
Created attachment 150515 [details] Patch part 1/3 - Enable RegisterProtocolHandler and stub out Each patch here builds on the previous and I would like to apply them sequentially to avoid churn. Also someone else will be submitting the third patch.
George Staikos
Comment 2 2012-07-02 20:51:55 PDT
Created attachment 150524 [details] Part 2/3 - Enable custom scheme handlers, stub, and remove the whitelist for BlackBerry
George Staikos
Comment 3 2012-07-02 20:52:41 PDT
Comment on attachment 150524 [details] Part 2/3 - Enable custom scheme handlers, stub, and remove the whitelist for BlackBerry Typo
George Staikos
Comment 4 2012-07-02 20:53:25 PDT
Created attachment 150525 [details] Part 2/3 - Enable custom scheme handlers, stub, and remove the whitelist for BlackBerry
Rob Buis
Comment 5 2012-07-03 07:56:07 PDT
Comment on attachment 150515 [details] Patch part 1/3 - Enable RegisterProtocolHandler and stub out View in context: https://bugs.webkit.org/attachment.cgi?id=150515&action=review LGTM. > Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp:814 > +void ChromeClientBlackBerry::registerProtocolHandler(const WTF::String&, const WTF::String&, const WTF::String&, const WTF::String&) You probably do not need the WTF prefixes.
Rob Buis
Comment 6 2012-07-03 07:58:47 PDT
Comment on attachment 150525 [details] Part 2/3 - Enable custom scheme handlers, stub, and remove the whitelist for BlackBerry LGTM.
George Staikos
Comment 7 2012-07-03 08:47:50 PDT
(In reply to comment #5) > (From update of attachment 150515 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=150515&action=review > > LGTM. > > > Source/WebKit/blackberry/WebCoreSupport/ChromeClientBlackBerry.cpp:814 > > +void ChromeClientBlackBerry::registerProtocolHandler(const WTF::String&, const WTF::String&, const WTF::String&, const WTF::String&) > > You probably do not need the WTF prefixes. Will check this in the following patch(es). I believe you're right, though of course it should be harmless.
WebKit Review Bot
Comment 8 2012-07-03 08:53:48 PDT
Comment on attachment 150515 [details] Patch part 1/3 - Enable RegisterProtocolHandler and stub out Clearing flags on attachment: 150515 Committed r121773: <http://trac.webkit.org/changeset/121773>
WebKit Review Bot
Comment 9 2012-07-03 09:00:39 PDT
Comment on attachment 150525 [details] Part 2/3 - Enable custom scheme handlers, stub, and remove the whitelist for BlackBerry Clearing flags on attachment: 150525 Committed r121774: <http://trac.webkit.org/changeset/121774>
WebKit Review Bot
Comment 10 2012-07-03 09:00:45 PDT
All reviewed patches have been landed. Closing bug.
George Staikos
Comment 11 2012-07-03 09:02:24 PDT
Reopening for the rest.
Chris.Guan
Comment 12 2012-07-16 03:02:58 PDT
Created attachment 152503 [details] Patch
WebKit Review Bot
Comment 13 2012-07-17 03:23:39 PDT
Comment on attachment 152503 [details] Patch Clearing flags on attachment: 152503 Committed r122832: <http://trac.webkit.org/changeset/122832>
WebKit Review Bot
Comment 14 2012-07-17 03:23:44 PDT
All reviewed patches have been landed. Closing bug.
Gyuyoung Kim
Comment 15 2012-07-17 03:33:50 PDT
I have a question. I land a RegisterProtocolHandlerClient to Modules/protocolhandler today. http://trac.webkit.org/changeset/122810 Don't you need to adjust it as well ?
Chris.Guan
Comment 16 2012-07-17 03:58:16 PDT
(In reply to comment #15) > I have a question. I land a RegisterProtocolHandlerClient to Modules/protocolhandler today. > http://trac.webkit.org/changeset/122810 > > Don't you need to adjust it as well ? Thanks, Gyuyoung Kim. We saw your two patches for modular and client. I prefer sync them up in other bugs. Anyway, many thanks for your reminder:)
Gyuyoung Kim
Comment 17 2012-07-17 04:11:53 PDT
(In reply to comment #16) > (In reply to comment #15) > > I have a question. I land a RegisterProtocolHandlerClient to Modules/protocolhandler today. > > http://trac.webkit.org/changeset/122810 > > > > Don't you need to adjust it as well ? > > Thanks, Gyuyoung Kim. We saw your two patches for modular and client. I prefer sync them up in other bugs. Anyway, many thanks for your reminder:) Ok, thanks.
Note You need to log in before you can comment on or make changes to this bug.