49358 2010-11-10 19:12:02 -0800 Empty spans after a line break are incorrectly positioned at the top left hand corner of their containing block 2010-12-17 14:00:49 -0800 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) PC OS X 10.5 NEW InRadar P2 Normal --- 1 justin.garcia webkit-unassigned ddkilzer hyatt mitz sam simon.fraser oldest_to_newest 307687 0 73569 justin.garcia 2010-11-10 19:12:02 -0800 Created attachment 73569 testcase Load the attached test case. Notice that there is no rect for the empty line in the middle (or the second rect is wrong). 307688 1 justin.garcia 2010-11-10 19:12:59 -0800 <rdar://problem/8639942> 323379 2 76807 justin.garcia 2010-12-16 13:24:35 -0800 Created attachment 76807 testcase Here's a better example. Basically getClientRects() will return a rect positioned at the top left hand corner for each empty span that occurs in the range. 323390 3 justin.garcia 2010-12-16 13:44:10 -0800 getClientRects is just one way to see the problem, which is that the spans are incorrectly positioned. Will attached a better reduction. 323392 4 76810 justin.garcia 2010-12-16 13:45:51 -0800 Created attachment 76810 testcase In this reduction, an empty span just after a <br> is positioned at the top left hand corner of its containing block, instead of after the br. 323405 5 justin.garcia 2010-12-16 13:59:53 -0800 (In reply to comment #4) > Created an attachment (id=76810) [details] > testcase This seems to work correctly in FireFox, by the way. 323975 6 justin.garcia 2010-12-17 13:56:47 -0800 The new radar is <rdar://problem/8784036>. 73569 2010-11-10 19:12:02 -0800 2010-12-16 13:24:35 -0800 testcase getClientRects.html text/html 569 justin.garcia PGJvZHk+PHNwYW4+TGluZSBPbmU8L3NwYW4+PGJyPjxzcGFuPjwvc3Bhbj48YnI+PHNwYW4+TGlu ZSBUaHJlZTwvc3Bhbj48YnI+PHByZSBpZD0iY29uc29sZSI+PC9wcmU+PHNjcmlwdD4KY2hpbGRy ZW4gPSBkb2N1bWVudC5ib2R5LmNoaWxkTm9kZXM7CmRvY3VtZW50LmV4ZWNDb21tYW5kKCJTZWxl Y3RBbGwiKTsKcmVjdHMgPSB3aW5kb3cuZ2V0U2VsZWN0aW9uKCkuZ2V0UmFuZ2VBdCgwKS5nZXRD bGllbnRSZWN0cygpOwoKY29uc29sZSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJjb25zb2xl Iik7CmNvbnNvbGUuaW5uZXJUZXh0ID0gIlRoZSBzZWxlY3RlZCBSYW5nZSdzIGdldENsaWVudFJl Y3RzKCk6XG4iOwpmb3IgKHZhciBpID0gMDsgaSA8IHJlY3RzLmxlbmd0aDsgaSsrKQogICAgY29u c29sZS5pbm5lclRleHQgPSBjb25zb2xlLmlubmVyVGV4dCArICJyZWN0WyIgKyBpICsgIl0gKCIg KyByZWN0c1tpXS50b3AgKyAiLCAiICsgcmVjdHNbaV0ubGVmdCArICIsICIgKyByZWN0c1tpXS53 aWR0aCArICIsICIgKyByZWN0c1tpXS5oZWlnaHQgKyAiKVxuIjsKCjwvc2NyaXB0PjwvYm9keT4= 76807 2010-12-16 13:24:35 -0800 2010-12-16 13:45:51 -0800 testcase span-range.html text/html 608 justin.garcia PGJvZHk+PHNwYW4+TGluZSBPbmU8L3NwYW4+PGJyPjxzcGFuPjwvc3Bhbj48YnI+PHNwYW4+TGlu ZSBUaHJlZTwvc3Bhbj48YnI+PHNwYW4+PC9zcGFuPjxicj48c3Bhbj5MaW5lIEZpdmU8L3NwYW4+ PHByZSBpZD0iY29uc29sZSI+PC9wcmU+PHNjcmlwdD4KY2hpbGRyZW4gPSBkb2N1bWVudC5ib2R5 LmNoaWxkTm9kZXM7CmRvY3VtZW50LmV4ZWNDb21tYW5kKCJTZWxlY3RBbGwiKTsKcmVjdHMgPSB3 aW5kb3cuZ2V0U2VsZWN0aW9uKCkuZ2V0UmFuZ2VBdCgwKS5nZXRDbGllbnRSZWN0cygpOwoKY29u c29sZSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJjb25zb2xlIik7CmNvbnNvbGUuaW5uZXJU ZXh0ID0gIlRoZSBzZWxlY3RlZCBSYW5nZSdzIGdldENsaWVudFJlY3RzKCk6XG4iOwpmb3IgKHZh ciBpID0gMDsgaSA8IHJlY3RzLmxlbmd0aDsgaSsrKQogICAgY29uc29sZS5pbm5lclRleHQgPSBj b25zb2xlLmlubmVyVGV4dCArICJyZWN0WyIgKyBpICsgIl0gKCIgKyByZWN0c1tpXS50b3AgKyAi LCAiICsgcmVjdHNbaV0ubGVmdCArICIsICIgKyByZWN0c1tpXS53aWR0aCArICIsICIgKyByZWN0 c1tpXS5oZWlnaHQgKyAiKVxuIjsKCjwvc2NyaXB0PjwvYm9keT4= 76810 2010-12-16 13:45:51 -0800 2010-12-16 13:45:51 -0800 testcase empty-span-position.html text/html 391 justin.garcia PGJvZHk+PHNwYW4+TGluZSBPbmU8L3NwYW4+PGJyPjxzcGFuPjwvc3Bhbj48YnI+PHNwYW4+TGlu ZSBUaHJlZTwvc3Bhbj48YnI+PHNwYW4gaWQ9InNwYW4iPjwvc3Bhbj48YnI+PHNwYW4+TGluZSBG aXZlPC9zcGFuPjxwcmUgaWQ9ImNvbnNvbGUiPjwvcHJlPjxzY3JpcHQ+CnNwYW4gPSBkb2N1bWVu dC5nZXRFbGVtZW50QnlJZCgic3BhbiIpOwpjb25zb2xlID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5 SWQoImNvbnNvbGUiKTsKY29uc29sZS5pbm5lckhUTUwgPSAiVGhlIGVtcHR5IHN0eWxlIHNwYW4g YWZ0ZXIgdGhlIGZvdXJ0aCBsaW5lIGlzIHBvc2l0aW9uZWQgYXQgKCIgKyBzcGFuLm9mZnNldFRv cCArICIsICIgKyBzcGFuLm9mZnNldFRvcCArICIpIjsKPC9zY3JpcHQ+PC9ib2R5Pg==