Oilpan: mark ResourceCacheValidationSuppressor as a stack allocated object.

Source/core/fetch/ResourceFetcher.h

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller <mueller@kde.org>
     Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
     Copyright (C) 2009 Torch Mobile Inc. http://www.torchmobile.com/
 
     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
     version 2 of the License, or (at your option) any later version.
@@ skipping 17 matching lines @@
 
 #include "core/fetch/CachePolicy.h"
 #include "core/fetch/FetchContext.h"
 #include "core/fetch/FetchInitiatorInfo.h"
 #include "core/fetch/FetchRequest.h"
 #include "core/fetch/Resource.h"
 #include "core/fetch/ResourceLoaderHost.h"
 #include "core/fetch/ResourceLoaderOptions.h"
 #include "core/fetch/ResourcePtr.h"
 #include "platform/Timer.h"
-#include "wtf/Deque.h"
 #include "wtf/HashMap.h"
 #include "wtf/HashSet.h"
 #include "wtf/ListHashSet.h"
 #include "wtf/text/StringHash.h"
 
 namespace blink {
 
 class ArchiveResourceCollection;
 class CSSStyleSheetResource;
 class DocumentResource;
@@ skipping 16 matching lines @@
 // and enforces a bunch of security checks and rules for resource revalidation.
 // Its lifetime is roughly per-DocumentLoader, in that it is generally created
 // in the DocumentLoader constructor and loses its ability to generate network
 // requests when the DocumentLoader is destroyed. Documents also hold a
 // RefPtr<ResourceFetcher> for their lifetime (and will create one if they
 // are initialized without a LocalFrame), so a Document can keep a ResourceFetcher
 // alive past detach if scripts still reference the Document.
 class ResourceFetcher final : public RefCountedWillBeGarbageCollectedFinalized<ResourceFetcher>, public ResourceLoaderHost {
     WTF_MAKE_NONCOPYABLE(ResourceFetcher); WTF_MAKE_FAST_ALLOCATED_WILL_BE_REMOVED;
     WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(ResourceFetcher);
-friend class ImageLoader;
-friend class ResourceCacheValidationSuppressor;
-
 public:
     static PassRefPtrWillBeRawPtr<ResourceFetcher> create(PassOwnPtrWillBeRawPtr<FetchContext> context) { return adoptRefWillBeNoop(new ResourceFetcher(context)); }
     virtual ~ResourceFetcher();
     DECLARE_VIRTUAL_TRACE();
 
 #if !ENABLE(OILPAN)
     using RefCounted<ResourceFetcher>::ref;
     using RefCounted<ResourceFetcher>::deref;
 #endif
 
@@ skipping 83 matching lines @@
     void requestLoadStarted(Resource*, const FetchRequest&, ResourceLoadStartType);
     static const ResourceLoaderOptions& defaultResourceOptions();
 
     String getCacheIdentifier() const;
 
     virtual ResourceLoaderHost::LoaderHostType objectType() const override { return ResourceFetcherType; };
 
     static ResourceFetcher* toResourceFetcher(ResourceLoaderHost*);
 
 private:
+    friend class ImageLoader;
+    friend class ResourceCacheValidationSuppressor;
     friend class ResourceFetcherUpgradeTest;
     friend class ResourceFetcherHintsTest;
 
     explicit ResourceFetcher(PassOwnPtrWillBeRawPtr<FetchContext>);
 
     bool shouldLoadNewResource(Resource::Type) const;
 
     ResourcePtr<Resource> requestResource(Resource::Type, FetchRequest&);
     ResourcePtr<Resource> createResourceForRevalidation(const FetchRequest&, Resource*);
     ResourcePtr<Resource> createResourceForLoading(Resource::Type, FetchRequest&, const String& charset);
@@ skipping 61 matching lines @@
     DeadResourceStatsRecorder m_deadStatsRecorder;
 
     // 29 bits left
     bool m_autoLoadImages : 1;
     bool m_imagesEnabled : 1;
     bool m_allowStaleResources : 1;
 };
 
 class ResourceCacheValidationSuppressor {
     WTF_MAKE_NONCOPYABLE(ResourceCacheValidationSuppressor);
-    WTF_MAKE_FAST_ALLOCATED;
+    STACK_ALLOCATED();
 public:
     ResourceCacheValidationSuppressor(ResourceFetcher* loader)
         : m_loader(loader)
         , m_previousState(false)
     {
         if (m_loader) {
             m_previousState = m_loader->m_allowStaleResources;
             m_loader->m_allowStaleResources = true;
         }
     }
     ~ResourceCacheValidationSuppressor()
     {
         if (m_loader)
             m_loader->m_allowStaleResources = m_previousState;
     }
 private:
-    ResourceFetcher* m_loader;
+    RawPtrWillBeMember<ResourceFetcher> m_loader;
     bool m_previousState;
 };
 
 } // namespace blink
 
-#endif
+#endif // ResourceFetcher_h