DevTools: Test for disabling PortForwardingController

chrome/browser/devtools/device/port_forwarding_browsertest.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 "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/string_number_conversions.h"
 #include "chrome/browser/devtools/browser_list_tabcontents_provider.h"
 #include "chrome/browser/devtools/device/port_forwarding_controller.h"
@@ skipping 20 matching lines @@
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     InProcessBrowserTest::SetUpCommandLine(command_line);
     command_line->AppendSwitchASCII(switches::kRemoteDebuggingPort,
         base::IntToString(kDefaultDebuggingPort));
   }
 
  protected:
   class Listener : public PortForwardingController::Listener {
    public:
     explicit Listener(Profile* profile)
-        : profile_(profile) {
+        : profile_(profile),
+          skip_empty_devices_(true) {
       PortForwardingController::Factory::GetForProfile(profile_)->
           AddListener(this);
     }
 
     virtual ~Listener() {
       PortForwardingController::Factory::GetForProfile(profile_)->
           RemoveListener(this);
     }
 
     virtual void PortStatusChanged(const DevicesStatus& status) OVERRIDE {
-      if (status.empty())
+      if (status.empty() && skip_empty_devices_)
         return;
       base::MessageLoop::current()->PostTask(
           FROM_HERE, base::MessageLoop::QuitClosure());
     }
 
+    void set_skip_empty_devices(bool skip_empty_devices) {
+      skip_empty_devices_ = skip_empty_devices;
+    }
+
    private:
     Profile* profile_;
+    bool skip_empty_devices_;
   };
 };
 
 IN_PROC_BROWSER_TEST_F(PortForwardingTest,
                        LoadPageWithStyleAnsScript) {
   Profile* profile = browser()->profile();
 
   AndroidDeviceManager::DeviceProviders device_providers;
   device_providers.push_back(new SelfAsDeviceProvider(kDefaultDebuggingPort));
   DevToolsAndroidBridge::Factory::GetForProfile(profile)->
@@ skipping 38 matching lines @@
           "window.domAutomationController.send(getBodyTextContent())",
           &result));
   ASSERT_EQ("content", result) << "Javascript has not loaded.";
 
   ASSERT_TRUE(
       content::ExecuteScriptAndExtractString(
           rvh,
           "window.domAutomationController.send(getBodyMarginLeft())",
           &result));
   ASSERT_EQ("100px", result) << "CSS has not loaded.";
+
+  // Test that disabling port forwarding is handled normally.
+  wait_for_port_forwarding.set_skip_empty_devices(false);
+  prefs->SetBoolean(prefs::kDevToolsPortForwardingEnabled, false);
+  content::RunMessageLoop();
 }