Bug 90873 - [WK2] WebKitTestRunner needs to print information about Web intents
Summary: [WK2] WebKitTestRunner needs to print information about Web intents
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on:
Blocks: 88303
  Show dependency treegraph
 
Reported: 2012-07-10 05:25 PDT by Chris Dumez
Modified: 2012-07-23 13:38 PDT (History)
12 users (show)

See Also:


Attachments
Patch (52.80 KB, patch)
2012-07-10 05:46 PDT, Chris Dumez
webkit-ews: commit-queue-
Details | Formatted Diff | Diff
Patch (53.56 KB, patch)
2012-07-10 07:19 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (53.56 KB, patch)
2012-07-10 07:25 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2012-07-10 05:25:48 PDT
The tests under webintents/* expect the WebKitTestRunner to print out information about new intent requests and new intent services registration.

Several test cases should start passing once the information is printed by WTR.
We will then need to implement testRunner.deliverWebIntent() and testRunner.sendWebIntentResponse() to get more test cases to pass (probably in a separate patch).
Comment 1 Chris Dumez 2012-07-10 05:46:41 PDT
Created attachment 151446 [details]
Patch
Comment 2 Early Warning System Bot 2012-07-10 05:57:52 PDT
Comment on attachment 151446 [details]
Patch

Attachment 151446 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/13186087
Comment 3 Chris Dumez 2012-07-10 07:19:16 PDT
Created attachment 151456 [details]
Patch

Should fix Qt-WK2 build issue.
Comment 4 WebKit Review Bot 2012-07-10 07:22:25 PDT
Attachment 151456 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/CMakeLists.txt', u'Source/W..." exit_code: 1
Source/WebKit2/WebProcess/qt/QtBuiltinBundlePage.cpp:80:  One space before end of line comments  [whitespace/comments] [5]
Total errors found: 1 in 24 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Chris Dumez 2012-07-10 07:25:31 PDT
Created attachment 151458 [details]
Patch
Comment 6 Gyuyoung Kim 2012-07-11 18:06:47 PDT
Comment on attachment 151458 [details]
Patch

Though this is a little huge to review, looks good to me. BTW, I wonder how many test cases are covered by this patch? If this patch can cover webintents test cases, please write it to ChangeLog as well.
Comment 7 Chris Dumez 2012-07-11 22:49:53 PDT
(In reply to comment #6)
> (From update of attachment 151458 [details])
> Though this is a little huge to review, looks good to me. BTW, I wonder how many test cases are covered by this patch? If this patch can cover webintents test cases, please write it to ChangeLog as well.

The Changelog already states:
"Print out information about new intents requests and intent services registrations in WebKitTestRunner. This is expected by tests under webintents/*."

So the change is covered by pretty much all the tests under webintents/*. How do you suggest I improve the changelog?
Comment 8 WebKit Review Bot 2012-07-23 13:37:59 PDT
Comment on attachment 151458 [details]
Patch

Clearing flags on attachment: 151458

Committed r123374: <http://trac.webkit.org/changeset/123374>
Comment 9 WebKit Review Bot 2012-07-23 13:38:06 PDT
All reviewed patches have been landed.  Closing bug.