Title: [220986] trunk/Source/WebCore
Revision
220986
Author
mmaxfi...@apple.com
Date
2017-08-21 16:04:28 -0700 (Mon, 21 Aug 2017)

Log Message

Addressing post-review comments after r220725
https://bugs.webkit.org/show_bug.cgi?id=175382

Reviewed by Darin Adler.

* css/CSSFontFace.cpp:
(WebCore::CSSFontFace::setUnicodeRange):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (220985 => 220986)


--- trunk/Source/WebCore/ChangeLog	2017-08-21 22:17:24 UTC (rev 220985)
+++ trunk/Source/WebCore/ChangeLog	2017-08-21 23:04:28 UTC (rev 220986)
@@ -1,3 +1,13 @@
+2017-08-21  Myles C. Maxfield  <mmaxfi...@apple.com>
+
+        Addressing post-review comments after r220725
+        https://bugs.webkit.org/show_bug.cgi?id=175382
+
+        Reviewed by Darin Adler.
+
+        * css/CSSFontFace.cpp:
+        (WebCore::CSSFontFace::setUnicodeRange):
+
 2017-08-21  Youenn Fablet  <you...@apple.com>
 
         [Cache API] Add support for CacheStorage.match

Modified: trunk/Source/WebCore/css/CSSFontFace.cpp (220985 => 220986)


--- trunk/Source/WebCore/css/CSSFontFace.cpp	2017-08-21 22:17:24 UTC (rev 220985)
+++ trunk/Source/WebCore/css/CSSFontFace.cpp	2017-08-21 23:04:28 UTC (rev 220986)
@@ -260,17 +260,8 @@
         ranges.append({ range.from(), range.to() });
     }
 
-    if (ranges.size() == m_ranges.size()) {
-        bool same = true;
-        for (size_t i = 0; i < ranges.size(); ++i) {
-            if (ranges[i] != m_ranges[i]) {
-                same = false;
-                break;
-            }
-        }
-        if (same)
-            return true;
-    }
+    if (ranges == m_ranges)
+        return true;
 
     m_ranges = WTFMove(ranges);
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to