Suppress NEW_NAVIGATION_ENTRY prerenderer failures for ORIGIN_OFFLINE

chrome/browser/prerender/prerender_contents.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/prerender/prerender_contents.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <functional>
@@ skipping 567 matching lines @@
     const content::LoadCommittedDetails& details,
     const content::FrameNavigateParams& params) {
   // If the prerender made a second navigation entry, abort the prerender. This
   // avoids having to correctly implement a complex history merging case (this
   // interacts with location.replace) and correctly synchronize with the
   // renderer. The final status may be monitored to see we need to revisit this
   // decision. This does not affect client redirects as those do not push new
   // history entries. (Calls to location.replace, navigations before onload, and
   // <meta http-equiv=refresh> with timeouts under 1 second do not create
   // entries in Blink.)
-  if (prerender_contents_->GetController().GetEntryCount() > 1) {
+  if (origin() != ORIGIN_OFFLINE &&
+      prerender_contents_->GetController().GetEntryCount() > 1) {
     Destroy(FINAL_STATUS_NEW_NAVIGATION_ENTRY);
     return;
   }
 
   // Add each redirect as an alias. |params.url| is included in
   // |params.redirects|.
   //
   // TODO(davidben): We do not correctly patch up history for renderer-initated
   // navigations which add history entries. http://crbug.com/305660.
   for (size_t i = 0; i < params.redirects.size(); i++) {

[pasko, 2017/01/24 19:43:38]

sending these IPCs to the renderer is not needed for offline, so it would be
cleaner to:

if (origin() == ORIGIN_OFFLINE)
  return;

[dougarnett, 2017/01/24 21:14:03]

Done.

     if (!AddAliasURL(params.redirects[i]))
       return;
   }
 }
 
 void PrerenderContents::DidGetRedirectForResourceRequest(
     const content::ResourceRedirectDetails& details) {
   // DidGetRedirectForResourceRequest can come for any resource on a page.  If
   // it's a redirect on the top-level resource, the name needs to be remembered
   // for future matching, and if it redirects to an https resource, it needs to
@@ skipping 137 matching lines @@
 void PrerenderContents::AddResourceThrottle(
     const base::WeakPtr<PrerenderResourceThrottle>& throttle) {
   resource_throttles_.push_back(throttle);
 }
 
 void PrerenderContents::AddNetworkBytes(int64_t bytes) {
   network_bytes_ += bytes;
 }
 
 }  // namespace prerender