Bug 108457

Summary: [EFL] Disable Web Intents
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit EFLAssignee: Chris Dumez <cdumez>
Severity: Normal CC: abarth, andersca, ap, benjamin, dbates, gyuyoung.kim, kenneth, kihong.kwon, lucas.de.marchi, rakuco, simon.fraser, thakis, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description Chris Dumez 2013-01-31 04:15:47 PST
The Web Intents spec seems to be pretty much dead and Google is planning to remove support for it from WebKit soon (See mailing list announcement). We should therefore disable Web Intents in EFL port and remove the corresponding code.
Comment 1 Kenneth Rohde Christiansen 2013-01-31 04:38:25 PST
I totally agree
Comment 2 Gyuyoung Kim 2013-01-31 04:44:25 PST
AFAIK, Tizen WebKit also disable it because of same reason. CC'ing Kihong.
Comment 3 Chris Dumez 2013-01-31 05:44:17 PST
(In reply to comment #2)
> AFAIK, Tizen WebKit also disable it because of same reason. CC'ing Kihong.

Yes, it was never enabled for Tizen AFAIK.
Comment 4 Chris Dumez 2013-01-31 05:49:20 PST
Created attachment 185764 [details]
Comment 5 Kenneth Rohde Christiansen 2013-01-31 05:51:17 PST
Comment on attachment 185764 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=185764&action=review


> Tools/DumpRenderTree/efl/TestRunnerEfl.cpp:911
> -    DumpRenderTreeSupportEfl::deliverWebIntent(browser->mainFrame(), action, type, data);
> +    notImplemented();

maybe add a comment that it is not intended to be implemented?
Comment 6 Chris Dumez 2013-01-31 05:54:54 PST
Created attachment 185766 [details]

Take Kenneth's comments into consideration.
Comment 7 Alexey Proskuryakov 2013-01-31 10:07:43 PST
Comment on attachment 185766 [details]

Comment 8 WebKit Review Bot 2013-01-31 10:45:16 PST
Comment on attachment 185766 [details]

Clearing flags on attachment: 185766

Committed r141439: <http://trac.webkit.org/changeset/141439>
Comment 9 WebKit Review Bot 2013-01-31 10:45:21 PST
All reviewed patches have been landed.  Closing bug.