Description

Different drivers currently have different behaviors. This brings everything into agreement.

Motivation and Context

Addresses #11643
This is also in accordance with what we discussed in #11410 as well as:

• Remove / Deprecate undesired behavior
• Improve logging ability

Specifics:

1. Deprecates the constructors that do not include the timeout (users should be using builder anyway; IE has has even kept the constructor private for this reason)
2. Deprecates the special Firefox behaviors that came from Legacy driver (FirefoxBinary related)
3. Adds support for turning on Safari driver logging, including preventing users from calling superclass method to set log file location (which isn't supported)
4. Adds missing features to GeckoDriver service
   • log level
   • log truncation
   • setting alternate profile directory (required for snap)
5. Allows Chrome, Edge & IE to send all output from the process to the location of user's choosing.

The last item is the biggest one. By default geckodriver logs everything at an INFO level to stderr. This is annoying, so it was allowed to use the System Property webdriver.firefox.logLevel to set the "/dev/null" for it to discard the output. But only being able to do this by using a system property is bad, so I added methods and support for it in all browsers.

For comparison, Ruby has always sent everything from driver process to null output as a default.

This PR allows either of these:

    ChromeDriverService service = new ChromeDriverService.Builder()
        .withLogOutput(DriverService.LOG_NULL)
        .build();

or

    System.setProperty(ChromeDriverService.CHROME_DRIVER_LOG_PROPERTY,
        DriverService.LOG_NULL);