139345 2014-12-06 11:56:26 -0800 RenderObject::absoluteBoundingBoxRect() returns an incorrect height for inline images 2014-12-08 17:14:29 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED INVALID P2 Normal --- 1 mmaxfield mmaxfield simon.fraser oldest_to_newest 1053049 0 mmaxfield 2014-12-06 11:56:26 -0800 RenderInline::generateCulledLineBoxRects() disregards the height of inline boxes (such as images). Instead, it always uses the font metrics's line height for the line. 1053051 1 mmaxfield 2014-12-06 12:04:52 -0800 int logicalTop = rootBox.logicalTop() + (rootBox.lineStyle().font().fontMetrics().ascent() - containerStyle.font().fontMetrics().ascent()); int logicalHeight = containerStyle.font().fontMetrics().height(); 1053589 2 242863 mmaxfield 2014-12-08 17:02:45 -0800 Created attachment 242863 Test 1053594 3 mmaxfield 2014-12-08 17:14:29 -0800 Turns out this is correct behavior. 242863 2014-12-08 17:02:45 -0800 2014-12-08 17:02:45 -0800 Test bug-139345-20141208170324.patch text/plain 2342 mmaxfield U3VidmVyc2lvbiBSZXZpc2lvbjogMTc2OTkyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggZWI2MzRmNzhiY2M1YzYxMDkxMWQz ZmJhYjQ3NGJkNDMwOGYzMzYwYy4uYjI4ZWQzODcwMWE4ZTI4OGQxNjZjODMzNzU0ZjdjOTM0Nzlj N2E3YiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDE0LTEyLTA4ICBNeWxlcyBDLiBNYXhmaWVs ZCAgPG1tYXhmaWVsZEBhcHBsZS5jb20+CisKKyAgICAgICAgUmVuZGVyT2JqZWN0OjphYnNvbHV0 ZUJvdW5kaW5nQm94UmVjdCgpIHJldHVybnMgYW4gaW5jb3JyZWN0IGhlaWdodCBmb3IgaW5saW5l IGltYWdlcworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9 MTM5MzQ1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg KiBmYXN0L2RvbS9FbGVtZW50L3JlcGxhY2VkLWVsZW1lbnQtYm91bmRpbmctYm94Lmh0bWw6IEFk ZGVkLgorICAgICAgICAqIGZhc3QvZG9tL0VsZW1lbnQvcmVzb3VyY2VzL2dyZWVuLnBuZzogQWRk ZWQuCisKIDIwMTQtMTItMDggIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEBhcHBsZS5jb20+CiAK ICAgICAgICAgUkVHUkVTU0lPTiAoMTczMzk0KTogU3VwcG9ydCBmb3Igd2ViY2FtIGlzIGJyb2tl bgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFzdC9kb20vRWxlbWVudC9yZXBsYWNlZC1lbGVt ZW50LWJvdW5kaW5nLWJveC5odG1sIGIvTGF5b3V0VGVzdHMvZmFzdC9kb20vRWxlbWVudC9yZXBs YWNlZC1lbGVtZW50LWJvdW5kaW5nLWJveC5odG1sCm5ldyBmaWxlIG1vZGUgMTAwNjQ0CmluZGV4 IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLmJhMTkzNzQ3YWFlMDBl ZmRhMWJhOTc0NTY1YmNkODgyMTg0Zjc2YzMKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlvdXRUZXN0 cy9mYXN0L2RvbS9FbGVtZW50L3JlcGxhY2VkLWVsZW1lbnQtYm91bmRpbmctYm94Lmh0bWwKQEAg LTAsMCArMSwxOSBAQAorPCFET0NUWVBFIGh0bWw+Cis8aHRtbD4KKyAgPGhlYWQ+CisgICAgPHNj cmlwdCBzcmM9Ii4uLy4uLy4uL3Jlc291cmNlcy9qcy10ZXN0LXByZS5qcyI+PC9zY3JpcHQ+Cisg IDwvaGVhZD4KKyAgPGJvZHk+CisgICAgICA8YSBocmVmPSIjIiBpZD0iZWxlbWVudCI+CisgICAg ICAgIDxpbWcgc3JjPSJyZXNvdXJjZXMvZ3JlZW4ucG5nIiBoZWlnaHQ9IjEwMCIgd2lkdGg9Ijc2 IiAgYm9yZGVyPSIwIi8+CisgICAgICA8L2E+CisgICAgPHNjcmlwdD4KKyAgICAgICAgZGVzY3Jp cHRpb24oIlRoaXMgdGVzdCBtYWtlcyBzdXJlIHRoYXQgcmVwbGFjZWQgZWxlbWVudHMnIGJvdW5k aW5nIGJveGVzIGhhdmUgY29ycmVjdCBoZWlnaHQuIik7CisgICAgICAgIHZhciBjbGllbnRSZWN0 TGlzdCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJlbGVtZW50IikuZ2V0Q2xpZW50UmVjdHMo KTsKKyAgICAgICAgc2hvdWxkQmUoImNsaWVudFJlY3RMaXN0Lmxlbmd0aCIsICIxIik7CisgICAg ICAgIHNob3VsZEJlKCJjbGllbnRSZWN0TGlzdFswXS5oZWlnaHQiLCAiMTAwIik7CisgICAgPC9z Y3JpcHQ+CisgICAgPHNjcmlwdCBzcmM9Ii4uLy4uLy4uL3Jlc291cmNlcy9qcy10ZXN0LXBvc3Qu anMiPjwvc2NyaXB0PgorICA8L2JvZHk+Cis8L2h0bWw+CisKZGlmZiAtLWdpdCBhL0xheW91dFRl c3RzL2Zhc3QvZG9tL0VsZW1lbnQvcmVzb3VyY2VzL2dyZWVuLnBuZyBiL0xheW91dFRlc3RzL2Zh c3QvZG9tL0VsZW1lbnQvcmVzb3VyY2VzL2dyZWVuLnBuZwpuZXcgZmlsZSBtb2RlIDEwMDY0NApp bmRleCAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwLi4zNWY1ZGVjM2Uz ZWI3MWEyN2Y3ZmI1ZjBkMmE4Mzk5MWZlNDkxOGY5CkdJVCBiaW5hcnkgcGF0Y2gKbGl0ZXJhbCAy NDMKemNtZUFTQE4/KG9sSHlgdVZCcSFpYTB2cF5NaipfPTFTQldNJTBCfkFqS3g5alA3TGVMJC1E JHxTa2ZKUjlUXnhsCnpfSCtNOVdDaWpTbDBBWmE4NXBZNjcjSkVfNyNNeTVnJkpOa0ZxOWZGRnVZ MSZWNmQ5T3ojdntRWElHI05QQD5TJgp6I1c1dHB7cTI+NW9DaDN1bT1BaEMqUj1LSVJPMHdEPE1C UEdIUGBvQHQ1diVubyY3PC0+PSZDd2h1N0t5b3dnWWEKeiZmSUlLRnxXTmRLUmluSilPTU1TciRL Z3dWIUImJGt7JlRjV28hbFZjWVcmSW9VdCVmU2VYNVMpOVItVitTak4wCmsmdkU2fW9qdD4xZ1hl MztaSUA1KFRnSyh4NTQ0PVkpNzgmcW9sYDsrME41Ti1ZNSlLTAoKbGl0ZXJhbCAwCkhjbVY/ZDAw MDAxCgo=