Summary: | [WK2] Replace getRect() to pixelSnappedBoundingBox(). | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eunmi Lee <enmi.lee> | ||||||||||
Component: | WebKit2 | Assignee: | Eunmi Lee <enmi.lee> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | andersca, benjamin, cdumez, commit-queue, gyuyoung.kim, noam, sam, simon.fraser | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 118585 | ||||||||||||
Attachments: |
|
Description
Eunmi Lee
2013-07-12 01:29:06 PDT
Created attachment 206512 [details]
Patch
Comment on attachment 206512 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=206512&action=review > Source/WebKit2/ChangeLog:3 > + Replace getRect() to pixelSnappedBoundingBox(). Missing [WK2] tag. (In reply to comment #2) > (From update of attachment 206512 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=206512&action=review > > > Source/WebKit2/ChangeLog:3 > > + Replace getRect() to pixelSnappedBoundingBox(). > > Missing [WK2] tag. Thanks, I will add [WK2]. :) Created attachment 206726 [details]
Patch
Add [WK2] tag.
Comment on attachment 206726 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=206726&action=review > Source/WebKit2/ChangeLog:9 > + The Node's getRect() function was removed, so use > + pixelSnappedBoundingBox() instead. Was it removed in a particular revision? Can you name the revision in the changelog? Comment on attachment 206726 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=206726&action=review >> Source/WebKit2/ChangeLog:9 >> + pixelSnappedBoundingBox() instead. > > Was it removed in a particular revision? Can you name the revision in the changelog? Sure, I will update the changelog. Created attachment 206954 [details]
Patch
Add revision which removes getRect() to the changelog.
Comment on attachment 206954 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=206954&action=review > Source/WebKit2/ChangeLog:8 > + The Node's getRect() function was removed in the Say Node::getRect() instead of The Node's getRect() function Otherwise LGTM Change component with WebKit2. Created attachment 206955 [details]
Patch
Comment on attachment 206955 [details] Patch Clearing flags on attachment: 206955 Committed r152837: <http://trac.webkit.org/changeset/152837> All reviewed patches have been landed. Closing bug. |