RESOLVED FIXED 73525
[Chromium] Use contentBounds instead of bounds for invalidation. 
https://bugs.webkit.org/show_bug.cgi?id=73525
Summary [Chromium] Use contentBounds instead of bounds for invalidation.
David Reveman
Reported 2011-11-30 22:53:41 PST
Use setNeedsDisplay() instead of setNeedsDisplayRect() when possible.
Attachments
Patch (1.22 KB, patch)
2011-11-30 23:02 PST, David Reveman 		no flags
DetailsFormatted DiffDiff
Patch (3.01 KB, patch)
2011-12-01 10:09 PST, David Reveman 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
David Reveman
Comment 1 2011-11-30 23:02:40 PST
Created attachment 117348 [details] Patch
Adrienne Walker
Comment 2 2011-12-01 09:18:01 PST
This looks great. This needs a ChangeLog, or you could probably just wrap this into your other invalidation-changing patch if you wanted.
Adrienne Walker
Comment 3 2011-12-01 09:41:42 PST
(In reply to comment #2) > This looks great. > > This needs a ChangeLog, or you could probably just wrap this into your other invalidation-changing patch if you wanted. Oh, you already landed that, whoops. :)
David Reveman
Comment 4 2011-12-01 10:09:48 PST
Created attachment 117440 [details] Patch
WebKit Review Bot
Comment 5 2011-12-01 13:32:26 PST
Comment on attachment 117440 [details] Patch Clearing flags on attachment: 117440 Committed r101701: <http://trac.webkit.org/changeset/101701>
WebKit Review Bot
Comment 6 2011-12-01 13:32:30 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.