Bug 181146

Summary: WebKit fixes to enable -Wstrict-prototypes for Safari
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: WebKit Misc.Assignee: David Kilzer (:ddkilzer) <ddkilzer>
Status: RESOLVED FIXED    
Severity: Normal CC: aestes, ews-watchlist, joepeck, mitz, rniwa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=180757
Attachments:
Description Flags
Patch v1
none
Patch v2
none
Archive of layout-test-results from ews105 for mac-elcapitan-wk2 none

Description David Kilzer (:ddkilzer) 2017-12-23 17:09:52 PST
Safari uses some WebKit SPI that needs to be fixed in order to enable -Wstrict-prototypes.
Comment 1 David Kilzer (:ddkilzer) 2017-12-24 09:33:54 PST
Created attachment 330172 [details]
Patch v1
Comment 2 Radar WebKit Bug Importer 2017-12-24 09:34:57 PST
<rdar://problem/36206257>
Comment 3 EWS Watchlist 2017-12-24 09:35:41 PST
Attachment 330172 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/ios/wak/WebCoreThreadRun.cpp:136:  _WebThreadRun is incorrectly named. Don't use underscores in your identifier names.  [readability/naming/underscores] [4]
Total errors found: 1 in 12 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 David Kilzer (:ddkilzer) 2017-12-24 09:43:40 PST
Created attachment 330173 [details]
Patch v2
Comment 5 EWS Watchlist 2017-12-24 09:47:08 PST
Attachment 330173 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/ios/wak/WebCoreThreadRun.cpp:136:  _WebThreadRun is incorrectly named. Don't use underscores in your identifier names.  [readability/naming/underscores] [4]
Total errors found: 1 in 12 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 EWS Watchlist 2017-12-24 10:56:51 PST
Comment on attachment 330173 [details]
Patch v2

Attachment 330173 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/5819881

New failing tests:
imported/w3c/web-platform-tests/service-workers/service-worker/ServiceWorkerGlobalScope/update.https.html
Comment 7 EWS Watchlist 2017-12-24 10:56:52 PST
Created attachment 330174 [details]
Archive of layout-test-results from ews105 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 8 David Kilzer (:ddkilzer) 2017-12-24 13:35:01 PST
Comment on attachment 330173 [details]
Patch v2

Clearing flags on attachment: 330173

Committed r226291: <https://trac.webkit.org/changeset/226291>
Comment 9 David Kilzer (:ddkilzer) 2017-12-24 13:35:03 PST
All reviewed patches have been landed.  Closing bug.