Use LocalFrame::client() than getting the FrameLoader and then call client() on it.

third_party/WebKit/Source/core/loader/FrameFetchContext.cpp

 /*
  * 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 166 matching lines @@
 
   // Add the metadata that this page has scripts inserted via document.write
   // that are eligible for blocking. Note that if there are multiple scripts
   // the flag will be conveyed to the browser process only once.
   document.loader()->didObserveLoadingBehavior(
       WebLoadingBehaviorFlag::WebLoadingBehaviorDocumentWriteBlock);
 
   const bool is2G =
       networkStateNotifier().connectionType() == WebConnectionTypeCellular2G;
   WebEffectiveConnectionType effectiveConnection =
-      document.frame()->loader().client()->getEffectiveConnectionType();
+      document.frame()->client()->getEffectiveConnectionType();
   const bool is2GOrLike2G =
       is2G || isConnectionEffectively2G(effectiveConnection);
 
   return document.settings()
              ->getDisallowFetchForDocWrittenScriptsInMainFrame() ||
          (document.settings()
               ->getDisallowFetchForDocWrittenScriptsInMainFrameOnSlowConnections() &&
           is2G) ||
          (document.settings()
               ->getDisallowFetchForDocWrittenScriptsInMainFrameIfEffectively2G() &&
@@ skipping 23 matching lines @@
 
 LocalFrame* FrameFetchContext::frame() const {
   if (!m_documentLoader)
     return frameOfImportsController();
 
   LocalFrame* frame = m_documentLoader->frame();
   DCHECK(frame);
   return frame;
 }
 
+FrameLoaderClient* FrameFetchContext::frameLoaderClient() const {
+  return frame()->client();
+}
+
 void FrameFetchContext::addAdditionalRequestHeaders(ResourceRequest& request,
                                                     FetchResourceType type) {
   bool isMainResource = type == FetchMainResource;
   if (!isMainResource) {
     if (!request.didSetHTTPReferrer()) {
       DCHECK(m_document);
       request.setHTTPReferrer(SecurityPolicy::generateReferrer(
           m_document->getReferrerPolicy(), request.url(),
           m_document->outgoingReferrer()));
       request.addHTTPOriginIfNeeded(m_document->getSecurityOrigin());
@@ skipping 143 matching lines @@
     int intraPriorityValue) {
   TRACE_EVENT1(
       "devtools.timeline", "ResourceChangePriority", "data",
       InspectorChangeResourcePriorityEvent::data(identifier, loadPriority));
   InspectorInstrumentation::didChangeResourcePriority(frame(), identifier,
                                                       loadPriority);
 }
 
 void FrameFetchContext::prepareRequest(ResourceRequest& request) {
   frame()->loader().applyUserAgent(request);
-  frame()->loader().client()->dispatchWillSendRequest(request);
+  frameLoaderClient()->dispatchWillSendRequest(request);
 }
 
 void FrameFetchContext::dispatchWillSendRequest(
     unsigned long identifier,
     ResourceRequest& request,
     const ResourceResponse& redirectResponse,
     const FetchInitiatorInfo& initiatorInfo) {
   TRACE_EVENT1("devtools.timeline", "ResourceSendRequest", "data",
                InspectorSendRequestEvent::data(identifier, frame(), request));
   // For initial requests, prepareRequest() is called in
@@ skipping 83 matching lines @@
 }
 
 void FrameFetchContext::dispatchDidLoadResourceFromMemoryCache(
     unsigned long identifier,
     Resource* resource,
     WebURLRequest::FrameType frameType,
     WebURLRequest::RequestContext requestContext) {
   ResourceRequest request(resource->url());
   request.setFrameType(frameType);
   request.setRequestContext(requestContext);
-  frame()->loader().client()->dispatchDidLoadResourceFromMemoryCache(
+  frameLoaderClient()->dispatchDidLoadResourceFromMemoryCache(
       request, resource->response());
   dispatchWillSendRequest(identifier, request, ResourceResponse(),
                           resource->options().initiatorInfo);
 
   InspectorInstrumentation::markResourceAsCached(frame(), identifier);
   if (!resource->response().isNull()) {
     dispatchDidReceiveResponseInternal(identifier, resource->response(),
                                        frameType, requestContext, resource,
                                        LinkLoader::DoNotLoadResources);
   }
@@ skipping 75 matching lines @@
   Document* initiatorDocument = m_document && info.isMainResource()
                                     ? m_document->parentDocument()
                                     : m_document.get();
   if (!initiatorDocument || !initiatorDocument->domWindow())
     return;
   DOMWindowPerformance::performance(*initiatorDocument->domWindow())
       ->addResourceTiming(info);
 }
 
 bool FrameFetchContext::allowImage(bool imagesEnabled, const KURL& url) const {
-  return frame()->loader().client()->allowImage(imagesEnabled, url);
+  return frameLoaderClient()->allowImage(imagesEnabled, url);
 }
 
 void FrameFetchContext::printAccessDeniedMessage(const KURL& url) const {
   if (url.isNull())
     return;
 
   String message;
   if (!m_document || m_document->url().isNull()) {
     message = "Unsafe attempt to load URL " + url.elidedString() + '.';
   } else if (url.isLocalFile() || m_document->url().isLocalFile()) {
@@ skipping 120 matching lines @@
     if (!shouldBypassMainWorldCSP &&
         !m_document->contentSecurityPolicy()->allowRequest(
             resourceRequest.requestContext(), url,
             options.contentSecurityPolicyNonce, options.integrityMetadata,
             options.parserDisposition, redirectStatus, cspReporting))
       return ResourceRequestBlockedReason::CSP;
   }
 
   if (type == Resource::Script || type == Resource::ImportResource) {
     DCHECK(frame());
-    if (!frame()->loader().client()->allowScriptFromSource(
+    if (!frameLoaderClient()->allowScriptFromSource(
             !frame()->settings() || frame()->settings()->getScriptEnabled(),
             url)) {
-      frame()->loader().client()->didNotAllowScript();
+      frameLoaderClient()->didNotAllowScript();
       // TODO(estark): Use a different ResourceRequestBlockedReason here, since
       // this check has nothing to do with CSP. https://crbug.com/600795
       return ResourceRequestBlockedReason::CSP;
     }
   } else if (type == Resource::Media || type == Resource::TextTrack) {
     DCHECK(frame());
-    if (!frame()->loader().client()->allowMedia(url))
+    if (!frameLoaderClient()->allowMedia(url))
       return ResourceRequestBlockedReason::Other;
   }
 
   // SVG Images have unique security rules that prevent all subresource requests
   // except for data urls.
   if (type != Resource::MainResource &&
       frame()->chromeClient().isSVGImageChromeClient() && !url.protocolIsData())
     return ResourceRequestBlockedReason::Origin;
 
   // Measure the number of legacy URL schemes ('ftp://') and the number of
@@ skipping 56 matching lines @@
 
   if (m_documentLoader) {
     return m_documentLoader->frame()
         ->loader()
         .client()
         ->isControlledByServiceWorker(*m_documentLoader);
   }
   // m_documentLoader is null while loading resources from an HTML import. In
   // such cases whether the request is controlled by ServiceWorker or not is
   // determined by the document loader of the frame.
-  return frame()->loader().client()->isControlledByServiceWorker(
+  return frameLoaderClient()->isControlledByServiceWorker(
       *frame()->loader().documentLoader());
 }
 
 int64_t FrameFetchContext::serviceWorkerID() const {
   DCHECK(m_documentLoader || frame()->loader().documentLoader());
   if (m_documentLoader) {
-    return m_documentLoader->frame()->loader().client()->serviceWorkerID(
+    return m_documentLoader->frame()->client()->serviceWorkerID(
         *m_documentLoader);
   }
   // m_documentLoader is null while loading resources from an HTML import.
   // In such cases a service worker ID could be retrieved from the document
   // loader of the frame.
-  return frame()->loader().client()->serviceWorkerID(
+  return frameLoaderClient()->serviceWorkerID(
       *frame()->loader().documentLoader());
 }
 
 bool FrameFetchContext::isMainFrame() const {
   return frame()->isMainFrame();
 }
 
 bool FrameFetchContext::defersLoading() const {
   return frame()->page()->suspended();
 }
@@ skipping 193 matching lines @@
       response.httpHeaderField(HTTPNames::Link), response.url(),
       frame()->document(), NetworkHintsInterfaceImpl(), resourceLoadingPolicy,
       LinkLoader::LoadAll, nullptr);
 
   if (response.hasMajorCertificateErrors()) {
     MixedContentChecker::handleCertificateError(frame(), response, frameType,
                                                 requestContext);
   }
 
   frame()->loader().progress().incrementProgress(identifier, response);
-  frame()->loader().client()->dispatchDidReceiveResponse(response);
+  frameLoaderClient()->dispatchDidReceiveResponse(response);
   DocumentLoader* documentLoader = masterDocumentLoader();
   InspectorInstrumentation::didReceiveResourceResponse(
       frame(), identifier, documentLoader, response, resource);
   // It is essential that inspector gets resource response BEFORE console.
   frame()->console().reportResourceResponseReceived(documentLoader, identifier,
                                                     response);
 }
 
 DEFINE_TRACE(FrameFetchContext) {
   visitor->trace(m_document);
   visitor->trace(m_documentLoader);
   FetchContext::trace(visitor);
 }
 
 }  // namespace blink