Add support for testing web notifications.
*** Bug 79493 has been marked as a duplicate of this bug. ***
<rdar://problem/10357618>
Created attachment 160656 [details] 1/9 - Update LayoutTest API for web notifications
Created attachment 160657 [details] 2/9 - Add SPI to retrieve internal IDs for notifications
Created attachment 160658 [details] 3/9 - Add SPI to be able to manually set permissions for origins in tests
Created attachment 160659 [details] 4/9 - DRT support
Created attachment 160660 [details] 5/9 - WTR support
Attachment 160657 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebKit2/WebProcess/Notifications/WebNotificationManager.h:72: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Source/WebKit/mac/WebCoreSupport/WebNotificationClient.h:53: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 2 in 15 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 160661 [details] 6/9 - Support for adding test output for platform events in DRT
Created attachment 160662 [details] 7/9 - Add WK2 support for that output
Created attachment 160665 [details] 8/9 - WTR support for dump output for platform events
Created attachment 160666 [details] 9/9 - Added basic tests in http/tests/notifications showing output from DRT and WTR for web notifications
Created attachment 160668 [details] All 9 patches as a whole, for EWS
Attachment 160668 [details] did not pass style-queue: Tools/WebKitTestRunner/TestController.h:34: Alphabetical sorting problem. [build/include_order] [4] Tools/DumpRenderTree/mac/MockWebNotificationProvider.h:61: Could not find a newline character at the end of the file. [whitespace/ending_newline] [5] Source/WebKit/mac/WebCoreSupport/WebNotificationClient.h:53: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Source/WebKit2/WebProcess/InjectedBundle/API/c/WKBundleNotification.cpp:27: You should add a blank line after implementation file's own header. [build/include_order] [4] Source/WebKit2/WebProcess/InjectedBundle/API/c/WKBundleNotification.cpp:29: Alphabetical sorting problem. [build/include_order] [4] Tools/WebKitTestRunner/WebNotificationProvider.h:29: Alphabetical sorting problem. [build/include_order] [4] Source/WebKit2/WebProcess/Notifications/WebNotificationManager.h:75: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageNotificationClient.h:50: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Source/WebKit2/WebProcess/InjectedBundle/InjectedBundlePageNotificationClient.h:50: The parameter name "origin" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:167: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:167: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:168: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:168: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:169: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:169: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:170: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:170: The parameter name "origin" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:171: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:171: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:172: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:172: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:173: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:173: The parameter name "notification" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:174: The parameter name "page" adds no information, so it should be removed. [readability/parameter_name] [5] Tools/WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:174: TFailed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1 he parameter name "origin" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 25 in 89 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 160657 [details] 2/9 - Add SPI to retrieve internal IDs for notifications Attachment 160657 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/13608293
Comment on attachment 160660 [details] 5/9 - WTR support Attachment 160660 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/13620021
Comment on attachment 160657 [details] 2/9 - Add SPI to retrieve internal IDs for notifications Attachment 160657 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/13620022
Comment on attachment 160660 [details] 5/9 - WTR support Attachment 160660 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/13609238
Comment on attachment 160657 [details] 2/9 - Add SPI to retrieve internal IDs for notifications Attachment 160657 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13615072
Please do not upload multiple r? patches at the same time. That's confusing and makes it difficult to comment on individual patches. The best work-around would be: - Create a meta bug and upload one patch that implements everything, so that reviewers can understand what you are going to do. - Create one bug per one sub-patch and set r?, blocking the meta bug. When the first patch is landed, you can upload the second patch etc.
Comment on attachment 160660 [details] 5/9 - WTR support Attachment 160660 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13613071
Comment on attachment 160656 [details] 1/9 - Update LayoutTest API for web notifications Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160657 [details] 2/9 - Add SPI to retrieve internal IDs for notifications Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160660 [details] 5/9 - WTR support Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160661 [details] 6/9 - Support for adding test output for platform events in DRT Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160662 [details] 7/9 - Add WK2 support for that output Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160665 [details] 8/9 - WTR support for dump output for platform events Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160658 [details] 3/9 - Add SPI to be able to manually set permissions for origins in tests Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160659 [details] 4/9 - DRT support Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160666 [details] 9/9 - Added basic tests in http/tests/notifications showing output from DRT and WTR for web notifications Not to show up these patches in the review list, let me r- them for now.
Comment on attachment 160668 [details] All 9 patches as a whole, for EWS Not to show up these patches in the review list, let me r- them for now.
(In reply to comment #20) > Please do not upload multiple r? patches at the same time. That's confusing and makes it difficult to comment on individual patches. > > The best work-around would be: > > - Create a meta bug and upload one patch that implements everything, so that reviewers can understand what you are going to do. > > - Create one bug per one sub-patch and set r?, blocking the meta bug. When the first patch is landed, you can upload the second patch etc. All right, I'll split the patches up into smaller bugs.
I ended up doing double duty bringing web notifications support to Mac. There are a couple patches that are common to both test harnesses, so this will act as the master bug for both.