Ensure setEncodedSizeMemoryUsage() is called when |m_data| is cleared

third_party/WebKit/Source/core/fetch/Resource.h

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller <mueller@kde.org>
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All
     rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
@@ skipping 316 matching lines @@
   void notifyClientsInternal(MarkFinishedOption);
   void markClientFinished(ResourceClient*);
 
   virtual bool hasClientsOrObservers() const {
     return !m_clients.isEmpty() || !m_clientsAwaitingCallback.isEmpty() ||
            !m_finishedClients.isEmpty();
   }
   virtual void destroyDecodedDataForFailedRevalidation() {}
 
   void setEncodedSize(size_t);
-  void setEncodedSizeMemoryUsage(size_t);
   void setDecodedSize(size_t);
   void didAccessDecodedData();
 
   void finishPendingClients();
 
   virtual void didAddClient(ResourceClient*);
   void willAddClientOrObserver(PreloadReferencePolicy);
 
   void didRemoveClientOrObserver();
   virtual void allClientsAndObserversRemoved();
@@ skipping 27 matching lines @@
   }
   DataBufferingPolicy dataBufferingPolicy() const {
     return m_options.dataBufferingPolicy;
   }
 
   void setCachePolicyBypassingCache();
   void setLoFiStateOff();
   void clearRangeRequestHeader();
 
   SharedBuffer* data() const { return m_data.get(); }
-  void clearData() { m_data.clear(); }
+  void clearData();
 
   class ProhibitAddRemoveClientInScope : public AutoReset<bool> {
    public:
     ProhibitAddRemoveClientInScope(Resource* resource)
         : AutoReset(&resource->m_isAddRemoveClientProhibited, true) {}
   };
 
  private:
   class ResourceCallback;
   class CachedMetadataHandlerImpl;
   class ServiceWorkerResponseCachedMetadataHandler;
 
   void cancelTimerFired(TimerBase*);
 
   void revalidationSucceeded(const ResourceResponse&);
   void revalidationFailed();
 
+  void setEncodedSizeMemoryUsage(size_t);

[yhirano, 2016/11/08 01:59:41]

[optional] do we need this function?

[hiroshige, 2016/11/09 07:33:43]

Removed.

   size_t calculateOverheadSize() const;
 
   String reasonNotDeletable() const;
 
   // MemoryCoordinatorClient overrides:
   void onMemoryStateChange(MemoryState) override;
 
   Member<CachedMetadataHandlerImpl> m_cacheHandler;
   RefPtr<SecurityOrigin> m_fetcherSecurityOrigin;
 
@@ skipping 68 matching lines @@
 };
 
 #define DEFINE_RESOURCE_TYPE_CASTS(typeName)                   \
   DEFINE_TYPE_CASTS(typeName##Resource, Resource, resource,    \
                     resource->getType() == Resource::typeName, \
                     resource.getType() == Resource::typeName);
 
 }  // namespace blink
 
 #endif