Revert of [DevTools] Unpause renderer when paused RenderFrame is closed.

content/renderer/render_view_browsertest.cc

 // Copyright (c) 2012 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 "base/basictypes.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/memory/shared_memory.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 13 matching lines @@
 #include "content/public/common/url_constants.h"
 #include "content/public/common/url_utils.h"
 #include "content/public/renderer/content_renderer_client.h"
 #include "content/public/renderer/document_state.h"
 #include "content/public/renderer/navigation_state.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/frame_load_waiter.h"
 #include "content/public/test/render_view_test.h"
 #include "content/public/test/test_utils.h"
 #include "content/renderer/accessibility/renderer_accessibility.h"
-#include "content/renderer/devtools/devtools_agent.h"
 #include "content/renderer/history_controller.h"
 #include "content/renderer/history_serialization.h"
 #include "content/renderer/navigation_state_impl.h"
 #include "content/renderer/render_process.h"
 #include "content/renderer/render_view_impl.h"
 #include "content/shell/browser/shell.h"
 #include "content/shell/browser/shell_browser_context.h"
 #include "content/test/mock_keyboard.h"
 #include "net/base/net_errors.h"
 #include "net/cert/cert_status_flags.h"
@@ skipping 237 matching lines @@
   }
 
   void SetZoomLevel(double level) {
     view()->OnSetZoomLevelForView(false, level);
   }
 
  private:
   scoped_ptr<MockKeyboard> mock_keyboard_;
 };
 
-class DevToolsAgentTest : public RenderViewImplTest {
- public:
-  void Attach() {
-    std::string host_id = "host_id";
-    agent()->OnAttach(host_id);
-  }
-
-  void Detach() {
-    agent()->OnDetach();
-  }
-
-  bool IsPaused() {
-    return agent()->paused_;
-  }
-
-  void DispatchDevToolsMessage(const std::string& message) {
-    agent()->OnDispatchOnInspectorBackend(message);
-  }
-
-  void CloseWhilePaused() {
-    EXPECT_TRUE(IsPaused());
-    view()->NotifyOnClose();
-  }
-
- private:
-  DevToolsAgent* agent() {
-    return frame()->devtools_agent();
-  }
-};
-
 // Test for https://crbug.com/461191.
 TEST_F(RenderViewImplTest, RenderFrameMessageAfterDetach) {
   // Create a new main frame RenderFrame so that we don't interfere with the
   // shutdown of frame() in RenderViewTest.TearDown.
   blink::WebURLRequest popup_request(GURL("http://foo.com"));
   blink::WebView* new_web_view = view()->createView(
       GetMainFrame(), popup_request, blink::WebWindowFeatures(), "foo",
       blink::WebNavigationPolicyNewForegroundTab, false);
   RenderViewImpl* new_view = RenderViewImpl::FromWebView(new_web_view);
   RenderFrameImpl* new_frame =
@@ skipping 2032 matching lines @@
   request_params.page_id = -1;
   frame()->OnNavigate(CommonNavigationParams(), StartNavigationParams(),
                       request_params);
 
   // The history list in RenderView should have been updated.
   EXPECT_EQ(1, view()->historyBackListCount());
   EXPECT_EQ(2, view()->historyBackListCount() +
       view()->historyForwardListCount() + 1);
 }
 
-TEST_F(DevToolsAgentTest, DevToolsResumeOnClose) {
-  Attach();
-  EXPECT_FALSE(IsPaused());
-  DispatchDevToolsMessage("{\"id\":1,\"method\":\"Debugger.enable\"}");
-
-  // Executing javascript will pause the thread and create nested message loop.
-  // Posting task simulates message coming from browser.
-  base::MessageLoop::current()->PostTask(FROM_HERE, base::Bind(
-      &DevToolsAgentTest::CloseWhilePaused, base::Unretained(this)));
-  ExecuteJavaScript("debugger;");
-
-  // CloseWhilePaused should resume execution and continue here.
-  EXPECT_FALSE(IsPaused());
-  Detach();
-}
-
 }  // namespace content