Title: [169087] trunk/Source/WebKit2
Revision
169087
Author
[email protected]
Date
2014-05-19 17:24:55 -0700 (Mon, 19 May 2014)

Log Message

TelephoneNumberOverlayController should use a modern loop for text quad bounding 
boxes
https://bugs.webkit.org/show_bug.cgi?id=133096

Reviewed by Anders Carlsson.

* WebProcess/WebPage/mac/TelephoneNumberOverlayControllerMac.mm:
(WebKit::textQuadsToBoundingRectForRange):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (169086 => 169087)


--- trunk/Source/WebKit2/ChangeLog	2014-05-20 00:18:38 UTC (rev 169086)
+++ trunk/Source/WebKit2/ChangeLog	2014-05-20 00:24:55 UTC (rev 169087)
@@ -1,3 +1,14 @@
+2014-05-19  Beth Dakin  <[email protected]>
+
+        TelephoneNumberOverlayController should use a modern loop for text quad bounding 
+        boxes
+        https://bugs.webkit.org/show_bug.cgi?id=133096
+
+        Reviewed by Anders Carlsson.
+
+        * WebProcess/WebPage/mac/TelephoneNumberOverlayControllerMac.mm:
+        (WebKit::textQuadsToBoundingRectForRange):
+
 2014-05-19  Jeremy Jones  <[email protected]>
 
         PlatformCALayerRemoteCustom properties aren't initialized to match the PlatformLayer

Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/TelephoneNumberOverlayControllerMac.mm (169086 => 169087)


--- trunk/Source/WebKit2/WebProcess/WebPage/mac/TelephoneNumberOverlayControllerMac.mm	2014-05-20 00:18:38 UTC (rev 169086)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/TelephoneNumberOverlayControllerMac.mm	2014-05-20 00:24:55 UTC (rev 169087)
@@ -61,9 +61,8 @@
     Vector<FloatQuad> textQuads;
     range.textQuads(textQuads);
     FloatRect boundingRect;
-    size_t size = textQuads.size();
-    for (size_t i = 0; i < size; ++i)
-        boundingRect.unite(textQuads[i].boundingBox());
+    for (auto& quad : textQuads)
+        boundingRect.unite(quad.boundingBox());
     return enclosingIntRect(boundingRect);
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to