Remove opaque alpha channel special case

third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTest.cpp

 /*
  * Copyright (C) 2012 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 335 matching lines @@
             ColorBehavior::transformToTargetForTesting());
 
     SkImageInfo pixInfo = SkImageInfo::MakeN32Premul(1, 1);
 
     size_t rowBytes = pixInfo.minRowBytes();
     size_t size = pixInfo.getSafeSize(rowBytes);
 
     Vector<char> storage(size);
     SkPixmap pixmap(pixInfo, storage.data(), rowBytes);
 
-    // Before decoding, the frame is not known to be opaque.
+    // Before decoding, the frame's alpha is unknown and assumed to be
+    // transparent. This works well with partial decoding, which will be
+    // transparent until the full first frame is decoded.
     sk_sp<SkImage> frame = decoder->createFrameAtIndex(0);
     ASSERT_TRUE(frame);
     EXPECT_FALSE(frame->isOpaque());
 
     // Force a lazy decode by reading pixels.
     EXPECT_TRUE(frame->readPixels(pixmap, 0, 0));
 
     // After decoding, the frame is known to be opaque.
     frame = decoder->createFrameAtIndex(0);
     ASSERT_TRUE(frame);
-    EXPECT_TRUE(frame->isOpaque());
+    // The GIF file specifies it is opaque.
+    // Without the file, we remain at the default: transparent
+    EXPECT_EQ(testGIF, frame->isOpaque());
 
     // Re-generating the opaque-marked frame should not fail.
     EXPECT_TRUE(frame->readPixels(pixmap, 0, 0));
   }
 }
 
 // The DeferredImageDecoder would sometimes assume that a frame was a certain
 // size even if the actual decoder reported it had a size of 0 bytes. This test
 // verifies that it's fixed by always checking with the actual decoder when
 // creating a frame.
@@ skipping 16 matching lines @@
       SharedBuffer::create(m_data->data(), m_data->size());
   EXPECT_EQ(originalData->size(), m_data->size());
   m_lazyDecoder->setData(originalData, false);
   RefPtr<SharedBuffer> newData = m_lazyDecoder->data();
   EXPECT_EQ(originalData->size(), newData->size());
   EXPECT_EQ(
       0, std::memcmp(originalData->data(), newData->data(), newData->size()));
 }
 
 }  // namespace blink