[webkit-reviews] review denied: [Bug 63326] [Qt][WK2] Don't run web process at a low priority : [Attachment 98497] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 28 11:32:37 PDT 2011


Benjamin Poulain <benjamin at webkit.org> has denied Siddharth Mathur
<siddharth.mathur at nokia.com>'s request for review:
Bug 63326: [Qt][WK2] Don't run web process at a low priority
https://bugs.webkit.org/show_bug.cgi?id=63326

Attachment 98497: patch
https://bugs.webkit.org/attachment.cgi?id=98497&action=review

------- Additional Comments from Benjamin Poulain <benjamin at webkit.org>
You should git-blame the file to see where that come from.

The UI must stay responsive, that means it should have more CPU time if it
needs it. You should prove you are not going below 60 FPS on device with this
patch. IIRC, Jocelyn refined the priority changes in the WebKit 2 branch, you
should have a look at that.


More information about the webkit-reviews mailing list