Bug 1282566 (Part 2) - Remove some unused fields from nsPNGDecoder. r=edwin
authorSeth Fowler <mark.seth.fowler@gmail.com>
Mon, 27 Jun 2016 12:01:50 -0700
changeset 343212 442c01c74c5f715e2f6263257ae994a07fa1956d
parent 343211 748f5424739f63fd4596f7e5adb876b28a4cfc79
child 343213 3405db522027deb4b59112d9ff5f3ecde8e00cf7
push id6389
push userraliiev@mozilla.com
push dateMon, 19 Sep 2016 13:38:22 +0000
treeherdermozilla-beta@01d67bfe6c81 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersedwin
bugs1282566
milestone50.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1282566 (Part 2) - Remove some unused fields from nsPNGDecoder. r=edwin
image/decoders/nsPNGDecoder.cpp
image/decoders/nsPNGDecoder.h
--- a/image/decoders/nsPNGDecoder.cpp
+++ b/image/decoders/nsPNGDecoder.cpp
@@ -96,17 +96,16 @@ nsPNGDecoder::nsPNGDecoder(RasterImage* 
  : Decoder(aImage)
  , mPNG(nullptr)
  , mInfo(nullptr)
  , mCMSLine(nullptr)
  , interlacebuf(nullptr)
  , mInProfile(nullptr)
  , mTransform(nullptr)
  , format(gfx::SurfaceFormat::UNKNOWN)
- , mHeaderBytesRead(0)
  , mCMSMode(0)
  , mChannels(0)
  , mPass(0)
  , mFrameIsHidden(false)
  , mDisablePremultipliedAlpha(false)
  , mNumFrames(0)
 { }
 
--- a/image/decoders/nsPNGDecoder.h
+++ b/image/decoders/nsPNGDecoder.h
@@ -90,20 +90,16 @@ public:
   nsIntRect mFrameRect;
   uint8_t* mCMSLine;
   uint8_t* interlacebuf;
   qcms_profile* mInProfile;
   qcms_transform* mTransform;
 
   gfx::SurfaceFormat format;
 
-  // For metadata decodes.
-  uint8_t mSizeBytes[8]; // Space for width and height, both 4 bytes
-  uint32_t mHeaderBytesRead;
-
   // whether CMS or premultiplied alpha are forced off
   uint32_t mCMSMode;
 
   uint8_t mChannels;
   uint8_t mPass;
   bool mFrameIsHidden;
   bool mDisablePremultipliedAlpha;