Refactoring: Make member variables in Resource private

third_party/WebKit/Source/core/fetch/RawResource.cpp

 /*
  * Copyright (C) 2011 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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 180 matching lines @@
 
 void RawResource::reportResourceTimingToClients(const ResourceTimingInfo& info)
 {
     ResourceClientWalker<RawResourceClient> w(clients());
     while (RawResourceClient* c = w.next())
         c->didReceiveResourceTiming(this, info);
 }
 
 void RawResource::setDefersLoading(bool defers)
 {
-    if (m_loader)
-        m_loader->setDefersLoading(defers);
+    if (loader())
+        loader()->setDefersLoading(defers);
 }
 
 static bool shouldIgnoreHeaderForCacheReuse(AtomicString headerName)
 {
     // FIXME: This list of headers that don't affect cache policy almost certainly isn't complete.
     DEFINE_STATIC_LOCAL(HashSet<AtomicString>, headers, ({
         "Cache-Control",
         "If-Modified-Since",
         "If-None-Match",
         "Origin",
@@ skipping 11 matching lines @@
     // Per http://www.w3.org/Protocols/rfc2616/rfc2616-sec13.html#sec13.10,
     // these methods always invalidate the cache entry.
     return method != "POST" && method != "PUT" && method != "DELETE";
 }
 
 bool RawResource::canReuse(const ResourceRequest& newRequest) const
 {
     if (dataBufferingPolicy() == DoNotBufferData)
         return false;
 
-    if (!isCacheableHTTPMethod(m_resourceRequest.httpMethod()))
+    if (!isCacheableHTTPMethod(resourceRequest().httpMethod()))
         return false;
-    if (m_resourceRequest.httpMethod() != newRequest.httpMethod())
+    if (resourceRequest().httpMethod() != newRequest.httpMethod())
         return false;
 
-    if (m_resourceRequest.httpBody() != newRequest.httpBody())
+    if (resourceRequest().httpBody() != newRequest.httpBody())
         return false;
 
-    if (m_resourceRequest.allowStoredCredentials() != newRequest.allowStoredCredentials())
+    if (resourceRequest().allowStoredCredentials() != newRequest.allowStoredCredentials())
         return false;
 
     // Ensure most headers match the existing headers before continuing.
     // Note that the list of ignored headers includes some headers explicitly related to caching.
     // A more detailed check of caching policy will be performed later, this is simply a list of
     // headers that we might permit to be different and still reuse the existing Resource.
     const HTTPHeaderMap& newHeaders = newRequest.httpHeaderFields();
-    const HTTPHeaderMap& oldHeaders = m_resourceRequest.httpHeaderFields();
+    const HTTPHeaderMap& oldHeaders = resourceRequest().httpHeaderFields();
 
     for (const auto& header : newHeaders) {
         AtomicString headerName = header.key;
         if (!shouldIgnoreHeaderForCacheReuse(headerName) && header.value != oldHeaders.get(headerName))
             return false;
     }
 
     for (const auto& header : oldHeaders) {
         AtomicString headerName = header.key;
         if (!shouldIgnoreHeaderForCacheReuse(headerName) && header.value != newHeaders.get(headerName))
             return false;
     }
 
     return true;
 }
 
 } // namespace blink