Change the BMP decoder to not reject paletted images which use color indexes

Source/platform/image-decoders/bmp/BMPImageReader.h

 /*
  * Copyright (c) 2008, 2009, 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 314 matching lines @@
     // the 32 bits of pixel data, and left shift to scale the color values
     // up for fields with less than 8 bits of precision.  Sadly, we can't
     // just combine these into one shift value because the net shift amount
     // could go either direction.  (If only "<< -x" were equivalent to
     // ">> x"...)
     uint32_t m_bitMasks[4];
     int m_bitShiftsRight[4];
     int m_bitShiftsLeft[4];
 
     // The color palette, for paletted formats.
-    size_t m_tableSizeInBytes;

[Peter Kasting, 2014/03/26 21:35:52]

This is just cleanup unconnected to the bugfix: we don't need this to be a
member, it can be a function-local.

     Vector<RGBTriple> m_colorTable;
 
     // The coordinate to which we've decoded the image.
     IntPoint m_coord;
 
     // Variables that track whether we've seen pixels with alpha values != 0
     // and == 0, respectively.  See comments in processNonRLEData() on how
     // these are used.
     bool m_seenNonZeroAlphaPixel;
     bool m_seenZeroAlphaPixel;
 
     // ICOs store a 1bpp "mask" immediately after the main bitmap image data
     // (and, confusingly, add its height to the biHeight value in the info
     // header, thus doubling it).  This variable tracks whether we have such
     // a mask and if we've started decoding it yet.
     AndMaskState m_andMaskState;
 };
 
 } // namespace WebCore
 
 #endif