Revert "Keep a copy of page id in RenderViewHost."

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index 5a515c1fae2951ad41186b8cee4b1d68123aaa48..c9c7c8ce8593dbfc8462ed7ab5dd9d573e39ae15 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -7,11 +7,9 @@
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/containers/hash_tables.h"
-#include "base/debug/crash_logging.h"
 #include "base/lazy_instance.h"
 #include "base/metrics/histogram.h"
 #include "base/metrics/user_metrics_action.h"
-#include "base/strings/string_number_conversions.h"
 #include "base/time/time.h"
 #include "content/browser/accessibility/accessibility_mode_helper.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
@@ -571,19 +569,8 @@ void RenderFrameHostImpl::OnDidRedirectProvisionalLoad(
     int32 page_id,
     const GURL& source_url,
     const GURL& target_url) {
-  if (render_view_host_->page_id_ != page_id) {
-    base::debug::SetCrashKeyValue("url1",
-                                  source_url.possibly_invalid_spec());
-    base::debug::SetCrashKeyValue("url2",
-                                  target_url.possibly_invalid_spec());
-    base::debug::SetCrashKeyValue(
-        "id1", base::IntToString(render_view_host_->page_id_));
-    base::debug::SetCrashKeyValue("id2",
-                                  base::IntToString(page_id));
-    CHECK(false);
-  }
   frame_tree_node_->navigator()->DidRedirectProvisionalLoad(
-      this, render_view_host_->page_id_, source_url, target_url);
+      this, page_id, source_url, target_url);
 }
 
 // Called when the renderer navigates.  For every frame loaded, we'll get this
@@ -605,14 +592,6 @@ void RenderFrameHostImpl::OnNavigate(const IPC::Message& msg) {
   TRACE_EVENT1("navigation", "RenderFrameHostImpl::OnNavigate",
                "url", validated_params.url.possibly_invalid_spec());
 
-  // Update the RVH's current page ID so that future IPCs from the renderer
-  // correspond to the new page.
-  render_view_host_->page_id_ = validated_params.page_id;
-
-  // Update the RVH's current page ID so that future IPCs from the renderer
-  // correspond to the new page.
-  render_view_host_->page_id_ = validated_params.page_id;
-
   // If we're waiting for a cross-site beforeunload ack from this renderer and
   // we receive a Navigate message from the main frame, then the renderer was
   // navigating already and sent it before hearing the FrameMsg_Stop message.
@@ -946,15 +925,6 @@ void RenderFrameHostImpl::OnUpdateTitle(
     int32 page_id,
     const base::string16& title,
     blink::WebTextDirection title_direction) {
-  if (render_view_host_->page_id_ != page_id) {
-    base::debug::SetCrashKeyValue(
-        "url1", GetLastCommittedURL().possibly_invalid_spec());
-    base::debug::SetCrashKeyValue(
-        "id1", base::IntToString(render_view_host_->page_id_));
-    base::debug::SetCrashKeyValue("id2",
-                                  base::IntToString(page_id));
-    CHECK(false);
-  }
   // This message is only sent for top-level frames. TODO(avi): when frame tree
   // mirroring works correctly, add a check here to enforce it.
   if (title.length() > kMaxTitleChars) {
@@ -962,7 +932,7 @@ void RenderFrameHostImpl::OnUpdateTitle(
     return;
   }
 
-  delegate_->UpdateTitle(this, render_view_host_->page_id_, title,
+  delegate_->UpdateTitle(this, page_id, title,
                          WebTextDirectionToChromeTextDirection(
                              title_direction));
 }