Loading: move V8DOMActivityLogger dependency from core/fetch

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

 /*
  * Copyright (C) 2013 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 23 matching lines @@
 #include "core/CoreExport.h"
 #include "core/fetch/CachePolicy.h"
 #include "core/fetch/FetchInitiatorInfo.h"
 #include "core/fetch/FetchRequest.h"
 #include "core/fetch/Resource.h"
 #include "platform/heap/Handle.h"
 #include "platform/network/ResourceLoadPriority.h"
 #include "platform/network/ResourceRequest.h"
 #include "wtf/Noncopyable.h"
 
+namespace wtf {
+
+class AtomicString;
+
+}  // namespace wtf
+
 namespace blink {
 
 class KURL;
 class MHTMLArchive;
 class ResourceError;
 class ResourceResponse;
 class ResourceTimingInfo;
 class WebTaskRunner;
 enum class WebCachePolicy;
 
@@ skipping 63 matching lines @@
   virtual void dispatchDidFail(unsigned long identifier,
                                const ResourceError&,
                                int64_t encodedDataLength,
                                bool isInternalRequest);
 
   virtual bool shouldLoadNewResource(Resource::Type) const { return false; }
   // Called when a resource load is first requested, which may not be when the
   // load actually begins.
   virtual void willStartLoadingResource(unsigned long identifier,
                                         ResourceRequest&,
-                                        Resource::Type);
+                                        Resource::Type,
+                                        const AtomicString& fetchInitiatorName,
+                                        bool forPreload);

[yhirano, 2016/12/19 11:44:47]

Please pass an enum instead of boolean.

[Takashi Toyoshima, 2016/12/19 12:58:54]

I feel the enum is better to be defined in FetchRequest.
So, I want to leave TODOs in this CL, and fix it in a separate CL.

   virtual void didLoadResource(Resource*);
 
   virtual void addResourceTiming(const ResourceTimingInfo&);
   virtual bool allowImage(bool, const KURL&) const { return false; }
   virtual ResourceRequestBlockedReason canRequest(
       Resource::Type,
       const ResourceRequest&,
       const KURL&,
       const ResourceLoaderOptions&,
       bool forPreload,
@@ skipping 36 matching lines @@
 
   virtual WebTaskRunner* loadingTaskRunner() const { return nullptr; }
 
  protected:
   FetchContext() {}
 };
 
 }  // namespace blink
 
 #endif