Eliminate copies of encoded image data

third_party/WebKit/Source/platform/image-decoders/webp/WEBPImageDecoder.cpp

 /*
  * Copyright (C) 2010 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 144 matching lines @@
 }
 
 void WEBPImageDecoder::clearDecoder()
 {
     WebPIDelete(m_decoder);
     m_decoder = 0;
     m_decodedHeight = 0;
     m_frameBackgroundHasAlpha = false;
 }
 
-void WEBPImageDecoder::onSetData(SharedBuffer*)
+void WEBPImageDecoder::onSetData(SegmentReader*)
 {
     m_haveAlreadyParsedThisData = false;
 }
 
 int WEBPImageDecoder::repetitionCount() const
 {
     return failed() ? cAnimationLoopOnce : m_repetitionCount;
 }
 
 bool WEBPImageDecoder::frameIsCompleteAtIndex(size_t index) const
@@ skipping 19 matching lines @@
     if (m_haveAlreadyParsedThisData)
         return true;
 
     m_haveAlreadyParsedThisData = true;
 
     const unsigned webpHeaderSize = 30;
     if (m_data->size() < webpHeaderSize)
         return false; // Await VP8X header so WebPDemuxPartial succeeds.
 
     WebPDemuxDelete(m_demux);
-    WebPData inputData = { reinterpret_cast<const uint8_t*>(m_data->data()), m_data->size() };
+    m_encodedData = skia::AdoptRef(m_data->getAsSkData().leakRef());

[Peter Kasting, 2016/03/23 02:42:59]

Is AdoptRef(leakRef()) really the only way to do this?  I don't know these
interfaces that well but it would sure be nice to ditch both those...

[f(malita), 2016/03/23 16:41:58]

If we switch to RefPtr, this should simplify to m_encodedData =
m_data->getAsSkData();

[scroggo_chromium, 2016/03/24 13:59:46]

Done.

+    WebPData inputData = { reinterpret_cast<const uint8_t*>(m_encodedData->data()), m_encodedData->size() };
     m_demux = WebPDemuxPartial(&inputData, &m_demuxState);
     if (!m_demux || (isAllDataReceived() && m_demuxState != WEBP_DEMUX_DONE))
         return setFailed();
 
     ASSERT(m_demuxState > WEBP_DEMUX_PARSING_HEADER);
     if (!WebPDemuxGetI(m_demux, WEBP_FF_FRAME_COUNT))
         return false; // Wait until the encoded image frame data arrives.
 
     if (!isDecodedSizeAvailable()) {
         int width = WebPDemuxGetI(m_demux, WEBP_FF_CANVAS_WIDTH);
@@ skipping 340 matching lines @@
             return false;
         }
         // FALLTHROUGH
     default:
         clear();
         return setFailed();
     }
 }
 
 } // namespace blink