RESOLVED FIXED 21328
Make invalidation of widgets more cross-platform 
https://bugs.webkit.org/show_bug.cgi?id=21328
Summary Make invalidation of widgets more cross-platform
Dave Hyatt
Reported 2008-10-02 23:40:56 PDT
Make invalidation of widgets more cross-platform
Attachments
Patch (31.95 KB, patch)
2008-10-02 23:45 PDT, Dave Hyatt 		sam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dave Hyatt
Comment 1 2008-10-02 23:45:01 PDT
Created attachment 24041 [details] Patch
Sam Weinig
Comment 2 2008-10-03 00:00:55 PDT
Comment on attachment 24041 [details] Patch Couple of nits. + ::InvalidateRect(m_popup, &r,false); Missing space. +/* void Widget::invalidateRect(const IntRect& r) Please remove this commented out code. + // Do windowless plugin invalidation. + invalidateWindowlessPluginRect(rect); The comment doesn't really add anything here. + PluginWidget(NSView *view = 0) + : Widget(view) The initializer list should be indented more. + [platformWidget() setNeedsDisplayInRect: rect]; Extra space. - : Widget(view) + : PluginWidget(view) Initialize list issue again. r=me!
Dave Hyatt
Comment 3 2008-10-03 00:06:49 PDT
Fixed in r37223.
Note You need to log in before you can comment on or make changes to this bug.