Bug 141797

Summary: Move WorkQueue from WK2 to WTF
Product: WebKit Reporter: Antti Koivisto <koivisto>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, commit-queue, ossy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch
andersca: review+
another
none
another
none
another
none
another
none
another none

Description Antti Koivisto 2015-02-19 06:58:36 PST
It might be useful in WebCore.
Comment 1 Antti Koivisto 2015-02-19 07:14:41 PST
Created attachment 246891 [details]
patch
Comment 2 WebKit Commit Bot 2015-02-19 07:17:08 PST
Attachment 246891 [details] did not pass style-queue:


ERROR: Source/WTF/wtf/WorkQueue.h:72:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:73:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:78:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:81:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:37:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:55:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:63:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:56:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:76:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:81:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:103:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:107:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:110:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:114:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:31:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:40:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 16 in 18 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Anders Carlsson 2015-02-19 13:45:17 PST
Comment on attachment 246891 [details]
patch

r=me if you fix the build.
Comment 4 Antti Koivisto 2015-02-20 02:28:02 PST
Created attachment 246952 [details]
another
Comment 5 WebKit Commit Bot 2015-02-20 02:30:03 PST
Attachment 246952 [details] did not pass style-queue:


ERROR: Source/WTF/wtf/WorkQueue.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:69:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:37:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:55:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:63:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:56:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:76:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:81:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:103:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:107:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:110:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:114:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:31:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:40:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 16 in 18 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 Antti Koivisto 2015-02-20 04:12:59 PST
Created attachment 246961 [details]
another
Comment 7 WebKit Commit Bot 2015-02-20 04:15:09 PST
Attachment 246961 [details] did not pass style-queue:


ERROR: Source/WTF/wtf/WorkQueue.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:69:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:37:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:55:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:63:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:56:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:76:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:81:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:103:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:107:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:110:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:114:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:31:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:40:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 16 in 18 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 8 Antti Koivisto 2015-02-20 05:08:40 PST
Created attachment 246964 [details]
another
Comment 9 WebKit Commit Bot 2015-02-20 05:11:15 PST
Attachment 246964 [details] did not pass style-queue:


ERROR: Source/WTF/wtf/WorkQueue.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:69:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:37:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:55:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:63:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:56:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:76:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:81:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:103:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:107:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:110:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:114:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:31:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:40:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 16 in 18 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 10 Antti Koivisto 2015-02-20 05:17:26 PST
Created attachment 246965 [details]
another
Comment 11 WebKit Commit Bot 2015-02-20 05:20:34 PST
Attachment 246965 [details] did not pass style-queue:


ERROR: Source/WTF/wtf/WorkQueue.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:69:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:37:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:55:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:63:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:58:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:78:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:83:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:105:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:109:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:112:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:116:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:31:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:40:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 16 in 18 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 12 Antti Koivisto 2015-02-20 05:25:12 PST
Created attachment 246967 [details]
another
Comment 13 WebKit Commit Bot 2015-02-20 05:30:36 PST
Attachment 246967 [details] did not pass style-queue:


ERROR: Source/WTF/wtf/WorkQueue.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:69:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:37:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:55:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/efl/WorkQueueEfl.cpp:63:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:59:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:79:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:84:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:106:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:110:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:113:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/gtk/WorkQueueGtk.cpp:117:  More than one command on the same line  [whitespace/newline] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:31:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:40:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 16 in 18 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 14 WebKit Commit Bot 2015-02-20 06:19:06 PST
Comment on attachment 246967 [details]
another

Clearing flags on attachment: 246967

Committed r180410: <http://trac.webkit.org/changeset/180410>
Comment 15 WebKit Commit Bot 2015-02-20 06:19:12 PST
All reviewed patches have been landed.  Closing bug.
Comment 16 Csaba Osztrogon√°c 2015-02-20 12:02:28 PST
(In reply to comment #14)
> Comment on attachment 246967 [details]
> another
> 
> Clearing flags on attachment: 246967
> 
> Committed r180410: <http://trac.webkit.org/changeset/180410>

EFL buildfixed landed in:
http://trac.webkit.org/changeset/180433
http://trac.webkit.org/changeset/180437 

Unfortunalty EWS didn't notice it in time, because 
WK2 developers broke the EFL build.