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 6ba898f25d45ac3cc1d1fc93206c7f2aaf400b0a..320e32f936a0faa7af07f2fd58df155c6b5b4807 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 c17f00bff2869e63a97d1c85153a3c87ea406dbb..66947d781b8c81d8421fe092a1b6acbea4234b9f 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 1b714414ef4ce935af6e0a78ef04470546990cfa..7ed77466b424293dceb7f3b5524be184e1c5a82c 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 002b3086ac548a4df1e7f366b7c15644fc19f159..c05ed18e4ce8af51f8f2e8041251ab54001dc2c7 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;