Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

Commit

Permalink
fix(navigation): fix using browser.get with safari driver
Browse files Browse the repository at this point in the history
SafariDriver fails with data urls - see #1049. Reverting to use about:blank
for now.
  • Loading branch information
juliemr committed Jul 8, 2014
1 parent 290f572 commit ab1d0be
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion lib/runner.js
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,9 @@ Runner.prototype.setupGlobals_ = function(driver) {
driver.getCapabilities().then(function(caps) {
// Internet Explorer does not accept data URLs, which are the default
// reset URL for Protractor.
if (caps.get('browserName') === 'internet explorer') {
// Safari accepts data urls, but SafariDriver fails after one is used.
var browserName = caps.get('browserName');
if (browserName === 'internet explorer' || browserName === 'safari') {
browser.resetUrl = 'about:blank';
}
});
Expand Down

0 comments on commit ab1d0be

Please sign in to comment.