2009-07-27 Albert J. Wong <ajwong@chromium.org>
authorjianli@chromium.org <jianli@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Jul 2009 21:17:46 +0000 (21:17 +0000)
committerjianli@chromium.org <jianli@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Jul 2009 21:17:46 +0000 (21:17 +0000)
        Reviewed by David Levin.

        Add in trivial implementation of FontPlatformData::description() for
        linux to fix build bustage in chromium.

        Fix chromium linux build by adding missing function implementation.
        https://bugs.webkit.org/show_bug.cgi?id=27732

        Tested with a build of chromium on linux.

        * platform/graphics/chromium/FontPlatformDataLinux.cpp:
        (WebCore::FontPlatformData::description):
        * platform/graphics/chromium/FontPlatformDataLinux.h:

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

WebCore/ChangeLog
WebCore/platform/graphics/chromium/FontPlatformDataLinux.cpp
WebCore/platform/graphics/chromium/FontPlatformDataLinux.h

index b057035..3fb0472 100644 (file)
@@ -1,3 +1,19 @@
+2009-07-27  Albert J. Wong  <ajwong@chromium.org>
+
+        Reviewed by David Levin.
+
+        Add in trivial implementation of FontPlatformData::description() for
+        linux to fix build bustage in chromium.
+
+        Fix chromium linux build by adding missing function implementation.
+        https://bugs.webkit.org/show_bug.cgi?id=27732
+
+        Tested with a build of chromium on linux.
+
+        * platform/graphics/chromium/FontPlatformDataLinux.cpp:
+        (WebCore::FontPlatformData::description):
+        * platform/graphics/chromium/FontPlatformDataLinux.h:
+
 2009-07-27  Brent Fulgham  <bfulgham@webkit.org>
 
         Build correct, no review.
index ce072ac..bf4697f 100644 (file)
@@ -32,8 +32,9 @@
 #include "FontPlatformData.h"
 
 #include "HarfbuzzSkia.h"
-#include "StringImpl.h"
 #include "NotImplemented.h"
+#include "PlatformString.h"
+#include "StringImpl.h"
 
 #include "SkPaint.h"
 #include "SkTypeface.h"
@@ -109,6 +110,13 @@ FontPlatformData& FontPlatformData::operator=(const FontPlatformData& src)
     return *this;
 }
 
+#ifndef NDEBUG
+String FontPlatformData::description() const
+{
+    return String();
+}
+#endif
+
 void FontPlatformData::setupPaint(SkPaint* paint) const
 {
     const float ts = m_textSize > 0 ? m_textSize : 12;
index c9c0917..29ce8e7 100644 (file)
@@ -43,6 +43,7 @@ struct HB_FaceRec_;
 namespace WebCore {
 
 class FontDescription;
+class String;
 
 // -----------------------------------------------------------------------------
 // FontPlatformData is the handle which WebKit has on a specific face. A face
@@ -106,6 +107,10 @@ public:
     FontPlatformData& operator=(const FontPlatformData&);
     bool isHashTableDeletedValue() const { return m_typeface == hashTableDeletedFontValue(); }
 
+#ifndef NDEBUG
+    String description() const;
+#endif
+
     HB_FaceRec_* harfbuzzFace() const;
 
     // -------------------------------------------------------------------------