Title: [196784] trunk/Source/WebCore
Revision
196784
Author
[email protected]
Date
2016-02-18 16:03:58 -0800 (Thu, 18 Feb 2016)

Log Message

Addressing post-review comments after r196747.

Unreviewed.

* css/CSSFontFaceSet.h:
* css/FontFaceSet.cpp:
(WebCore::FontFaceSet::size):
(WebCore::FontFaceSet::clear):
* css/FontFaceSet.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (196783 => 196784)


--- trunk/Source/WebCore/ChangeLog	2016-02-19 00:01:32 UTC (rev 196783)
+++ trunk/Source/WebCore/ChangeLog	2016-02-19 00:03:58 UTC (rev 196784)
@@ -1,3 +1,15 @@
+2016-02-18  Myles C. Maxfield  <[email protected]>
+
+        Addressing post-review comments after r196747.
+
+        Unreviewed.
+
+        * css/CSSFontFaceSet.h:
+        * css/FontFaceSet.cpp:
+        (WebCore::FontFaceSet::size):
+        (WebCore::FontFaceSet::clear):
+        * css/FontFaceSet.h:
+
 2016-02-18  Zalan Bujtas  <[email protected]>
 
         Soft hyphen is not shown when it is placed at the end of an inline element

Modified: trunk/Source/WebCore/css/CSSFontFaceSet.h (196783 => 196784)


--- trunk/Source/WebCore/css/CSSFontFaceSet.h	2016-02-19 00:01:32 UTC (rev 196783)
+++ trunk/Source/WebCore/css/CSSFontFaceSet.h	2016-02-19 00:03:58 UTC (rev 196784)
@@ -47,7 +47,7 @@
     ~CSSFontFaceSet();
 
     bool hasFace(const CSSFontFace&) const;
-    size_t size() const { return m_faces.size(); }
+    size_t faceCount() const { return m_faces.size(); }
     void add(CSSFontFace&);
     void remove(const CSSFontFace&);
     const CSSFontFace& operator[](size_t i) const { return m_faces[i]; }

Modified: trunk/Source/WebCore/css/FontFaceSet.cpp (196783 => 196784)


--- trunk/Source/WebCore/css/FontFaceSet.cpp	2016-02-19 00:01:32 UTC (rev 196783)
+++ trunk/Source/WebCore/css/FontFaceSet.cpp	2016-02-19 00:03:58 UTC (rev 196784)
@@ -87,7 +87,7 @@
 
 size_t FontFaceSet::size() const
 {
-    return m_backing.size();
+    return m_backing.faceCount();
 }
 
 FontFaceSet& FontFaceSet::add(FontFace* face)
@@ -110,7 +110,7 @@
 
 void FontFaceSet::clear()
 {
-    while (m_backing.size())
+    while (m_backing.faceCount())
         m_backing.remove(m_backing[0]);
 }
 

Modified: trunk/Source/WebCore/css/FontFaceSet.h (196783 => 196784)


--- trunk/Source/WebCore/css/FontFaceSet.h	2016-02-19 00:01:32 UTC (rev 196783)
+++ trunk/Source/WebCore/css/FontFaceSet.h	2016-02-19 00:03:58 UTC (rev 196784)
@@ -63,7 +63,7 @@
     void load(const String& font, const String& text, DeferredWrapper&& promise, ExceptionCode&);
     bool check(const String& font, ExceptionCode& ec) { return check(font, String(" ", String::ConstructFromLiteral), ec); }
     bool check(const String& font, const String& text, ExceptionCode&);
-    
+
     String status() const;
 
     typedef DOMPromise<FontFaceSet&, DOMCoreException&> Promise;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to