Move Font related classes to Source/platform/fonts

Source/platform/fonts/FontCacheKey.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 24 matching lines @@
 #include "wtf/HashTableDeletedValueType.h"
 #include "wtf/text/AtomicStringHash.h"
 #include "wtf/text/StringHash.h"
 
 namespace WebCore {
 
 // Multiplying the floating point size by 100 gives two decimal point
 // precision which should be sufficient.
 static const unsigned s_fontSizePrecisionMultiplier = 100;
 
-struct FontCacheKey {
+struct PLATFORM_EXPORT FontCacheKey {

[abarth-chromium, 2013/10/29 05:26:35]

Don't we need to #include "platform/PlatformExport.h" ?  I don't see any headers
that would pull that in.

Actually, this class is entirely inline.  There's no reason to export it (and
there's no complication unit to export it from anyway).

[rwlbuis, 2013/10/29 18:38:00]

Ok, good point.

     WTF_MAKE_FAST_ALLOCATED;
 public:
     FontCacheKey()
         : m_familyName()
         , m_fontSize(0)
         , m_options(0) { }
     FontCacheKey(AtomicString familyName, float fontSize, unsigned options)
         : m_familyName(familyName)
         , m_fontSize(fontSize * s_fontSizePrecisionMultiplier)
         , m_options(options) { }
@@ skipping 45 matching lines @@
     }
 
     static bool equal(const FontCacheKey& a, const FontCacheKey& b)
     {
         return a == b;
     }
 
     static const bool safeToCompareToEmptyOrDeleted = true;
 };
 
-struct FontCacheKeyTraits : WTF::SimpleClassHashTraits<FontCacheKey> { };
+struct PLATFORM_EXPORT FontCacheKeyTraits : WTF::SimpleClassHashTraits<FontCacheKey> { };

[abarth-chromium, 2013/10/29 05:26:35]

This class looks entirely inline.  I don't think there's a reason to export it.

[rwlbuis, 2013/10/29 18:38:00]

Fixed.

 
 }
 
 #endif // FontCacheKey_h