PlzNavigate: make content unit tests work with browser side navigation
This CL fixes several WebContentsImplTests and RenderFrameHostManagerTests so
that they work with PlzNavigate enabled by making use of the helper functions
introduced in https://chromiumcodereview.appspot.com/761013003/.
BUG=439423
Review URL: https://codereview.chromium.org/789643005
Cr-Commit-Position: refs/heads/master@{#312435}
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 64c7bcd..672f0bd 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -4032,7 +4032,7 @@
if (DevToolsAgentHost::IsDebuggerAttached(this))
return;
- if (rfhi->is_waiting_for_beforeunload_ack() ||
+ if (rfhi->IsWaitingForBeforeUnloadACK() ||
rfhi->IsWaitingForUnloadACK()) {
// Hang occurred while firing the beforeunload/unload handler.
// Pretend the handler fired so tab closing continues as if it had.
@@ -4048,7 +4048,7 @@
// close. Otherwise, pretend the unload listeners have all fired and close
// the tab.
bool close = true;
- if (rfhi->is_waiting_for_beforeunload_ack() && delegate_) {
+ if (rfhi->IsWaitingForBeforeUnloadACK() && delegate_) {
delegate_->BeforeUnloadFired(this, true, &close);
}
if (close)
