Copy max_restored_page_id_ when needed.

content/test/test_web_contents.cc

Index: content/test/test_web_contents.cc
diff --git a/content/test/test_web_contents.cc b/content/test/test_web_contents.cc
index d023bd4ade8718e8f7b780679b4ee771bd1f9158..b1bb7e99c92d62a05b0187e9ec5fb0ee621d0acb 100644
--- a/content/test/test_web_contents.cc
+++ b/content/test/test_web_contents.cc
@@ -100,6 +100,7 @@ bool TestWebContents::CreateRenderViewForRenderManager(
     RenderViewHost* render_view_host,
     int opener_route_id,
     CrossProcessFrameConnector* frame_connector) {
+  UpdateMaxPageIDIfNecessary(render_view_host);

[Charlie Reis, 2014/04/24 18:02:02]

Looks like we override this method to cut out the view-related parts.  Would it
make sense to include the GetMaxPageIDForSiteInstance call as well, and pass the
resulting max_page_id to CreateRenderView below?

[Alexander Semashko, 2014/04/24 21:40:37]

Currently TestRenderViewHost::CreateRenderView does not use max_page_id, so
this change alone won't have any effect. It must be accompanied by changes to
other test classes, but I don't know what exactly should be done here and what
are the possible uses of this.

[Charlie Reis, 2014/04/24 21:48:05]

Ok, let's leave it as is until a test needs it.

   // This will go to a TestRenderViewHost.
   static_cast<RenderViewHostImpl*>(
       render_view_host)->CreateRenderView(base::string16(),