Bug 1369827 - Update geckodriver docs for proxy port changes. r=ato

MozReview-Commit-ID: 4yBC9AAHU5X

--HG--
extra : rebase_source : b13451573091fb731ee31b928a3d79a5b7361d0f

testing/geckodriver/CHANGES.md

@@ -16,9 +16,12 @@ All notable changes to this program is documented in this file.
 [`SetWindowRect`]: https://docs.rs/webdriver/0.29.0/webdriver/command/enum.WebDriverCommand.html#variant.SetWindowRect
 
 ### Changed
+- Removed deprecated `socksProxyVersion` in favor of `socksVersion`.
+- Removed `ftpProxyPort`, `httpProxyPort`, `sslProxyPort`, and `socksProxyPort` because _ports_ have to be set for `ftpProxy`, `httpProxy`, `sslProxy`, and `socksProxy` using ":<PORT>".
 - To make sure no browser process is left behind when the [`New Session` command](https://docs.rs/webdriver/0.27.0/webdriver/command/enum.WebDriverCommand.html#variant.NewSession) fails, the process is closed immediately now.
 - The `proxyType` `noproxy` has been replaced with `direct` in accordance with recent WebDriver specification changes
 - `/moz/addon/install` command accepts an `addon` parameter, in lieu of `path`, containing an addon as a Base64 string
+- [webdriver crate](https://crates.io/crates/webdriver) upgraded to version 0.30.0
 - [mozrunner crate](https://crates.io/crates/mozrunner) upgraded to version 0.4.2
 
 ## 0.18.0 (2017-07-10)

testing/geckodriver/README.md

@@ -137,74 +137,40 @@ geckodriver supports a number of
  <tr>
   <td><code>ftpProxy</code>
   <td>string
-  <td>Defines the proxy hostname for FTP traffic.
-   Should only be set then the <code>proxyType</code>
+  <td>Defines the proxy hostname with an optional port for FTP traffic.
+   This property should only be set when <code>proxyType</code>
    is set to <code>manual</code>.
  </tr>
 
- <tr>
-  <td><code>ftpProxyPort</code>
-  <td>number
-  <td>Defines the proxy port for FTP traffic.
-   This property should only be set
-   when <code>proxyType</code> is <code>manual</code>.
- </tr>
-
  <tr>
   <td><code>httpProxy</code>
   <td>string
-  <td>Defines the hostname for HTTP traffic.
-   This property should only be set
-   when <code>proxyType</code> is <code>manual</code>.
- </tr>
-
- <tr>
-  <td><code>httpProxyPort</code>
-  <td>number
-  <td>Defines the proxy port for HTTP traffic.
-   This property should only be set
-   when <code>proxyType</code> is <code>manual</code>.
+  <td>Defines the proxy hostname with an optional port for HTTP traffic.
+   This property should only be set when <code>proxyType</code>
+   is set to <code>manual</code>.
  </tr>
 
  <tr>
   <td><code>sslProxy</code>
   <td>string
-  <td>Defines the proxy hostname
-   for encrypted TLS traffic.
-   This property should only be set
-   when <code>proxyType</code> is <code>manual</code>.
- </tr>
-
- <tr>
-  <td><code>sslProxyPort</code>
-  <td>number
-  <td>Defines the proxy port for SSL traffic.
-   This property should only be set
-   when <code>proxyType</code> is <code>manual</code>.
+  <td>Defines the proxy hostname with an optional port for encrypted TLS traffic.
+   This property should only be set when <code>proxyType</code>
+   is set to <code>manual</code>.
  </tr>
 
  <tr>
   <td><code>socksProxy</code>
   <td>string
-  <td>Defines the proxy hostname for a SOCKS proxy.
-   This property should only be set
-   when <code>proxyType</code> is <code>manual</code>.
- </tr>
-
- <tr>
-  <td><code>socksProxyPort</code>
-  <td>number
-  <td>Defines the proxy port for a SOCKS proxy.
-   This property should only be set
-   when <code>proxyType</code> is <code>manual</code>.
+  <td>Defines the hostname with on optional port for a SOCKS proxy.
+   This property should only be set when <code>proxyType</code>
+   is set to <code>manual</code>.
  </tr>
 
  <tr>
   <td><code>socksVersion</code>
   <td>number
-  <td>Defines the SOCKS proxy version.
-   This property should only be set
-   when <code>proxyType</code> is <code>manual</code>.
+  <td>Defines the SOCKS proxy version. This property has only to be set
+   when <code>proxyType</code> is set to <code>manual</code>.
  </tr>
 
  <tr>