range.getBoundingClientRect() returns incorrect value for collapsed ranges

Source/core/dom/Range.cpp

Index: Source/core/dom/Range.cpp
diff --git a/Source/core/dom/Range.cpp b/Source/core/dom/Range.cpp
index dffd506bc95c1120b7bb0e26b1ce20e35a1bc6cc..29c2579d011403f3fb3c37a1336aca43ad9bba7d 100644
--- a/Source/core/dom/Range.cpp
+++ b/Source/core/dom/Range.cpp
@@ -1786,8 +1786,12 @@ FloatRect Range::boundingRect() const
     getBorderAndTextQuads(quads);
 
     FloatRect result;
-    for (const FloatQuad& quad : quads)
-        result.unite(quad.boundingBox());
+    for (const FloatQuad& quad : quads) {
+        if (result.isEmpty())
+            result = quad.boundingBox();

[eae, 2015/03/19 20:52:21]

This doesn't seem right, if the last quad is empty the entire rect is reset
based on the last item. Is that really the desired behavior?

[Abhijeet Kandalkar Slow, 2015/03/20 07:48:47]

Done.

+        else
+            result.uniteEvenIfEmpty(quad.boundingBox());
+    }
 
     return result;
 }