[headless] Ensure devtooled tests call Disable/RemoveObserver.

headless/lib/headless_devtools_client_browsertest.cc

 // Copyright 2016 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 <memory>
 
 #include "base/json/json_reader.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/url_constants.h"
@@ skipping 47 matching lines @@
     std::unique_ptr<page::NavigateParams> params =
         page::NavigateParams::Builder()
             .SetUrl(embedded_test_server()->GetURL("/hello.html").spec())
             .Build();
     devtools_client_->GetPage()->GetExperimental()->AddObserver(this);
     devtools_client_->GetPage()->Enable();
     devtools_client_->GetPage()->Navigate(std::move(params));
   }
 
   void OnLoadEventFired(const page::LoadEventFiredParams& params) override {
+    devtools_client_->GetPage()->Disable();
     devtools_client_->GetPage()->GetExperimental()->RemoveObserver(this);
     FinishAsynchronousTest();
   }
 
   // Check that events with no parameters still get a parameters object.
   void OnFrameResized(const page::FrameResizedParams& params) override {}
 };
 
 HEADLESS_ASYNC_DEVTOOLED_TEST_F(HeadlessDevToolsClientNavigationTest);
 
@@ skipping 89 matching lines @@
 
   void OnResponseReceived(
       const network::ResponseReceivedParams& params) override {
     EXPECT_EQ(200, params.GetResponse()->GetStatus());
     EXPECT_EQ("OK", params.GetResponse()->GetStatusText());
     std::string content_type;
     EXPECT_TRUE(params.GetResponse()->GetHeaders()->GetString("Content-Type",
                                                               &content_type));
     EXPECT_EQ("text/html", content_type);
 
+    devtools_client_->GetNetwork()->Disable();
     devtools_client_->GetNetwork()->RemoveObserver(this);
     FinishAsynchronousTest();
   }
 };
 
 HEADLESS_ASYNC_DEVTOOLED_TEST_F(HeadlessDevToolsClientObserverTest);
 
 class HeadlessDevToolsClientExperimentalTest
     : public HeadlessAsyncDevTooledBrowserTest,
       page::ExperimentalObserver {
@@ skipping 14 matching lines @@
         runtime::RunIfWaitingForDebuggerParams::Builder().Build());
 
     devtools_client_->GetPage()->GetExperimental()->AddObserver(this);
     devtools_client_->GetPage()->Enable();
     devtools_client_->GetPage()->Navigate(
         embedded_test_server()->GetURL("/hello.html").spec());
   }
 
   void OnFrameStoppedLoading(
       const page::FrameStoppedLoadingParams& params) override {
+    devtools_client_->GetPage()->Disable();
+    devtools_client_->GetPage()->GetExperimental()->RemoveObserver(this);
+
     // Check that a non-experimental command which has no return value can be
     // called with a void() callback.
     devtools_client_->GetPage()->Reload(
         page::ReloadParams::Builder().Build(),
         base::Bind(&HeadlessDevToolsClientExperimentalTest::OnReloadStarted,
                    base::Unretained(this)));
   }
 
   void OnReloadStarted() { FinishAsynchronousTest(); }
 };
@@ skipping 557 matching lines @@
 
   void OnQuerySelector(std::unique_ptr<dom::QuerySelectorResult> result) {
     EXPECT_EQ(nullptr, result);
     FinishAsynchronousTest();
   }
 };
 
 HEADLESS_ASYNC_DEVTOOLED_TEST_F(HeadlessDevToolsMethodCallErrorTest);
 
 }  // namespace headless