Bug 73525

Summary: [Chromium] Use contentBounds instead of bounds for invalidation.
Product: WebKit Reporter: David Reveman <reveman>
Component: WebCore Misc.Assignee: David Reveman <reveman>
Status: RESOLVED FIXED    
Severity: Normal CC: cc-bugs, enne, jamesr, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 71872    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch none

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.