Bug 16388

Summary: [GTK] Widget::setCursor() gets called frequently
Product: WebKit Reporter: Alp Toker <alp>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal Keywords: Gtk
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Workaround (cache the current GdkCursor*) mjs: review+

Alp Toker
Reported 2007-12-10 14:33:01 PST
Widget::setCursor() seems to get called on every mouse motion event and it seems the underlying implementation gdk_window_set_cursor() isn't too cheap. This looks like it might be a WebCore bug. Confirmation on ports other than GTK+ would be helpful.
Attachments
Workaround (cache the current GdkCursor*) (1.84 KB, patch)
2007-12-10 14:34 PST, Alp Toker 		mjs: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alp Toker
Comment 1 2007-12-10 14:34:46 PST
Created attachment 17827 [details] Workaround (cache the current GdkCursor*)
Maciej Stachowiak
Comment 2 2007-12-11 19:26:27 PST
Comment on attachment 17827 [details] Workaround (cache the current GdkCursor*) r=me
Alp Toker
Comment 3 2007-12-12 09:34:33 PST
Landed in r28655.
Note You need to log in before you can comment on or make changes to this bug.