Bug 172156

Summary: Bring Notification.idl up to spec
Product: WebKit Reporter: Sam Weinig <sam>
Component: New BugsAssignee: Sam Weinig <sam>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, cdumez, commit-queue, sam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
URL: https://notifications.spec.whatwg.org/#api
Attachments:
Description Flags
Patch
none
Patch none

Description Sam Weinig 2017-05-15 19:29:51 PDT
Bring Notification.idl up to spec
Comment 1 Sam Weinig 2017-05-16 10:47:41 PDT
Created attachment 310274 [details]
Patch
Comment 2 Build Bot 2017-05-16 10:50:48 PDT
Attachment 310274 [details] did not pass style-queue:


ERROR: Source/WebKit/mac/WebView/WebNotification.mm:138:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Total errors found: 1 in 34 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Sam Weinig 2017-05-16 15:02:31 PDT
Created attachment 310302 [details]
Patch
Comment 4 Build Bot 2017-05-16 15:05:16 PDT
Attachment 310302 [details] did not pass style-queue:


ERROR: Source/WebKit/mac/WebView/WebNotification.mm:138:  A case label should not be indented, but line up with its switch statement.  [whitespace/indent] [4]
Total errors found: 1 in 34 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Chris Dumez 2017-05-16 15:56:00 PDT
BTW, it seems there are notifications tests in web-platform-tests we may want to import.
Comment 6 Sam Weinig 2017-05-16 16:06:54 PDT
Ok, filed https://bugs.webkit.org/show_bug.cgi?id=172196 to track that.
Comment 7 Chris Dumez 2017-05-16 19:54:43 PDT
Comment on attachment 310302 [details]
Patch

r=me
Comment 8 WebKit Commit Bot 2017-05-16 20:22:42 PDT
Comment on attachment 310302 [details]
Patch

Clearing flags on attachment: 310302

Committed r216967: <http://trac.webkit.org/changeset/216967>
Comment 9 WebKit Commit Bot 2017-05-16 20:22:44 PDT
All reviewed patches have been landed.  Closing bug.