Tests that the title only ever gets replaced for the current page.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 2e6a5f7556dd8378d2b796a9d676fa9c9ce0472c..51c78e331ab2bda495483c0947ab92519bdbd33d 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -3868,6 +3868,11 @@ void WebContentsImpl::UpdateTitle(RenderFrameHost* render_frame_host,
   NavigationEntryImpl* entry = controller_.GetEntryWithPageID(
       rvh->GetSiteInstance(), page_id);
 
+  // Re http://crbug.com/369661, page id is going away. This function should
+  // only ever be called for the current entry. When this is verified, this
+  // function can be greatly simplified.
+  CHECK_EQ(entry, controller_.GetEntryAtOffset(0));

[Charlie Reis, 2015/05/11 20:32:38]

Interesting that this passed.  Would it fail if we used GetLastCommittedEntry,
as in https://codereview.chromium.org/1135853002/?  I'd prefer to use the last
committed entry for this if it works, but we can comment why it must be the
current (pending/transient) if that's needed.

[Avi (use Gerrit), 2015/05/11 20:52:46]

Dunno! Let's find out.

+
   // We can handle title updates when we don't have an entry in
   // UpdateTitleForEntry, but only if the update is from the current RVH.
   // TODO(avi): Change to make decisions based on the RenderFrameHost.