Remove the use of page id from building the commit params.

content/browser/web_contents/web_contents_impl_unittest.cc

Index: content/browser/web_contents/web_contents_impl_unittest.cc
diff --git a/content/browser/web_contents/web_contents_impl_unittest.cc b/content/browser/web_contents/web_contents_impl_unittest.cc
index e82a5daa73c58c0851e596c68a15c272e2729347..ded5a4be207d4f581cabbe833f670c4554cfbc5c 100644
--- a/content/browser/web_contents/web_contents_impl_unittest.cc
+++ b/content/browser/web_contents/web_contents_impl_unittest.cc
@@ -2909,7 +2909,7 @@ TEST_F(WebContentsImplTest, StartStopEventsBalance) {
     // Commit the navigation in the child frame and send the DidStopLoading
     // message.
     contents()->TestDidNavigate(
-        subframe, 3, bar_url, ui::PAGE_TRANSITION_TYPED);
+        subframe, 3, bar_url, ui::PAGE_TRANSITION_MANUAL_SUBFRAME);

[Avi (use Gerrit), 2015/02/03 17:39:39]

This seemed to work before because the code in the NavigatorImpl was overriding
this value.

AFAIK, subframes can only ever have the transition types manual subframe and
auto subframe. Charlie, is that true? It certainly seems to be enforced by the
code.

[nasko, 2015/02/03 17:45:48]

Totally makes sense. More of a copy/paste error on my side than intentional
value.

[Charlie Reis, 2015/02/03 17:46:56]

Great.  LGTM.

     subframe->OnMessageReceived(
         FrameHostMsg_DidStopLoading(subframe->GetRoutingID()));
   }