Bug 83073

Summary: Fix usage of LayoutUnits and snapping for scrolling in RenderBox
Product: WebKit Reporter: Emil A Eklund <eae>
Component: Layout and RenderingAssignee: Emil A Eklund <eae>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, eric, jchaffraix, leviw, simon.fraser, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 60318    
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ec2-cr-linux-03
none
Patch none

Emil A Eklund
Reported Tuesday, April 3, 2012 10:34:41 PM UTC
Fix usage of LayoutUnits and snapping for scrolling in RenderBox
Attachments
Patch (7.41 KB, patch)
2012-04-03 14:48 PDT, Emil A Eklund
no flags
Archive of layout-test-results from ec2-cr-linux-03 (6.53 MB, application/zip)
2012-04-03 16:35 PDT, WebKit Review Bot
no flags
Patch (7.23 KB, patch)
2012-04-03 17:05 PDT, Emil A Eklund
no flags
Emil A Eklund
Comment 1 Tuesday, April 3, 2012 10:48:27 PM UTC
WebKit Review Bot
Comment 2 Wednesday, April 4, 2012 12:35:40 AM UTC
Comment on attachment 135434 [details] Patch Attachment 135434 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12315008 New failing tests: fast/dom/Element/scrollWidth.html
WebKit Review Bot
Comment 3 Wednesday, April 4, 2012 12:35:46 AM UTC
Created attachment 135453 [details] Archive of layout-test-results from ec2-cr-linux-03 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-03 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Emil A Eklund
Comment 4 Wednesday, April 4, 2012 1:05:47 AM UTC
Emil A Eklund
Comment 5 Wednesday, April 4, 2012 1:13:41 AM UTC
Turns out we do not snap the scrollWidth in RTL for a reason, had forgotten about that. Good thing we have great the test coverage for this code.
Eric Seidel (no email)
Comment 6 Wednesday, April 4, 2012 3:08:35 AM UTC
Comment on attachment 135461 [details] Patch OK.
WebKit Review Bot
Comment 7 Wednesday, April 4, 2012 8:10:18 PM UTC
Comment on attachment 135461 [details] Patch Clearing flags on attachment: 135461 Committed r113223: <http://trac.webkit.org/changeset/113223>
WebKit Review Bot
Comment 8 Wednesday, April 4, 2012 8:10:32 PM UTC
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.