Bug Fix: Adding the support for timing operations to zope.testbrowser…

… broke the

ability to provide a URL to the Browser constructor, this adds a doctest to
exercise the bug and a fix.

README.txt

@@ -18,6 +18,12 @@ testing of Zope 3 applications, it can be imported from
     >>> from zope.testbrowser import Browser
     >>> browser = Browser()
 
+An ititial page to load can be passed to the ``Browser`` constructor:
+
+    >>> browser = Browser('http://localhost/@@/testbrowser/simple.html')
+    >>> browser.url
+    'http://localhost/@@/testbrowser/simple.html'
+
 The browser can send arbitrary headers; this is helpful for setting the
 "Authorization" header or a language value, so that your tests format values
 the way you expect in your tests, if you rely on zope.i18n locale-based
@@ -26,7 +32,7 @@ formatting or a similar approach.
     >>> browser.addHeader('Authorization', 'Basic mgr:mgrpw')
     >>> browser.addHeader('Accept-Language', 'en-US')
 
-The browser can `open` web pages:
+An existing browser instance can also `open` web pages:
 
     >>> browser.open('http://localhost/@@/testbrowser/simple.html')
     >>> browser.url

browser.py

@@ -146,11 +146,12 @@ def __init__(self, url=None, mech_browser=None):
         if mech_browser is None:
             mech_browser = mechanize.Browser()
         self.mech_browser = mech_browser
-        if url is not None:
-            self.open(url)
         self.timer = PystoneTimer()
         self._enable_setattr_errors = True
 
+        if url is not None:
+            self.open(url)
+
     @property
     def url(self):
         """See zope.testbrowser.interfaces.IBrowser"""