Don't use FetchRequest in FrameFetchContext

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 25 matching lines @@
 #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 blink {
 
+class ClientHintsPreferences;
 class KURL;
 class MHTMLArchive;
 class ResourceError;
 class ResourceResponse;
 class ResourceTimingInfo;
 class WebTaskRunner;
 enum class WebCachePolicy;
 
 enum FetchResourceType { FetchMainResource, FetchSubresource };
 
@@ skipping 93 matching lines @@
   virtual bool defersLoading() const { return false; }
   virtual bool isLoadComplete() const { return false; }
   virtual bool pageDismissalEventBeingDispatched() const { return false; }
   virtual bool updateTimingInfoForIFrameNavigation(ResourceTimingInfo*) {
     return false;
   }
   virtual void sendImagePing(const KURL&);
   virtual void addConsoleMessage(const String&,
                                  LogMessageType = LogErrorMessage) const;
   virtual SecurityOrigin* getSecurityOrigin() const { return nullptr; }
-  virtual void modifyRequestForCSP(ResourceRequest&);
-  virtual void addClientHintsIfNecessary(FetchRequest&);
-  virtual void addCSPHeaderIfNecessary(Resource::Type, FetchRequest&);
-  virtual void populateRequestData(ResourceRequest&);
+
+  virtual void setFirstPartyCookieAndRequestorOrigin(ResourceRequest&);
+  virtual void populateResourceRequest(Resource::Type,

[yhirano, 2016/12/26 08:56:45]

Can you add a method comment?

[tyoshino (SeeGerritForStatus), 2017/01/10 09:36:28]

Done.

+                                       const ClientHintsPreferences&,
+                                       const FetchRequest::ResourceWidth&,
+                                       ResourceRequest&);
 
   virtual MHTMLArchive* archive() const { return nullptr; }
 
   virtual ResourceLoadPriority modifyPriorityForExperiments(
       ResourceLoadPriority priority) {
     return priority;
   }
 
   virtual WebTaskRunner* loadingTaskRunner() const { return nullptr; }
 
  protected:
   FetchContext() {}
 };
 
 }  // namespace blink
 
 #endif