Changeset 166381 in webkit


Ignore:
Timestamp:
Mar 27, 2014 4:23:37 PM (10 years ago)
Author:
commit-queue@webkit.org
Message:

Unreviewed, rolling out r166364.
https://bugs.webkit.org/show_bug.cgi?id=130872

Caused a test assertion (Requested by smfr on #webkit).

Reverted changeset:

"Operator stretching: expose a math data API"
https://bugs.webkit.org/show_bug.cgi?id=130572
http://trac.webkit.org/changeset/166364

Location:
trunk/Source/WebCore
Files:
2 deleted
11 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/CMakeLists.txt

    r166378 r166381  
    19491949    platform/graphics/filters/SourceGraphic.cpp
    19501950    platform/graphics/filters/SpotLightSource.cpp
    1951 
    1952     platform/graphics/opentype/OpenTypeMathData.cpp
    19531951
    19541952    platform/graphics/texmap/TextureMapper.cpp
  • trunk/Source/WebCore/ChangeLog

    r166380 r166381  
     12014-03-27  Commit Queue  <commit-queue@webkit.org>
     2
     3        Unreviewed, rolling out r166364.
     4        https://bugs.webkit.org/show_bug.cgi?id=130872
     5
     6        Caused a test assertion (Requested by smfr on #webkit).
     7
     8        Reverted changeset:
     9
     10        "Operator stretching: expose a math data API"
     11        https://bugs.webkit.org/show_bug.cgi?id=130572
     12        http://trac.webkit.org/changeset/166364
     13
    1142014-03-27  Benjamin Poulain  <bpoulain@apple.com>
    215
  • trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj

    r166378 r166381  
    79507950    <ClCompile Include="..\platform\graphics\win\IntSizeWin.cpp" />
    79517951    <ClCompile Include="..\platform\graphics\win\MediaPlayerPrivateFullscreenWindow.cpp" />
    7952     <ClCompile Include="..\platform\graphics\opentype\OpenTypeMathData.cpp" />
    79537952    <ClCompile Include="..\platform\graphics\opentype\OpenTypeUtilities.cpp" />
    79547953    <ClCompile Include="..\platform\graphics\opentype\OpenTypeVerticalData.cpp" />
     
    1937419373    <ClInclude Include="..\platform\graphics\win\LocalWindowsContext.h" />
    1937519374    <CustomBuildStep Include="..\platform\graphics\win\MediaPlayerPrivateFullscreenWindow.h" />
    19376     <ClInclude Include="..\platform\graphics\opentype\OpenTypeMathData.h" />
    1937719375    <ClInclude Include="..\platform\graphics\opentype\OpenTypeTypes.h" />
    1937819376    <ClInclude Include="..\platform\graphics\opentype\OpenTypeUtilities.h" />
  • trunk/Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters

    r166378 r166381  
    14771477      <Filter>platform\graphics\win</Filter>
    14781478    </ClCompile>
    1479     <ClCompile Include="..\platform\graphics\opentype\OpenTypeMathData.cpp">
    1480       <Filter>platform\graphics\win</Filter>
    1481     </ClCompile>
    14821479    <ClCompile Include="..\platform\graphics\opentype\OpenTypeUtilities.cpp">
    14831480      <Filter>platform\graphics\win</Filter>
     
    84338430    </ClInclude>
    84348431    <ClInclude Include="..\platform\graphics\win\LocalWindowsContext.h">
    8435       <Filter>platform\graphics\win</Filter>
    8436     </ClInclude>
    8437     <ClInclude Include="..\platform\graphics\opentype\OpenTypeMathData.h">
    84388432      <Filter>platform\graphics\win</Filter>
    84398433    </ClInclude>
  • trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj

    r166378 r166381  
    45944594                B2C3DA640D006CD600EF6F26 /* SimpleFontData.cpp in Sources */ = {isa = PBXBuildFile; fileRef = B2C3DA530D006CD600EF6F26 /* SimpleFontData.cpp */; };
    45954595                B2C3DA650D006CD600EF6F26 /* SimpleFontData.h in Headers */ = {isa = PBXBuildFile; fileRef = B2C3DA540D006CD600EF6F26 /* SimpleFontData.h */; settings = {ATTRIBUTES = (Private, ); }; };
    4596                 B2D3DA640D006CD600EF6F27 /* OpenTypeMathData.cpp in Sources */ = {isa = PBXBuildFile; fileRef = B2D3DA530D006CD600EF6F27 /* OpenTypeMathData.cpp */; };
    4597                 B2D3DA650D006CD600EF6F27 /* OpenTypeMathData.h in Headers */ = {isa = PBXBuildFile; fileRef = B2D3DA540D006CD600EF6F27 /* OpenTypeMathData.h */; settings = {ATTRIBUTES = (Private, ); }; };
    45984596                B2C3DA660D006CD600EF6F26 /* FontDescription.h in Headers */ = {isa = PBXBuildFile; fileRef = B2C3DA550D006CD600EF6F26 /* FontDescription.h */; settings = {ATTRIBUTES = (Private, ); }; };
    45994597                B2C3DA670D006CD600EF6F26 /* FontGlyphs.cpp in Sources */ = {isa = PBXBuildFile; fileRef = B2C3DA560D006CD600EF6F26 /* FontGlyphs.cpp */; };
     
    1184411842                B2C3DA530D006CD600EF6F26 /* SimpleFontData.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = SimpleFontData.cpp; sourceTree = "<group>"; };
    1184511843                B2C3DA540D006CD600EF6F26 /* SimpleFontData.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = SimpleFontData.h; sourceTree = "<group>"; };
    11846                 B2D3DA530D006CD600EF6F27 /* OpenTypeMathData.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = opentype/OpenTypeMathData.cpp; sourceTree = "<group>"; };
    11847                 B2D3DA540D006CD600EF6F27 /* OpenTypeMathData.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = opentype/OpenTypeMathData.h; sourceTree = "<group>"; };
    1184811844                B2C3DA550D006CD600EF6F26 /* FontDescription.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = FontDescription.h; sourceTree = "<group>"; };
    1184911845                B2C3DA560D006CD600EF6F26 /* FontGlyphs.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = FontGlyphs.cpp; sourceTree = "<group>"; };
     
    2013420130                                B2C3DA530D006CD600EF6F26 /* SimpleFontData.cpp */,
    2013520131                                B2C3DA540D006CD600EF6F26 /* SimpleFontData.h */,
    20136                                 B2D3DA530D006CD600EF6F27 /* OpenTypeMathData.cpp */,
    20137                                 B2D3DA540D006CD600EF6F27 /* OpenTypeMathData.h */,
    2013820132                                CD641EB21818F5ED00EE4C41 /* SourceBufferPrivate.h */,
    2013920133                                CDC61DA0180867D8004B913F /* SourceBufferPrivate.h */,
     
    2547625470                                41D168EE10226E89009BC827 /* SharedWorkerThread.h in Headers */,
    2547725471                                B2C3DA650D006CD600EF6F26 /* SimpleFontData.h in Headers */,
    25478                                 B2D3DA650D006CD600EF6F27 /* OpenTypeMathData.h in Headers */,
    2547925472                                E48944A3180B57D800F165D8 /* SimpleLineLayout.h in Headers */,
    2548025473                                E4E9B11D1814569C003ACCDF /* SimpleLineLayoutFunctions.h in Headers */,
     
    2884028833                                41D168ED10226E89009BC827 /* SharedWorkerThread.cpp in Sources */,
    2884128834                                B2C3DA640D006CD600EF6F26 /* SimpleFontData.cpp in Sources */,
    28842                                 B2D3DA640D006CD600EF6F27 /* OpenTypeMathData.cpp in Sources */,
    2884328835                                163E88F7118A39D200ED9231 /* SimpleFontDataCoreText.cpp in Sources */,
    2884428836                                37E65950163B10C200EB4574 /* SimpleFontDataIOS.mm in Sources */,
  • trunk/Source/WebCore/platform/graphics/FontCache.cpp

    r166364 r166381  
    3535#include "FontPlatformData.h"
    3636#include "FontSelector.h"
    37 #include "OpenTypeMathData.h"
    3837#include "WebKitFontFamilyNames.h"
    3938#include <wtf/HashMap.h>
     
    268267}
    269268
    270 struct FontFileCacheKeyHash {
     269#if ENABLE(OPENTYPE_VERTICAL)
     270struct FontVerticalDataCacheKeyHash {
    271271    static unsigned hash(const FontCache::FontFileKey& fontFileKey)
    272272    {
     
    282282};
    283283
    284 struct FontFileCacheKeyTraits : WTF::GenericHashTraits<FontCache::FontFileKey> {
     284struct FontVerticalDataCacheKeyTraits : WTF::GenericHashTraits<FontCache::FontFileKey> {
    285285    static const bool emptyValueIsZero = true;
    286286    static const bool needsDestruction = true;
     
    300300};
    301301
    302 typedef HashMap<FontCache::FontFileKey, RefPtr<OpenTypeMathData>, FontFileCacheKeyHash, FontFileCacheKeyTraits> FontMathDataCache;
    303 
    304 static FontMathDataCache& fontMathDataCacheInstance()
    305 {
    306     static NeverDestroyed<FontMathDataCache> fontMathDataCache;
    307     return fontMathDataCache;
    308 }
    309 
    310 PassRefPtr<OpenTypeMathData> FontCache::getMathData(const FontFileKey& key, const FontPlatformData& fontData)
    311 {
    312     FontMathDataCache& fontMathDataCache = fontMathDataCacheInstance();
    313     FontMathDataCache::iterator result = fontMathDataCache.find(key);
    314     if (result != fontMathDataCache.end())
    315         return result.get()->value;
    316 
    317     RefPtr<OpenTypeMathData> mathData = OpenTypeMathData::create(fontData);
    318     if (!mathData->hasMathData())
    319         mathData.clear();
    320     fontMathDataCache.set(key, mathData);
    321     return mathData;
    322 }
    323 
    324 #if ENABLE(OPENTYPE_VERTICAL)
    325 typedef HashMap<FontCache::FontFileKey, RefPtr<OpenTypeVerticalData>, FontFileCacheKeyHash, FontFileCacheKeyTraits> FontVerticalDataCache;
    326 
    327 static FontVerticalDataCache& fontVerticalDataCacheInstance()
     302typedef HashMap<FontCache::FontFileKey, RefPtr<OpenTypeVerticalData>, FontVerticalDataCacheKeyHash, FontVerticalDataCacheKeyTraits> FontVerticalDataCache;
     303
     304FontVerticalDataCache& fontVerticalDataCacheInstance()
    328305{
    329306    static NeverDestroyed<FontVerticalDataCache> fontVerticalDataCache;
  • trunk/Source/WebCore/platform/graphics/FontCache.h

    r166364 r166381  
    5555class FontData;
    5656class FontSelector;
    57 class OpenTypeMathData;
    5857class OpenTypeVerticalData;
    5958class SimpleFontData;
     
    153152#endif
    154153
     154#if ENABLE(OPENTYPE_VERTICAL)
    155155    typedef AtomicString FontFileKey;
    156     PassRefPtr<OpenTypeMathData> getMathData(const FontFileKey&, const FontPlatformData&);
    157 #if ENABLE(OPENTYPE_VERTICAL)
    158156    PassRefPtr<OpenTypeVerticalData> getVerticalData(const FontFileKey&, const FontPlatformData&);
    159157#endif
  • trunk/Source/WebCore/platform/graphics/FontPlatformData.cpp

    r166364 r166381  
    2727#include <wtf/text/StringHash.h>
    2828#include <wtf/text/WTFString.h>
    29 
    30 #if OS(DARWIN) && USE(CG)
    31 #include "SharedBuffer.h"
    32 #include <CoreGraphics/CGFont.h>
    33 #endif
    3429
    3530namespace WebCore {
     
    175170}
    176171
    177 #if OS(DARWIN) && USE(CG)
    178 PassRefPtr<SharedBuffer> FontPlatformData::openTypeTable(uint32_t table) const
    179 {
    180     if (CFDataRef data = CGFontCopyTableForTag(cgFont(), table))
    181         return SharedBuffer::wrapCFData(data);
    182    
    183     return nullptr;
    184172}
    185 #endif
    186 
    187 }
  • trunk/Source/WebCore/platform/graphics/FontPlatformData.h

    r166364 r166381  
    217217    }
    218218
    219 #if (OS(DARWIN) && USE(CG)) || (PLATFORM(WIN) && (USE(CG) || USE(CAIRO)))
     219#if PLATFORM(WIN) && (USE(CG) || USE(CAIRO))
    220220    PassRefPtr<SharedBuffer> openTypeTable(uint32_t table) const;
    221221#endif
  • trunk/Source/WebCore/platform/graphics/SimpleFontData.cpp

    r166364 r166381  
    3333#include "Font.h"
    3434#include "FontCache.h"
    35 #include "OpenTypeMathData.h"
    3635#include <wtf/MathExtras.h>
    3736
     
    5453    , m_isTextOrientationFallback(isTextOrientationFallback)
    5554    , m_isBrokenIdeographFallback(false)
    56     , m_mathData(nullptr)
    5755#if ENABLE(OPENTYPE_VERTICAL)
    5856    , m_verticalData(0)
     
    6361    platformGlyphInit();
    6462    platformCharWidthInit();
    65 
    66     bool tryMathData = false;
    67 #if PLATFORM(WIN) && (USE(CG) || USE(CAIRO))
    68     tryMathData = platformData.hfont();
    69 #elif OS(DARWIN) && USE(CG)
    70     tryMathData = platformData.font();
    71 #elif USE(FREETYPE)
    72     tryMathData = platformData.hash();
    73 #endif
    74     if (tryMathData)
    75         m_mathData = fontCache()->getMathData(String::number(platformData.hash()), platformData);
    76 
    7763#if ENABLE(OPENTYPE_VERTICAL)
    7864    if (platformData.orientation() == Vertical && !isTextOrientationFallback) {
     
    9177    , m_isTextOrientationFallback(false)
    9278    , m_isBrokenIdeographFallback(false)
    93     , m_mathData(nullptr)
    9479#if ENABLE(OPENTYPE_VERTICAL)
    9580    , m_verticalData(0)
  • trunk/Source/WebCore/platform/graphics/SimpleFontData.h

    r166364 r166381  
    3333#include "GlyphMetricsMap.h"
    3434#include "GlyphPageTreeNode.h"
    35 #include "OpenTypeMathData.h"
    3635#if ENABLE(OPENTYPE_VERTICAL)
    3736#include "OpenTypeVerticalData.h"
     
    9493
    9594    const FontPlatformData& platformData() const { return m_platformData; }
    96     const OpenTypeMathData* mathData() const { return m_mathData ? m_mathData.get() : nullptr; }
    9795#if ENABLE(OPENTYPE_VERTICAL)
    9896    const OpenTypeVerticalData* verticalData() const { return m_verticalData.get(); }
     
    262260    bool m_isTextOrientationFallback;
    263261    bool m_isBrokenIdeographFallback;
    264     RefPtr<OpenTypeMathData> m_mathData;
    265262#if ENABLE(OPENTYPE_VERTICAL)
    266263    RefPtr<OpenTypeVerticalData> m_verticalData;
Note: See TracChangeset for help on using the changeset viewer.