Fix some unit_tests under PlzNavigate

content/browser/frame_host/render_frame_host_manager.h

Index: content/browser/frame_host/render_frame_host_manager.h
diff --git a/content/browser/frame_host/render_frame_host_manager.h b/content/browser/frame_host/render_frame_host_manager.h
index 0fb1e5dc1c3bf1547ca820793a8eaadd0c3fb06d..41698cc2104c5ce77662a02b6278b22369d368ca 100644
--- a/content/browser/frame_host/render_frame_host_manager.h
+++ b/content/browser/frame_host/render_frame_host_manager.h
@@ -524,6 +524,7 @@ class CONTENT_EXPORT RenderFrameHostManager
  private:
   friend class NavigatorTestWithBrowserSideNavigation;
   friend class RenderFrameHostManagerTest;
+  friend class RenderFrameHostTester;

[clamy, 2016/08/31 18:23:21]

Why is this needed?

[scottmg, 2016/08/31 19:01:46]

This is for RenderFrameHostTester::GetPendingForController in
content/public/test/test_renderer_host.cc, for speculative_render_frame_host_.

[nasko, 2016/08/31 20:26:43]

With your latest patch, GetPendingForController is gone. This can go too, right?

[scottmg, 2016/08/31 21:05:48]

No, unfortunately RenderFrameHostTester::CommitPendingLoad() needs to do the
same thing.

   friend class TestWebContents;
 
   enum class SiteInstanceRelation {