Bug 90354 - Position replaced elements on pixel bounds
Summary: Position replaced elements on pixel bounds
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Emil A Eklund
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-01 16:36 PDT by Emil A Eklund
Modified: 2012-07-02 19:23 PDT (History)
5 users (show)

See Also:


Attachments
Patch (13.48 KB, patch)
2012-07-01 16:40 PDT, Emil A Eklund
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-03 (748.26 KB, application/zip)
2012-07-01 17:47 PDT, WebKit Review Bot
no flags Details
Patch for landing (25.00 KB, patch)
2012-07-02 16:34 PDT, Emil A Eklund
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Emil A Eklund 2012-07-01 16:36:35 PDT
To avoid sizing and repaint issues we should layout replaced elements on pixel bounds. We already ensure that replaced elements are sized in full pixels and that they are painted on pixel bounds. By also ensuring that they are placed on pixel bounds we avoid pixel having the size be expanded by pixel snapping and repainting/invalidation rect issues when scrolling.

Downstream chromium bug: http://code.google.com/p/chromium/issues/detail?id=133792
Comment 1 Emil A Eklund 2012-07-01 16:40:03 PDT
Created attachment 150343 [details]
Patch
Comment 2 WebKit Review Bot 2012-07-01 17:47:27 PDT
Comment on attachment 150343 [details]
Patch

Attachment 150343 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13131010

New failing tests:
platform/chromium/compositing/layout-width-change.html
platform/chromium/compositing/render-surface-alpha-blending.html
platform/chromium/compositing/tiny-layer-rotated.html
platform/chromium/compositing/huge-layer-rotated.html
fast/loader/loadInProgress.html
platform/chromium/compositing/3d-corners.html
platform/chromium/compositing/video-frame-size-change.html
platform/chromium/compositing/perpendicular-layer-sorting.html
fast/loader/unload-form-post-about-blank.html
platform/chromium/compositing/child-layer-3d-sorting.html
platform/chromium/compositing/lost-compositor-context-permanently.html
platform/chromium/compositing/lost-compositor-context-with-video.html
http/tests/xmlhttprequest/zero-length-response.html
http/tests/security/script-crossorigin-loads-correctly.html
fast/repaint/block-selection-gap-in-composited-layer.html
media/media-document-audio-repaint.html
platform/chromium/compositing/backface-visibility-transformed.html
platform/chromium/compositing/lost-compositor-context-with-rendersurface.html
platform/chromium/compositing/lost-compositor-context.html
inspector/timeline/timeline-receive-response-event.html
platform/chromium/compositing/img-layer-grow.html
platform/chromium/compositing/lost-compositor-context-twice.html
Comment 3 WebKit Review Bot 2012-07-01 17:47:31 PDT
Created attachment 150348 [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 Eric Seidel (no email) 2012-07-02 09:03:24 PDT
Comment on attachment 150343 [details]
Patch

Looks reasonable, but clearly you have some expecatations to update yet. :)
Comment 5 Emil A Eklund 2012-07-02 12:15:22 PDT
The chromium ews bot test results don't match what I'm seeing locally. Looking into it now.
Comment 6 Adam Barth 2012-07-02 14:02:46 PDT
> The chromium ews bot test results don't match what I'm seeing locally. Looking into it now.

We just switched hardware for the bots, so I wouldn't be surprised if it was having some trouble.
Comment 7 Emil A Eklund 2012-07-02 16:34:40 PDT
Created attachment 150495 [details]
Patch for landing
Comment 8 WebKit Review Bot 2012-07-02 19:23:06 PDT
Comment on attachment 150495 [details]
Patch for landing

Clearing flags on attachment: 150495

Committed r121722: <http://trac.webkit.org/changeset/121722>
Comment 9 WebKit Review Bot 2012-07-02 19:23:22 PDT
All reviewed patches have been landed.  Closing bug.