Bug 43994 - [GTK] GTK+ drawing idle starved
Summary: [GTK] GTK+ drawing idle starved
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Gustavo Noronha (kov)
URL: http://themaninblue.com/experiment/An...
Keywords: Gtk
Depends on:
Blocks:
 
Reported: 2010-08-13 15:50 PDT by Gustavo Noronha (kov)
Modified: 2010-08-16 10:52 PDT (History)
4 users (show)

See Also:


Attachments
proposed fix (1.49 KB, patch)
2010-08-13 15:59 PDT, Gustavo Noronha (kov)
kenneth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gustavo Noronha (kov) 2010-08-13 15:50:23 PDT
The page at http://themaninblue.com/experiment/AnimationBenchmark/canvas/ is able to starve the idle that is responsible for redrawing in GTK+. We need to further fine tune the priorities we use in our various timeouts.
Comment 1 Gustavo Noronha (kov) 2010-08-13 15:59:24 PDT
Created attachment 64383 [details]
proposed fix

This fixes the issue with that site for me, even with the browser under great stress, such as playing flash videos, some chromeexperiments, the "video blow up" demo, and playing http://trailers.apple.com/movies/paramount/ironman2/ironman2-34rgrwt9-tlr2_720p.mov. The priority used is the one GTK+ uses for its redraw idle.
Comment 2 Martin Robinson 2010-08-14 17:43:04 PDT
Wow! That was a fast fix. Maybe you should change the priority to GDK_PRIORITY_REDRAW? http://library.gnome.org/devel/gdk/stable/gdk-Events.html#GDK-PRIORITY-REDRAW:CAPS
Comment 3 Gustavo Noronha (kov) 2010-08-16 06:12:39 PDT
I could, but then I'd need to add the gdk include to this file, do you think it's worth it? =)
Comment 4 Martin Robinson 2010-08-16 08:12:54 PDT
Hrm. My preference is either the GDK include or a comment explaining where the "+ 20" comes from.
Comment 5 Gustavo Noronha (kov) 2010-08-16 10:12:01 PDT
I landed this as r65438 adding the include. Thanks for the reviews!
Comment 6 WebKit Review Bot 2010-08-16 10:52:59 PDT
http://trac.webkit.org/changeset/65438 might have broken SnowLeopard Intel Release (Tests)