We should use more expected and have a single error enumeration.
Created attachment 318911 [details] Patch
Created attachment 318917 [details] Patch
Attachment 318917 [details] did not pass style-queue: ERROR: Source/WebCore/Modules/cache/DOMCache.h:48: The parameter name "request" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 1 in 23 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 318919 [details] Patch
Attachment 318919 [details] did not pass style-queue: ERROR: Source/WebCore/Modules/cache/DOMCache.h:48: The parameter name "request" adds no information, so it should be removed. [readability/parameter_name] [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 on attachment 318919 [details] Patch Clearing flags on attachment: 318919 Committed r221112: <http://trac.webkit.org/changeset/221112>
All reviewed patches have been landed. Closing bug.
<rdar://problem/34045933>
(In reply to WebKit Commit Bot from comment #6) > Comment on attachment 318919 [details] > Patch > > Clearing flags on attachment: 318919 > > Committed r221112: <http://trac.webkit.org/changeset/221112> This change broke the Windows build: https://build.webkit.org/builders/Apple%20Win%20Release%20(Build)/builds/3884
Reopening to attach new patch.
Created attachment 318942 [details] Patch - windows fix
Comment on attachment 318942 [details] Patch - windows fix Clearing flags on attachment: 318942 Committed r221123: <http://trac.webkit.org/changeset/221123>