Remove duplication of encoded image data

third_party/WebKit/Source/platform/graphics/BitmapImage.h

 /*
  * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2004, 2005, 2006 Apple Computer, Inc.  All rights reserved.
  * Copyright (C) 2008-2009 Torch Mobile, Inc.
  *
  * 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.
@@ skipping 107 matching lines @@
 
     // Returns the total number of bytes allocated for all framebuffers, i.e.
     // the sum of m_source.frameBytesAtIndex(...) for all frames.
     size_t totalFrameBytes();
 
     // Called to wipe out the entire frame buffer cache and tell the image
     // source to destroy everything; this is used when e.g. we want to free
     // some room in the image cache.
     void destroyDecodedData() override;
 
+    PassRefPtr<SharedBuffer> data() override;
+    SharedBuffer* cachedData();

[scroggo_chromium, 2016/06/20 19:00:43]

Why does this return a raw pointer?

It calls a method that returns a PassRefPtr, and one of the callers then wraps
it back into a PassRefPtr. The other caller presumably doesn't care whether it's
a raw pointer or a PassRefPtr, but it seems like there's no problem with it
being a PassRefPtr?

[hajimehoshi, 2016/06/22 09:42:29]

Done.

+
     // Notifies observers that the memory footprint has changed.
     void notifyMemoryChanged();
 
     // Whether or not size is available yet.
     bool isSizeAvailable();
 
     // Animation.
     // We start and stop animating lazily.  Animation starts when the image is
     // rendered, and automatically stops once no observer wants to render the
     // image.
@@ skipping 42 matching lines @@
     mutable bool m_haveSize : 1; // Whether or not our |m_size| member variable has the final overall image size yet.
     bool m_sizeAvailable : 1; // Whether or not we can obtain the size of the first image frame yet from ImageIO.
     mutable bool m_haveFrameCount : 1;
 };
 
 DEFINE_IMAGE_TYPE_CASTS(BitmapImage);
 
 } // namespace blink
 
 #endif