Add |needs_reload| argument to NavigationController::CopyStateFrom().
This argument controls whether the WebContents needs to be reloaded
when activated. Not reloading the WebContents as soon as it is
activated will allow TabManager to implement more advanced reload
rules (e.g. browser window is active, tab has been active for x ms).
Note: Today, a WebContents is considered "active" if it is in a
selected tab, even if the browser window is not visible.
[email protected],[email protected],[email protected]
Bug: 731145
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
Change-Id: I87a16df112752b97bcc369e34a1e1f86d18d9b99
Reviewed-on: https://chromium-review.googlesource.com/531065
Commit-Queue: Francois Doray <[email protected]>
Reviewed-by: Charlie Reis <[email protected]>
Cr-Commit-Position: refs/heads/master@{#481778}diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 1d68687..52ddbab 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1615,7 +1615,7 @@
create_params.initial_size = GetContainerBounds().size();
WebContentsImpl* tc =
CreateWithOpener(create_params, frame_tree_.root()->opener());
- tc->GetController().CopyStateFrom(controller_);
+ tc->GetController().CopyStateFrom(controller_, true);
for (auto& observer : observers_)
observer.DidCloneToNewWebContents(this, tc);
return tc;
