Fix debug_url for latest version of chrome

it appears that devtoolsFrontendUrl returns a fully qualified URL in
the latest version of Chrome, so it should be returned as-is without
prepending anything to it.

Author: Jell

lib/capybara/cuprite/driver.rb

@@ -379,7 +379,11 @@ def dismiss_modal(type, options = {})
       private
 
       def build_remote_debug_url(path:)
-        "http://#{browser.process.host}:#{browser.process.port}#{path}"
+        uri = URI.parse(path)
+        uri.scheme ||= "http"
+        uri.host ||= browser.process.host
+        uri.port ||= browser.process.port
+        uri.to_s
       end
 
       def default_domain

spec/lib/driver_spec.rb

@@ -30,7 +30,7 @@
   end
 
   describe "debug_url" do
-    it "parses the devtools frontend url correctly" do
+    it "parses the devtools frontend url correctly when devtoolsFrontendUrl is relative" do
       driver = described_class.new(nil, { port: 12_345 })
       driver.browser # initialize browser before stubbing Net::HTTP as it also calls it
       uri = instance_double(URI)
@@ -40,6 +40,19 @@
 
       expect(driver.debug_url).to eq("http://127.0.0.1:12345/works")
     end
+
+    it "parses the devtools frontend url correctly when devtoolsFrontendUrl is fully qualified" do
+      driver = described_class.new(nil, { port: 12_346 })
+      driver.browser # initialize browser before stubbing Net::HTTP as it also calls it
+      uri = instance_double(URI)
+
+      allow(driver).to receive(:URI).with("http://127.0.0.1:12346/json").and_return(uri)
+      allow(Net::HTTP).to receive(:get).with(uri).and_return(
+        %([{"devtoolsFrontendUrl":"https://chrome-devtools-frontend.appspot.com/serve_rev?ws=123"}])
+      )
+
+      expect(driver.debug_url).to eq("https://chrome-devtools-frontend.appspot.com/serve_rev?ws=123")
+    end
   end
 
   private