90169 2012-06-28 06:28:49 -0700 Effective Image size deviates from width and height attributes with sub-pixel layout 2012-07-19 02:11:47 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED DUPLICATE 91464 P2 Normal --- 60318 89458 1 d-r webkit-unassigned eae leviw tmpsantos oldest_to_newest 659458 0 149937 d-r 2012-06-28 06:28:49 -0700 Created attachment 149937 014-wdiff.html failure When enabling sub-pixel layout, the DRT values for RenderImages are deviating from the specified width and height. This shows for example in fast/block/basic/014-wdiff.html (diff attached). I don't see a reason why the image size should be changed from what's specified as height (21) to 22 pixels height and then moved one pixel up (-1). I'm investigating, but if this is a known issue, Levi & Emil - please let me know. 662179 1 150597 d-r 2012-07-03 06:34:59 -0700 Created attachment 150597 Simplified test case This test case still results in layer at (0,0) size 800x600 RenderView at (0,0) size 800x600 layer at (0,0) size 800x600 RenderBlock {HTML} at (0,0) size 800x600 RenderBody {BODY} at (8,8) size 784x584 RenderImage {IMG} at (0,-1) size 21x22 [bgcolor=#008000] RenderText {#text} at (0,0) size 0x0 The image being vertically moved up, and sized larger than specified. This seems wrong to me. It's likely casued by the vertical-align: middle declaration, resulting in a shifted positioning. 662313 2 eae 2012-07-03 09:56:46 -0700 I believe the fix for 90354 addressed this. 662854 3 d-r 2012-07-04 04:18:39 -0700 Unfortunately, bug 90354 doesn't solve this. 673322 4 d-r 2012-07-19 02:11:47 -0700 Argh... I was soo close to finding the reason :-) - Good to see it's fixed. *** This bug has been marked as a duplicate of bug 91464 *** 149937 2012-06-28 06:28:49 -0700 2012-06-28 06:28:49 -0700 014-wdiff.html failure 014-wdiff.html text/html 888 d-r PGhlYWQ+PHN0eWxlPi5kZWwgeyBiYWNrZ3JvdW5kOiAjZmFhOyB9IC5hZGQgeyBiYWNrZ3JvdW5k OiAjYWZhOyB9PC9zdHlsZT48L2hlYWQ+PHByZT5sYXllciBhdCAoMCwwKSBzaXplIDgwMHg2MDAK ICBSZW5kZXJWaWV3IGF0ICgwLDApIHNpemUgODAweDYwMApsYXllciBhdCAoMCwwKSBzaXplIDgw MHg2MDAKICBSZW5kZXJCbG9jayB7SFRNTH0gYXQgKDAsMCkgc2l6ZSA4MDB4NjAwCiAgICBSZW5k ZXJCb2R5IHtCT0RZfSBhdCAoOCw4KSBzaXplIDc4NHg1ODQKICAgICAgUmVuZGVySW5saW5lIHtB fSBhdCAoMCwwKSBzaXplIDIxeDE2IFtjb2xvcj0jMDAwMEVFXQogICAgICAgIFJlbmRlckltYWdl IHtJTUd9IGF0IDxzcGFuIGNsYXNzPWRlbD4oMCwwKTwvc3Bhbj4gPHNwYW4gY2xhc3M9YWRkPigw LC0xKTwvc3Bhbj4gc2l6ZSA8c3BhbiBjbGFzcz1kZWw+MjF4MjE8L3NwYW4+IDxzcGFuIGNsYXNz PWFkZD4yMXgyMjwvc3Bhbj4gW2JnY29sb3I9IzAwODAwMF0KICAgICAgUmVuZGVyVGV4dCB7I3Rl eHR9IGF0ICgyMSwxKSBzaXplIDR4MTYKICAgICAgICB0ZXh0IHJ1biBhdCAoMjEsMSkgd2lkdGgg NDogIiAiCiAgICAgIFJlbmRlckltYWdlIHtJTUd9IGF0IDxzcGFuIGNsYXNzPWRlbD4oMjUsMCk8 L3NwYW4+IDxzcGFuIGNsYXNzPWFkZD4oMjUsLTEpPC9zcGFuPiBzaXplIDxzcGFuIGNsYXNzPWRl bD4yMXgyMTwvc3Bhbj4gPHNwYW4gY2xhc3M9YWRkPjIxeDIyPC9zcGFuPiBbYmdjb2xvcj0jMDA4 MDAwXQogICAgICBSZW5kZXJUZXh0IHsjdGV4dH0gYXQgKDQ2LDEpIHNpemUgMjk1eDE2CiAgICAg ICAgdGV4dCBydW4gYXQgKDQ2LDEpIHdpZHRoIDI5NTogIiBUaGUgdHdvIGdyZWVuIGJsb2NrcyBz aG91bGQgbGluZSB1cCB2ZXJ0aWNhbGx5LiIKPC9wcmU+ 150597 2012-07-03 06:34:59 -0700 2012-07-03 06:34:59 -0700 Simplified test case 014_imgonly.html text/html 166 d-r PGh0bWw+CjxoZWFkPgo8c3R5bGU+CmltZwp7CiAgICB2ZXJ0aWNhbC1hbGlnbjogbWlkZGxlOwog ICAgYm9yZGVyLXN0eWxlOiBub25lOwogICAgYmFja2dyb3VuZC1jb2xvcjpncmVlbjsKfQo8L3N0 eWxlPgo8L2hlYWQ+Cjxib2R5PjxpbWcgd2lkdGg9MjEgaGVpZ2h0PTIxPjwvYm9keT4KCg==