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

Description Emil A Eklund 2012-04-03 14:34:41 PDT
Fix usage of LayoutUnits and snapping for scrolling in RenderBox
Comment 1 Emil A Eklund 2012-04-03 14:48:27 PDT
Created attachment 135434 [details]
Patch
Comment 2 WebKit Review Bot 2012-04-03 16:35:40 PDT
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
Comment 3 WebKit Review Bot 2012-04-03 16:35:46 PDT
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
Comment 4 Emil A Eklund 2012-04-03 17:05:47 PDT
Created attachment 135461 [details]
Patch
Comment 5 Emil A Eklund 2012-04-03 17:13:41 PDT
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.
Comment 6 Eric Seidel (no email) 2012-04-03 19:08:35 PDT
Comment on attachment 135461 [details]
Patch

OK.
Comment 7 WebKit Review Bot 2012-04-04 12:10:18 PDT
Comment on attachment 135461 [details]
Patch

Clearing flags on attachment: 135461

Committed r113223: <http://trac.webkit.org/changeset/113223>
Comment 8 WebKit Review Bot 2012-04-04 12:10:32 PDT
All reviewed patches have been landed.  Closing bug.