RESOLVED WONTFIX 14797
[gtk] Make GraphicsContext use a full GdkEventExpose* to ease implementing Widget::paint 
https://bugs.webkit.org/show_bug.cgi?id=14797
Summary [gtk] Make GraphicsContext use a full GdkEventExpose* to ease implementing Wi...
Holger Freyther
Reported 2007-07-27 20:06:45 PDT
To implement Widget::paint we need to paint at the moment RenderWidget is going to request painting. To ease that we will store the full GdkEventExpose inside the GraphicsContext. And create a synthetic GdkEvent and ask gtkWidget to paint, this should be roughly similiar to how GtkContainer is handling exposing of children and will continue to work once we have multiple frames in Gtk as well.
Attachments
Store GdkEventExpose* in GraphicsContext and implement Widget::paint (4.66 KB, patch)
2007-07-27 20:11 PDT, Holger Freyther 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Holger Freyther
Comment 1 2007-07-27 20:11:31 PDT
Created attachment 15715 [details] Store GdkEventExpose* in GraphicsContext and implement Widget::paint -No ChangeLog entry as it requires 14781 -This is a patch from git and won't apply with svn-apply -setGdkExposeEvent matches the Qt naming of such methods (e.g. mousePressEvent,...) but the parameter has it reversed with GdkEventExpose. I think the way I named the method follows the WebCore naming style and is the way to go. -I ignore the clip rect of Widget::paint but that should directly come from the expose event anyway.
Holger Freyther
Comment 2 2007-07-30 12:44:40 PDT
#14729 has this part included as well.
Alp Toker
Comment 3 2007-08-01 20:16:52 PDT
Comment on attachment 15715 [details] Store GdkEventExpose* in GraphicsContext and implement Widget::paint Seems to be obsoleted by http://bugs.webkit.org/attachment.cgi?id=15794
Note You need to log in before you can comment on or make changes to this bug.