Deprecated the URL alteration in visit()

This was introduced to strip the base url from the url when using the
HttpKernel client in early versions of the driver. However, this was only
half-working as it was impacting the current url later. A proper way to
support the base url has been introduced for the HttpKernel client, so the
old one can be deprecated.

Author: stof

src/Behat/Mink/Driver/BrowserKitDriver.php

@@ -61,6 +61,7 @@ public function __construct(Client $client, $baseUrl = null)
 
         if ($baseUrl !== null && $client instanceof HttpKernelClient) {
             $client->setServerParameter('SCRIPT_FILENAME', parse_url($baseUrl, PHP_URL_PATH));
+            $this->removeScriptFromUrl = false;
         }
     }
 
@@ -86,19 +87,31 @@ public function setSession(Session $session)
      * Tells driver to remove hostname from URL.
      *
      * @param Boolean $remove
+     *
+     * @deprecated Deprecated as of 1.2, to be removed in 2.0. Pass the base url in the constructor instead.
      */
     public function setRemoveHostFromUrl($remove = true)
     {
+        trigger_error(
+            'setRemoveHostFromUrl() is deprecated as of 1.2 and will be removed in 2.0. Pass the base url in the constructor instead.',
+            E_USER_DEPRECATED
+        );
         $this->removeHostFromUrl = (bool) $remove;
     }
 
     /**
      * Tells driver to remove script name from URL.
      *
      * @param Boolean $remove
+     *
+     * @deprecated Deprecated as of 1.2, to be removed in 2.0. Pass the base url in the constructor instead.
      */
     public function setRemoveScriptFromUrl($remove = true)
     {
+        trigger_error(
+            'setRemoveScriptFromUrl() is deprecated as of 1.2 and will be removed in 2.0. Pass the base url in the constructor instead.',
+            E_USER_DEPRECATED
+        );
         $this->removeScriptFromUrl = (bool) $remove;
     }