Addressing post-review comments after r196747.
authormmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Feb 2016 00:03:58 +0000 (00:03 +0000)
committermmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 19 Feb 2016 00:03:58 +0000 (00:03 +0000)
Unreviewed.

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

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@196784 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/css/CSSFontFaceSet.h
Source/WebCore/css/FontFaceSet.cpp
Source/WebCore/css/FontFaceSet.h

index 6ba898f..320e32f 100644 (file)
@@ -1,3 +1,15 @@
+2016-02-18  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        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  <zalan@apple.com>
 
         Soft hyphen is not shown when it is placed at the end of an inline element
index c17f00b..66947d7 100644 (file)
@@ -47,7 +47,7 @@ public:
     ~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]; }
index 1b71441..7ed7746 100644 (file)
@@ -87,7 +87,7 @@ bool FontFaceSet::has(FontFace* face) const
 
 size_t FontFaceSet::size() const
 {
-    return m_backing.size();
+    return m_backing.faceCount();
 }
 
 FontFaceSet& FontFaceSet::add(FontFace* face)
@@ -110,7 +110,7 @@ bool FontFaceSet::remove(FontFace* face)
 
 void FontFaceSet::clear()
 {
-    while (m_backing.size())
+    while (m_backing.faceCount())
         m_backing.remove(m_backing[0]);
 }
 
index 002b308..c05ed18 100644 (file)
@@ -63,7 +63,7 @@ public:
     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;