RenderViewImpl::page_id_ isn't initialized for cross-process replacement navigations.

content/renderer/render_view_browsertest.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 <stddef.h>
 #include <stdint.h>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/command_line.h"
@@ skipping 621 matching lines @@
       PageStateToHistoryEntry(base::get<0>(host_nav_params).page_state);
   blink::WebHTTPBody body = entry->root().httpBody();
   blink::WebHTTPBody::Element element;
   bool successful = body.elementAt(0, element);
   EXPECT_TRUE(successful);
   EXPECT_EQ(blink::WebHTTPBody::Element::TypeData, element.type);
   EXPECT_EQ(length, element.data.size());
   EXPECT_EQ(0, memcmp(raw_data, element.data.data(), length));
 }
 
+// Check that page ID will be initialized in case of navigation
+// that replaces current entry.
+TEST_F(RenderViewImplTest, OnBrowserNavigationUpdatePageID) {
+  // An http url will trigger a resource load so cannot be used here.
+  CommonNavigationParams common_params;
+  StartNavigationParams start_params;
+  RequestNavigationParams request_params;
+  common_params.url = GURL("data:text/html,<div>Page</div>");
+  common_params.navigation_type = FrameMsg_Navigate_Type::NORMAL;
+  common_params.transition = ui::PAGE_TRANSITION_TYPED;
+
+  // Set up params to emulate a browser side navigation
+  // that should replace current entry.
+  common_params.should_replace_current_entry = true;
+  request_params.page_id = -1;
+  request_params.nav_entry_id = 1;
+  request_params.current_history_list_length = 1;
+
+  frame()->Navigate(common_params, start_params, request_params);
+  ProcessPendingMessages();
+
+  // Page ID should be initialized.
+  EXPECT_NE(view_page_id(), -1);
+
+  const IPC::Message* frame_navigate_msg =
+      render_thread_->sink().GetUniqueMessageMatching(
+          FrameHostMsg_DidCommitProvisionalLoad::ID);
+  EXPECT_TRUE(frame_navigate_msg);
+
+  FrameHostMsg_DidCommitProvisionalLoad::Param host_nav_params;
+  FrameHostMsg_DidCommitProvisionalLoad::Read(frame_navigate_msg,
+                                              &host_nav_params);
+  EXPECT_TRUE(base::get<0>(host_nav_params).page_state.IsValid());
+
+  const IPC::Message* frame_page_id_msg =
+      render_thread_->sink().GetUniqueMessageMatching(
+          FrameHostMsg_DidAssignPageId::ID);
+  EXPECT_TRUE(frame_page_id_msg);
+
+  FrameHostMsg_DidAssignPageId::Param host_page_id_params;
+  FrameHostMsg_DidAssignPageId::Read(frame_page_id_msg, &host_page_id_params);
+
+  EXPECT_EQ(base::get<0>(host_page_id_params), view_page_id());
+}
+
 #if defined(OS_ANDROID)
 TEST_F(RenderViewImplTest, OnNavigationLoadDataWithBaseURL) {
   CommonNavigationParams common_params;
   common_params.url = GURL("data:text/html,");
   common_params.navigation_type = FrameMsg_Navigate_Type::NORMAL;
   common_params.transition = ui::PAGE_TRANSITION_TYPED;
   common_params.base_url_for_data_url = GURL("about:blank");
   common_params.history_url_for_data_url = GURL("about:blank");
   RequestNavigationParams request_params;
   request_params.data_url_as_string =
@@ skipping 1803 matching lines @@
                               ","
                               "\"functionDeclaration\":\"function foo(){ "
                               "Promise.resolve().then(() => "
                               "console.log(239))}\""
                               "}"
                               "}");
   EXPECT_EQ(1, CountNotifications("Console.messageAdded"));
 }
 
 }  // namespace content