Frame serializer: do not ask for cache control on request

Frame serializer currently asks DocumentLoader's request whether
cache control no store is present.

However, we are not setting headers on fake navigation requests
for a long time already. See CreateURLRequestForNavigation
in render_frame_impl.cc for a reference.

This patch removes the check which is never true nowdays.

Bug: none
Change-Id: I6098ee39dabc414573f991a7904b53e344216eac
Reviewed-on: https://chromium-review.googlesource.com/c/1394997Reviewed-by: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Justin DeWitt <dewittj@chromium.org>
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#620013}

third_party/blink/renderer/core/exported/web_frame_serializer.cc

@@ -403,14 +403,10 @@ std::pair<Node*, Element*> MHTMLFrameSerializerDelegate::GetAuxiliaryDOMTree(
 
 bool CacheControlNoStoreHeaderPresent(
     const WebLocalFrameImpl& web_local_frame) {
-  const ResourceResponse& response =
-      web_local_frame.GetDocumentLoader()->GetResponse().ToResourceResponse();
-  if (response.CacheControlContainsNoStore())
-    return true;
-
-  DocumentLoader* document_loader =
-      web_local_frame.GetFrame()->Loader().GetDocumentLoader();
-  return document_loader->CacheControlContainsNoStore();
+  return web_local_frame.GetDocumentLoader()
+      ->GetResponse()
+      .ToResourceResponse()
+      .CacheControlContainsNoStore();
 }
 
 bool FrameShouldBeSerializedAsMHTML(

third_party/blink/renderer/core/loader/document_loader.cc

@@ -373,10 +373,6 @@ EncodedFormData* DocumentLoader::HttpBody() const {
   return request_.HttpBody();
 }
 
-bool DocumentLoader::CacheControlContainsNoStore() const {
-  return request_.CacheControlContainsNoStore();
-}
-
 void DocumentLoader::SetHistoryItemStateForCommit(
     HistoryItem* old_item,
     WebFrameLoadType load_type,

third_party/blink/renderer/core/loader/document_loader.h

@@ -132,7 +132,6 @@ class CORE_EXPORT DocumentLoader
   const KURL& UrlForHistory() const;
   const AtomicString& Referrer() const;
   EncodedFormData* HttpBody() const;
-  bool CacheControlContainsNoStore() const;
 
   void DidChangePerformanceTiming();
   void DidObserveLoadingBehavior(WebLoadingBehaviorFlag);