Encapsulate |MemoryCacheEntry::m_resource| and refactor pruneDead/LiveResources

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

Index: third_party/WebKit/Source/core/fetch/MemoryCache.h
diff --git a/third_party/WebKit/Source/core/fetch/MemoryCache.h b/third_party/WebKit/Source/core/fetch/MemoryCache.h
index 529a08128d0fe16d5478b9695005812d743cad6c..5aa2222de5c589a7237de55dfe4f81bdfd07ab84 100644
--- a/third_party/WebKit/Source/core/fetch/MemoryCache.h
+++ b/third_party/WebKit/Source/core/fetch/MemoryCache.h
@@ -74,8 +74,8 @@ public:
     }
     DECLARE_TRACE();
     void dispose();
+    Resource* resource();
 
-    Member<Resource> m_resource;
     bool m_inLiveDecodedResourcesList;
     unsigned m_accessCount;
     double m_lastDecodedAccessTime; // Used as a thrash guard
@@ -86,15 +86,17 @@ public:
     Member<MemoryCacheEntry> m_nextInAllResourcesList;
 
 private:
+    Member<Resource> m_resource;

[Nate Chapin, 2016/04/26 17:34:36]

Nit: I think it's an unwritten convention that private member variables are
placed below the private functions?

+
     explicit MemoryCacheEntry(Resource* resource)
-        : m_resource(resource)
-        , m_inLiveDecodedResourcesList(false)
+        : m_inLiveDecodedResourcesList(false)
         , m_accessCount(0)
         , m_lastDecodedAccessTime(0.0)
         , m_previousInLiveResourcesList(nullptr)
         , m_nextInLiveResourcesList(nullptr)
         , m_previousInAllResourcesList(nullptr)
         , m_nextInAllResourcesList(nullptr)
+        , m_resource(resource)
     {
     }
 };