You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 29, 2018. It is now read-only.
If a script performs calls the SafariDriver get method on a URI which contains a fragment
and is identical to the current URI (including fragment) then subsequent commands hang
since the driver is expecting the page to reload which doesn't occur.
What steps will reproduce the problem?
1. Run the attached script - the script hangs when trying to reload a URI which is
identical including the fragment (third get call in this example).
Expected: The script should complete without issue
Selenium version: 2.41.0
OS: OSX 10.9.2
Browser: Safari
Browser version: 7.0.2
I dug a little into this and the cause appears to be that safaridriver.extension.Tab.prototype.loadsNewPage
in safari-driver/extension/tab.js returns true in this example indicating the page
will be reloading which doesn't occur. I am fairly sure that the browser is correct
in not reloading the URI here (it shouldn't on fragment change) but not 100% sure whether
an loading an identical fragment is considered different.
Locally I have changed the loadsNewPage function and removed the check on goog.Uri(url).toString()
since although the comments suggest otherwise the toString() function seems to include
the fragment so may also cause issues if fragments differ (since this would evaluate
to false and the subsequent check would evaluate to true indicating a page reload)
and also resolves to true in this example. I have replaced this with some more detailed
checks on the various URI components which resolves the observed issue.
If my above thinking is correct I can submit my change as a patch.
Reported by stevejeffers on 2014-04-04 10:11:05
The text was updated successfully, but these errors were encountered:
Originally reported on Google Code with ID 7176
Reported by
stevejeffers
on 2014-04-04 10:11:05The text was updated successfully, but these errors were encountered: