Remove old session history logic.

content/test/layouttest_support.cc

Index: content/test/layouttest_support.cc
diff --git a/content/test/layouttest_support.cc b/content/test/layouttest_support.cc
index 3f981ddc9c333d4f23803857c9ab4321b90d9e6f..7fbc8f546f2bf7f81193d63e26f32abd3c8af7ba 100644
--- a/content/test/layouttest_support.cc
+++ b/content/test/layouttest_support.cc
@@ -24,7 +24,6 @@
 #include "content/browser/renderer_host/render_process_host_impl.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "content/common/renderer.mojom.h"
-#include "content/common/site_isolation_policy.h"
 #include "content/public/common/page_state.h"
 #include "content/public/common/screen_info.h"
 #include "content/public/renderer/renderer_gamepad_provider.h"
@@ -459,9 +458,6 @@ int GetLocalSessionHistoryLength(RenderView* render_view) {
 void SyncNavigationState(RenderView* render_view) {
   // TODO(creis): Add support for testing in OOPIF-enabled modes.
   // See https://crbug.com/477150.

[Charlie Reis, 2017/01/24 17:34:24]

There's lots of layout tests that use dumpBackForwardList and still pass (e.g.,
back-during-beforeunload.html, history-back-initial-vs-final-url.html, etc), so
I'm not sure if we need this code at all.  It hasn't been doing anything since
August, anyway, so I'll try removing it in a followup.

[nasko, 2017/01/26 00:56:20]

sgtm

-  if (SiteIsolationPolicy::UseSubframeNavigationEntries())
-    return;
-  static_cast<RenderViewImpl*>(render_view)->SendUpdateState();
 }
 
 void SetFocusAndActivate(RenderView* render_view, bool enable) {