RESOLVED LATER 145790
[GTK] Allow to dispatch tasks in WorkQueue with different priorities 
https://bugs.webkit.org/show_bug.cgi?id=145790
Summary [GTK] Allow to dispatch tasks in WorkQueue with different priorities
Carlos Garcia Campos
Reported 2015-06-09 00:46:58 PDT
In glib we can use different priorities for the sources in the same main loop. I want to use this to give more priority to read tasks in the disk cache IO WorkQueue, but it could be useful for other cases as well.
Attachments
Patch (5.02 KB, patch)
2015-06-09 00:51 PDT, Carlos Garcia Campos 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2015-06-09 00:51:32 PDT
Created attachment 254554 [details] Patch
WebKit Commit Bot
Comment 2 2015-06-09 00:52:34 PDT
Attachment 254554 [details] did not pass style-queue: ERROR: Source/WTF/wtf/WorkQueue.h:77: 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/glib/WorkQueueGLib.cpp:114: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:121: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:126: Extra space before ( in function call [whitespace/parens] [4] ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:130: More than one command on the same line [whitespace/newline] [4] ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:133: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 7 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
Michael Catanzaro
Comment 3 2015-06-09 05:22:14 PDT
In the test case, why name the nonmember variables with the m_ prefix?
Carlos Garcia Campos
Comment 4 2015-06-09 05:28:29 PDT
(In reply to comment #3) > In the test case, why name the nonmember variables with the m_ prefix? Good question, there's no reason other than I copy-pasted from Tools/TestWebKitAPI/Tests/WTF/WorkQueue.cpp :-P
Carlos Garcia Campos
Comment 5 2015-06-09 10:13:47 PDT
We are no longer using this for the disk cache, so I guess we don't really need it for now.
Zan Dobersek
Comment 6 2015-11-23 01:11:51 PST
RESOLVED LATER?
Zan Dobersek
Comment 7 2015-11-23 01:18:52 PST
RESOLVED LATER.
Note You need to log in before you can comment on or make changes to this bug.