Do not reset navigation state when BeforeUnload is cancelled by a commit

content/test/content_browser_test_utils_internal.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/test/content_browser_test_utils_internal.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <map>
@@ skipping 11 matching lines @@
 #include "content/browser/frame_host/frame_tree_node.h"
 #include "content/browser/frame_host/navigator.h"
 #include "content/browser/frame_host/render_frame_proxy_host.h"
 #include "content/browser/frame_host/render_widget_host_view_child_frame.h"
 #include "content/browser/renderer_host/render_widget_host_view_base.h"
 #include "content/public/browser/resource_dispatcher_host.h"
 #include "content/public/browser/resource_throttle.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/content_browser_test_utils.h"
 #include "content/shell/browser/shell.h"
+#include "content/shell/browser/shell_javascript_dialog_manager.h"
 #include "content/test/test_frame_navigation_observer.h"
 #include "net/url_request/url_request.h"
 
 namespace content {
 
+namespace {
+
+// Helper class used by the NavigationDelayer to pause navigations.
+class NavigationDelayerThrottle : public NavigationThrottle {
+ public:
+  NavigationDelayerThrottle(NavigationHandle* handle,
+                            base::Closure on_paused_closure)
+      : NavigationThrottle(handle), on_paused_closure_(on_paused_closure) {}
+  ~NavigationDelayerThrottle() override {}
+
+ private:
+  // NavigationThrottle implementation.
+  NavigationThrottle::ThrottleCheckResult WillStartRequest() override {
+    BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, on_paused_closure_);
+    return NavigationThrottle::DEFER;
+  }
+
+  base::Closure on_paused_closure_;
+};
+
+}  // namespace
+
 void NavigateFrameToURL(FrameTreeNode* node, const GURL& url) {
   TestFrameNavigationObserver observer(node);
   NavigationController::LoadURLParams params(url);
   params.transition_type = ui::PAGE_TRANSITION_LINK;
   params.frame_tree_node_id = node->frame_tree_node_id();
   node->navigator()->GetController()->LoadURLWithParams(params);
   observer.Wait();
 }
 
+void SetProceedByDefaultOnBeforeUnload(Shell* shell, bool proceed) {
+  ShellJavaScriptDialogManager* manager =
+      static_cast<ShellJavaScriptDialogManager*>(
+          shell->GetJavaScriptDialogManager(shell->web_contents()));
+  manager->set_proceed_beforeunload_default(proceed);
+}
+
 FrameTreeVisualizer::FrameTreeVisualizer() {
 }
 
 FrameTreeVisualizer::~FrameTreeVisualizer() {
 }
 
 std::string FrameTreeVisualizer::DepictFrameTree(FrameTreeNode* root) {
   // Tracks the sites actually used in this depiction.
   std::map<std::string, SiteInstance*> legend;
 
@@ skipping 262 matching lines @@
     net::URLRequest* request,
     content::ResourceContext* resource_context,
     content::AppCacheService* appcache_service,
     ResourceType resource_type,
     ScopedVector<content::ResourceThrottle>* throttles) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
   if (request->url() == url_)
     throttles->push_back(new HttpRequestStallThrottle);
 }
 
+NavigationDelayer::NavigationDelayer(WebContents* web_contents, const GURL& url)
+    : WebContentsObserver(web_contents),
+      url_(url),
+      navigation_paused_(false),
+      handle_(nullptr),
+      weak_factory_(this) {}
+
+NavigationDelayer::~NavigationDelayer() {}
+
+void NavigationDelayer::WaitForNavigationPaused() {
+  if (navigation_paused_)
+    return;
+  loop_runner_ = new MessageLoopRunner();

[nasko, 2016/03/29 20:19:06]

Why not create it in the constructor?

[clamy, 2016/03/30 13:31:11]

Because we will use it twice, and you can only use a MessageLoopRunner once
AFAIU.

+  loop_runner_->Run();
+  loop_runner_ = nullptr;
+}
+
+void NavigationDelayer::WaitForNavigationFinished() {
+  if (!handle_)
+    return;
+  loop_runner_ = new MessageLoopRunner();
+  loop_runner_->Run();
+  loop_runner_ = nullptr;
+}
+
+void NavigationDelayer::ResumeNavigation() {
+  if (!navigation_paused_ || !handle_)
+    return;
+  navigation_paused_ = false;
+  handle_->Resume();
+}
+
+void NavigationDelayer::DidStartNavigation(NavigationHandle* handle) {
+  if (handle_ || handle->GetURL() != url_)
+    return;
+
+  handle_ = handle;
+  scoped_ptr<NavigationThrottle> throttle(new NavigationDelayerThrottle(
+      handle_, base::Bind(&NavigationDelayer::OnPausedNavigation,
+                          weak_factory_.GetWeakPtr())));
+  handle_->RegisterThrottleForTesting(std::move(throttle));
+}
+
+void NavigationDelayer::DidFinishNavigation(NavigationHandle* handle) {
+  if (handle != handle_)
+    return;
+  handle_ = nullptr;
+  navigation_paused_ = false;
+  if (loop_runner_)
+    loop_runner_->Quit();
+}
+
+void NavigationDelayer::OnPausedNavigation() {
+  navigation_paused_ = true;
+  if (loop_runner_)
+    loop_runner_->Quit();
+}
+
 }  // namespace content