Changeset 53152 in webkit


Ignore:
Timestamp:
Jan 12, 2010 11:52:37 AM (14 years ago)
Author:
pkasting@chromium.org
Message:

Remove XBM support from the open-source image decoders.
https://bugs.webkit.org/show_bug.cgi?id=27823

Reviewed by Dimitri Glazkov.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCoreSources.bkl:
  • platform/image-decoders/ImageDecoder.cpp:

(WebCore::ImageDecoder::create):

  • platform/image-decoders/xbm: Removed.
  • platform/image-decoders/xbm/XBMImageDecoder.cpp: Removed.
  • platform/image-decoders/xbm/XBMImageDecoder.h: Removed.
Location:
trunk/WebCore
Files:
1 deleted
6 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebCore/ChangeLog

    r53151 r53152  
     12010-01-12  Peter Kasting  <pkasting@google.com>
     2
     3        Reviewed by Dimitri Glazkov.
     4
     5        Remove XBM support from the open-source image decoders.
     6        https://bugs.webkit.org/show_bug.cgi?id=27823
     7
     8        * GNUmakefile.am:
     9        * WebCore.gypi:
     10        * WebCore.vcproj/WebCore.vcproj:
     11        * WebCoreSources.bkl:
     12        * platform/image-decoders/ImageDecoder.cpp:
     13        (WebCore::ImageDecoder::create):
     14        * platform/image-decoders/xbm: Removed.
     15        * platform/image-decoders/xbm/XBMImageDecoder.cpp: Removed.
     16        * platform/image-decoders/xbm/XBMImageDecoder.h: Removed.
     17
    1182010-01-12  Alexey Proskuryakov  <ap@apple.com>
    219
  • trunk/WebCore/GNUmakefile.am

    r53086 r53152  
    4242        -I$(srcdir)/WebCore/platform/image-decoders/jpeg \
    4343        -I$(srcdir)/WebCore/platform/image-decoders/png \
    44         -I$(srcdir)/WebCore/platform/image-decoders/xbm \
    4544        -I$(srcdir)/WebCore/platform/mock \
    4645        -I$(srcdir)/WebCore/platform/network \
     
    20352034        WebCore/platform/image-decoders/png/PNGImageDecoder.cpp \
    20362035        WebCore/platform/image-decoders/png/PNGImageDecoder.h \
    2037         WebCore/platform/image-decoders/xbm/XBMImageDecoder.cpp \
    2038         WebCore/platform/image-decoders/xbm/XBMImageDecoder.h \
    20392036        WebCore/platform/network/soup/AuthenticationChallenge.h \
    20402037        WebCore/platform/network/soup/CookieJarSoup.cpp \
  • trunk/WebCore/WebCore.gypi

    r53086 r53152  
    23362336            'platform/image-decoders/skia/ImageDecoderSkia.cpp',
    23372337            'platform/image-decoders/wx/ImageDecoderWx.cpp',
    2338             'platform/image-decoders/xbm/XBMImageDecoder.cpp',
    2339             'platform/image-decoders/xbm/XBMImageDecoder.h',
    23402338            'platform/image-decoders/zlib/crc32.h',
    23412339            'platform/image-decoders/zlib/deflate.h',
  • trunk/WebCore/WebCore.vcproj/WebCore.vcproj

    r53086 r53152  
    2615626156                                        </File>
    2615726157                                </Filter>
    26158                                 <Filter
    26159                                         Name="xbm"
    26160                                         >
    26161                                         <File
    26162                                                 RelativePath="..\platform\image-decoders\xbm\XBMImageDecoder.cpp"
    26163                                                 >
    26164                                                 <FileConfiguration
    26165                                                         Name="Debug|Win32"
    26166                                                         ExcludedFromBuild="true"
    26167                                                         >
    26168                                                         <Tool
    26169                                                                 Name="VCCLCompilerTool"
    26170                                                         />
    26171                                                 </FileConfiguration>
    26172                                                 <FileConfiguration
    26173                                                         Name="Release|Win32"
    26174                                                         ExcludedFromBuild="true"
    26175                                                         >
    26176                                                         <Tool
    26177                                                                 Name="VCCLCompilerTool"
    26178                                                         />
    26179                                                 </FileConfiguration>
    26180                                                 <FileConfiguration
    26181                                                         Name="Debug_Internal|Win32"
    26182                                                         ExcludedFromBuild="true"
    26183                                                         >
    26184                                                         <Tool
    26185                                                                 Name="VCCLCompilerTool"
    26186                                                         />
    26187                                                 </FileConfiguration>
    26188                                                 <FileConfiguration
    26189                                                         Name="Debug_All|Win32"
    26190                                                         ExcludedFromBuild="true"
    26191                                                         >
    26192                                                         <Tool
    26193                                                                 Name="VCCLCompilerTool"
    26194                                                         />
    26195                                                 </FileConfiguration>
    26196                                         </File>
    26197                                         <File
    26198                                                 RelativePath="..\platform\image-decoders\xbm\XBMImageDecoder.h"
    26199                                                 >
    26200                                                 <FileConfiguration
    26201                                                         Name="Debug|Win32"
    26202                                                         ExcludedFromBuild="true"
    26203                                                         >
    26204                                                         <Tool
    26205                                                                 Name="VCCustomBuildTool"
    26206                                                         />
    26207                                                 </FileConfiguration>
    26208                                                 <FileConfiguration
    26209                                                         Name="Release|Win32"
    26210                                                         ExcludedFromBuild="true"
    26211                                                         >
    26212                                                         <Tool
    26213                                                                 Name="VCCustomBuildTool"
    26214                                                         />
    26215                                                 </FileConfiguration>
    26216                                                 <FileConfiguration
    26217                                                         Name="Debug_Internal|Win32"
    26218                                                         ExcludedFromBuild="true"
    26219                                                         >
    26220                                                         <Tool
    26221                                                                 Name="VCCustomBuildTool"
    26222                                                         />
    26223                                                 </FileConfiguration>
    26224                                                 <FileConfiguration
    26225                                                         Name="Debug_All|Win32"
    26226                                                         ExcludedFromBuild="true"
    26227                                                         >
    26228                                                         <Tool
    26229                                                                 Name="VCCustomBuildTool"
    26230                                                         />
    26231                                                 </FileConfiguration>
    26232                                         </File>
    26233                                 </Filter>
    26234                         </Filter>
    2623526158                        <Filter
    2623626159                                Name="animation"
  • trunk/WebCore/WebCoreSources.bkl

    r52949 r53152  
    954954    </set>
    955955
    956     <set append="1" var="WEBCORE_SOURCES_XBM">
    957         platform/image-decoders/xbm/XBMImageDecoder.cpp
    958     </set>
    959 
    960956    <set append="1" var="WEBCORE_SOURCES_RENDERING">
    961957        rendering/AutoTableLayout.cpp
  • trunk/WebCore/platform/image-decoders/ImageDecoder.cpp

    r53148 r53152  
    3434#include "PNGImageDecoder.h"
    3535#include "SharedBuffer.h"
    36 #include "XBMImageDecoder.h"
    3736
    3837using namespace std;
     
    5756ImageDecoder* ImageDecoder::create(const SharedBuffer& data)
    5857{
    59     // XBMs require 8 bytes of info.
    60     static const unsigned maxMarkerLength = 8;
    61 
     58    // We need at least 4 bytes to figure out what kind of image we're dealing with.
     59    static const unsigned maxMarkerLength = 4;
    6260    char contents[maxMarkerLength];
    6361    unsigned length = copyFromSharedBuffer(contents, maxMarkerLength, data, 0);
    64 
    65     // We need at least 4 bytes to figure out what kind of image we're dealing with.
    66     if (length < 4)
     62    if (length < maxMarkerLength)
    6763        return 0;
    6864
     
    9591        !memcmp(contents, "\000\000\002\000", 4))
    9692        return new ICOImageDecoder();
    97 
    98     // XBMs require 8 bytes of info.
    99     if (length >= 8 && strncmp(contents, "#define ", 8) == 0)
    100         return new XBMImageDecoder();
    10193
    10294    // Give up. We don't know what the heck this is.
Note: See TracChangeset for help on using the changeset viewer.