PlzNavigate: support upgrade-insecure-navigation.

third_party/WebKit/Source/core/loader/FrameFetchContext.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/ResourceFetcher.h"
 #include "core/frame/csp/ContentSecurityPolicy.h"
 #include "core/loader/LinkLoader.h"
 #include "platform/heap/Handle.h"
 #include "platform/network/ResourceRequest.h"
 
 namespace blink {
 
 class Document;
 class DocumentLoader;
+class FrameLoader;
 class LocalFrame;
 class ResourceError;
 class ResourceLoader;
 class ResourceResponse;
 class ResourceRequest;
 
 class CORE_EXPORT FrameFetchContext final : public FetchContext {
 public:
     static ResourceFetcher* createContextAndFetcher(DocumentLoader* loader, Document* document)
     {
@@ skipping 36 matching lines @@
     int64_t serviceWorkerID() const override;
 
     bool isMainFrame() const override;
     bool defersLoading() const override;
     bool isLoadComplete() const override;
     bool pageDismissalEventBeingDispatched() const override;
     bool updateTimingInfoForIFrameNavigation(ResourceTimingInfo*) override;
     void sendImagePing(const KURL&) override;
     void addConsoleMessage(const String&) const override;
     SecurityOrigin* getSecurityOrigin() const override;
+    static void upgradeInsecureRequest(ResourceRequest&, Document*, FrameLoader*);
     void upgradeInsecureRequest(ResourceRequest&) override;
     void addClientHintsIfNecessary(FetchRequest&) override;
     void addCSPHeaderIfNecessary(Resource::Type, FetchRequest&) override;
     void populateRequestData(ResourceRequest&) override;
 
     MHTMLArchive* archive() const override;
 
     ResourceLoadPriority modifyPriorityForExperiments(ResourceLoadPriority) override;
 
     void countClientHintsDPR() override;
     void countClientHintsResourceWidth() override;
     void countClientHintsViewportWidth() override;
 
     WebTaskRunner* loadingTaskRunner() const override;
 
     DECLARE_VIRTUAL_TRACE();
 
 private:
     explicit FrameFetchContext(DocumentLoader*, Document*);
     inline DocumentLoader* masterDocumentLoader() const;
 
     LocalFrame* frame() const; // Can be null

[arthursonzogni, 2016/08/29 08:37:22]

Because in the definition, there is an "ASSERT(frame)", the function is not
expected to return null. 
Maybe this was the case in the past and the comment "Can be null" was not
removed. Or maybe this is in case the definition could change in the future and
returns null.
Do you think I can I assume frame() will never return null ? Then maybe we could
remove the comment.

     void printAccessDeniedMessage(const KURL&) const;
     ResourceRequestBlockedReason canRequestInternal(Resource::Type, const ResourceRequest&, const KURL&, const ResourceLoaderOptions&, bool forPreload, FetchRequest::OriginRestriction, ResourceRequest::RedirectStatus) const;
 
     void prepareRequest(ResourceRequest&);
 
     void dispatchDidReceiveResponseInternal(unsigned long identifier, const ResourceResponse&, WebURLRequest::FrameType, WebURLRequest::RequestContext, Resource*, LinkLoader::CanLoadResources);
 
     // FIXME: Oilpan: Ideally this should just be a traced Member but that will
     // currently leak because ComputedStyle and its data are not on the heap.
     // See crbug.com/383860 for details.
     WeakMember<Document> m_document;
     Member<DocumentLoader> m_documentLoader;
 };
 
 } // namespace blink
 
 #endif