NEW 124339
[SubPixel] One pixel shifts with two nested composited layers 
https://bugs.webkit.org/show_bug.cgi?id=124339
Summary [SubPixel] One pixel shifts with two nested composited layers
Allan Sandfeld Jensen
Reported 2013-11-14 01:44:42 PST
In bug #115304 I solved the problem with composited layers shifting a pixel when nested inside a multiple layers. The patch however did not solve the problem in all cases if the ancestor layers were also composited.
Attachments
Patch (7.82 KB, patch)
2013-11-14 01:51 PST, Allan Sandfeld Jensen 		beidson: review-
buildbot: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion (499.55 KB, application/zip)
2013-11-14 03:18 PST, Build Bot 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Allan Sandfeld Jensen
Comment 1 2013-11-14 01:51:52 PST
Created attachment 216910 [details] Patch
Build Bot
Comment 2 2013-11-14 03:18:28 PST
Comment on attachment 216910 [details] Patch Attachment 216910 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/22720149 New failing tests: media/video-fast-seek.html
Build Bot
Comment 3 2013-11-14 03:18:30 PST
Created attachment 216913 [details] Archive of layout-test-results from webkit-ews-08 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-08 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Allan Sandfeld Jensen
Comment 4 2013-11-15 07:11:32 PST
(In reply to comment #3) > Created an attachment (id=216913) [details] > Archive of layout-test-results from webkit-ews-08 for mac-mountainlion > > The attached test failures were seen while running run-webkit-tests on the mac-ews. > Bot: webkit-ews-08 Port: mac-mountainlion Platform: Mac OS X 10.8.5 I think this is a false negative. The code does not affect Mac (no subpixel layout), and does not affect media playback.
Brady Eidson
Comment 5 2016-05-24 22:05:15 PDT
Comment on attachment 216910 [details] Patch Assuming that patches for review since 2013 are stale, r-
Note You need to log in before you can comment on or make changes to this bug.