Changeset 220986 in webkit

Timestamp:

Aug 21, 2017 4:04:28 PM (7 years ago)

Author:

mmaxfield@apple.com

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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• css/CSSFontFace.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-08-21  Myles C. Maxfield  <mmaxfield@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  <youenn@apple.com>
 

trunk/Source/WebCore/css/CSSFontFace.cpp

@@ -261 +261 @@
     }
 
-    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);