WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
90354
Position replaced elements on pixel bounds
https://bugs.webkit.org/show_bug.cgi?id=90354
Summary
Position replaced elements on pixel bounds
Emil A Eklund
Reported
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
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
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Emil A Eklund
Comment 1
2012-07-01 16:40:03 PDT
Created
attachment 150343
[details]
Patch
WebKit Review Bot
Comment 2
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
WebKit Review Bot
Comment 3
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
Eric Seidel (no email)
Comment 4
2012-07-02 09:03:24 PDT
Comment on
attachment 150343
[details]
Patch Looks reasonable, but clearly you have some expecatations to update yet. :)
Emil A Eklund
Comment 5
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.
Adam Barth
Comment 6
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.
Emil A Eklund
Comment 7
2012-07-02 16:34:40 PDT
Created
attachment 150495
[details]
Patch for landing
WebKit Review Bot
Comment 8
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
>
WebKit Review Bot
Comment 9
2012-07-02 19:23:22 PDT
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.
Top of Page
Format For Printing
XML
Clone This Bug