Small cleanup on BMPImageReader.cpp

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 245 matching lines @@
     inline void fillRGBA(int endCoord,
                          unsigned red,
                          unsigned green,
                          unsigned blue,
                          unsigned alpha)
     {
         while (m_coord.x() < endCoord)
             setRGBA(red, green, blue, alpha);
     }
 
+    // File header size + info header size in bytes.
+    inline size_t sizeOfHeaderAndInfoHeader() const

[Peter Kasting, 2015/02/03 23:56:43]

Don't add this.  Do it inline in the places you call it.

[changseok, 2015/02/04 04:10:32]

Done.

+    {
+        return m_headerOffset + m_infoHeader.biSize;
+    }
+
     // Resets the relevant local variables to start drawing at the left edge
     // of the "next" row, where "next" is above or below the current row
     // depending on the value of |m_isTopDown|.
     void moveBufferToNextRow();
 
     // The decoder that owns us.
     ImageDecoder* m_parent;
 
     // The destination for the pixel data.
     ImageFrame* m_buffer;
@@ skipping 69 matching lines @@
     // 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). If |m_isInICO| is true, this variable tracks
     // whether we've begun decoding this mask yet.
     bool m_decodingAndMask;
 };
 
 } // namespace blink
 
 #endif