feat: remove headless new (#518)

Author: route

CHANGELOG.md

@@ -13,6 +13,7 @@
 
 - `Ferrum::Network::Exchange#finished?` takes into account that exchange might be in an unknown state after loader changed(next `go_to` call) and considers
   such as finished [#426]
+- `headless: "new"` mode is removed as it's now default in modern Chrome, just use `headless: true` instead [#518]
 
 ### Fixed
 

README.md

@@ -150,8 +150,7 @@ Ferrum::Browser.new(options)
 ```
 
 * options `Hash`
-  * `:headless` (String | Boolean) - Set browser as headless or not, `true` by default. You can set `"new"` to support
-      [new headless mode](https://developer.chrome.com/articles/new-headless/).
+  * `:headless` (String | Boolean) - Set browser as headless or not, `true` by default.
   * `:xvfb` (Boolean) - Run browser in a virtual framebuffer, `false` by default.
   * `:flatten` (Boolean) - Use one websocket connection to the browser and all the pages in flatten mode.
   * `:window_size` (Array) - The dimensions of the browser window in which to

lib/ferrum/browser.rb

@@ -245,10 +245,6 @@ def version
       VersionInfo.new(command("Browser.getVersion"))
     end
 
-    def headless_new?
-      process&.command&.headless_new?
-    end
-
     private
 
     def start

lib/ferrum/browser/command.rb

@@ -39,10 +39,6 @@ def xvfb?
         !!options.xvfb
       end
 
-      def headless_new?
-        @flags["headless"] == "new"
-      end
-
       def to_a
         [path] + @flags.map { |k, v| v.nil? ? "--#{k}" : "--#{k}=#{v}" }
       end

lib/ferrum/browser/options/chrome.rb

@@ -76,13 +76,7 @@ def merge_required(flags, options, user_data_dir)
         end
 
         def merge_default(flags, options)
-          defaults = case options.headless
-                     when false
-                       except("headless", "disable-gpu")
-                     when "new"
-                       except("headless").merge("headless" => "new")
-                     end
-
+          defaults = except("headless", "disable-gpu") if options.headless == false
           defaults ||= DEFAULT_OPTIONS
           # On Windows, the --disable-gpu flag is a temporary work around for a few bugs.
           # See https://bugs.chromium.org/p/chromium/issues/detail?id=737678 for more information.

spec/network_spec.rb

@@ -517,6 +517,6 @@
       %r{Request to http://.*/with_js failed \(net::ERR_INTERNET_DISCONNECTED\)}
     )
 
-    expect(page.at_css("body").text).to match("No internet") if browser.headless_new?
+    expect(page.at_css("body").text).to match("No internet")
   end
 end

spec/page_spec.rb

@@ -55,7 +55,7 @@
 
   describe "#position=" do
     it "allows the window to be positioned" do
-      skip if Ferrum::Utils::Platform.mac? && !browser.headless_new?
+      skip if Ferrum::Utils::Platform.mac?
 
       expect do
         page.position = { left: 10, top: 20 }

spec/spec_helper.rb

@@ -33,7 +33,6 @@
   config.before(:all) do
     base_url = Ferrum::Server.server.base_url
     options = { base_url: base_url }
-    options.merge!(headless: "new") if ENV["HEADLESS"] == "new"
     options.merge!(headless: false) if ENV["HEADLESS"] == "false"
     options.merge!(slowmo: ENV["SLOWMO"].to_f) if ENV["SLOWMO"].to_f > 0
 

spec/support/application.rb

@@ -49,7 +49,7 @@ def authorized?(login, password)
 
     get "/" do
       response.set_cookie("cookie", value: "root cookie", domain: request.host, path: request.path)
-      %(Hello world! <a href="with_html">Relative</a>)
+      %(<link rel="icon" href="data:,">Hello world! <a href="with_html">Relative</a>)
     end
 
     get "/set_cookie" do