Bug 155608 - Images in feed on ebay.com jiggle when one is hovered
Summary: Images in feed on ebay.com jiggle when one is hovered
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: zalan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2016-03-17 16:20 PDT by zalan
Modified: 2016-03-17 18:54 PDT (History)
5 users (show)

See Also:


Attachments
Patch (6.57 KB, patch)
2016-03-17 16:25 PDT, zalan
simon.fraser: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zalan 2016-03-17 16:20:12 PDT
1. Have some eBay searches set up, so that when you are logged into ebay, and go to eBay.com, you get a “feed” page with a wall of items.
2. Now hover over one image on the left side of a row. Note how the other images on that row jiggle by a pixel.
Comment 1 zalan 2016-03-17 16:20:26 PDT
rdar://problem/25160681
Comment 2 zalan 2016-03-17 16:25:24 PDT
Created attachment 274339 [details]
Patch
Comment 3 Simon Fraser (smfr) 2016-03-17 16:29:17 PDT
Comment on attachment 274339 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=274339&action=review

> Source/WebCore/rendering/RenderLayerBacking.h:370
> +    LayoutSize m_layerBoundsDistanceFromGraphicsLayer; // This is the (subpixel) distance between the edge of the graphics layer and the layer bounds.

Maybe "m_compositedBoundsDeltaFromGraphicsLayer?

> LayoutTests/compositing/hidpi-viewport-clipping-on-composited-content.html:34
> +for (i = 0; i < 50; ++i) {
> +    var element = document.createElement("div");
> +    element.className = "jiggle";
> +    element.style.top = (6 * i) + "px";
> +    element.style.left = leftPos + "px";
> +	document.body.appendChild(element);
> +	leftPos -= 0.1;
> +}

Does this fit in the 600px tall snapshot?
Comment 4 zalan 2016-03-17 18:54:03 PDT
Committed r198374: <http://trac.webkit.org/changeset/198374>