PlzNavigate: Make ServiceWorker work with PlzNavigate.

content/test/test_render_frame_host.cc

 // Copyright 2013 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 "content/test/test_render_frame_host.h"
 
 #include "base/command_line.h"
 #include "content/browser/frame_host/frame_tree.h"
 #include "content/browser/frame_host/navigation_handle_impl.h"
 #include "content/browser/frame_host/navigation_request.h"
@@ skipping 310 matching lines @@
 
 void TestRenderFrameHost::SendRendererInitiatedNavigationRequest(
     const GURL& url,
     bool has_user_gesture) {
   // Since this is renderer-initiated navigation, the RenderFrame must be
   // initialized. Do it if it hasn't happened yet.
   InitializeRenderFrameIfNeeded();
 
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kEnableBrowserSideNavigation)) {
-    BeginNavigationParams begin_params("GET", std::string(), net::LOAD_NORMAL,
-                                       has_user_gesture);
+    BeginNavigationParams begin_params(
+        "GET", std::string(), net::LOAD_NORMAL, has_user_gesture, false,
+        REQUEST_CONTEXT_TYPE_LOCATION);

[clamy, 2015/08/31 12:09:27]

Regarding the type, 3 lines below we set the transition type to
PAGE_TRANSITION_LINK. I think it would be more coherent to have the type as
REQUEST_CONTEXT_TYPE_HYPERLINK. (LOCATION would mean that it is a client-side
redirect which is treated differently in the PageTransitions).

[Fabrice (no longer in Chrome), 2015/08/31 14:00:35]

Done.

     CommonNavigationParams common_params;
     common_params.url = url;
     common_params.referrer = Referrer(GURL(), blink::WebReferrerPolicyDefault);
     common_params.transition = ui::PAGE_TRANSITION_LINK;
     OnBeginNavigation(common_params, begin_params,
                       scoped_refptr<ResourceRequestBody>());
   }
 }
 
 void TestRenderFrameHost::DidChangeOpener(int opener_routing_id) {
@@ skipping 49 matching lines @@
   // cleared during DidFailProvisionalLoad).
   int page_id = entry ? entry->GetPageID() : -1;
   if (page_id == -1) {
     WebContentsImpl* web_contents = static_cast<WebContentsImpl*>(delegate());
     page_id = web_contents->GetMaxPageIDForSiteInstance(GetSiteInstance()) + 1;
   }
   return page_id;
 }
 
 }  // namespace content