-
Notifications
You must be signed in to change notification settings - Fork 27.6k
Firefox 86 breaks Defered Bootstrap (window.name empty) #17117
Comments
A temporary workaround for Protractor with FirefoxDriver appears to be: capabilities: {
browserName: 'firefox',
'moz:firefoxOptions': {
+ prefs: {'privacy.window.name.update.enabled': false}
}
}, Mozilla bug report for enabling this feature: https://bugzilla.mozilla.org/show_bug.cgi?id=1685089 Note that it says: |
Thx for reporting this, @zeha. We need to do some investigation to determine what can/should be done about this (esp. given the current LTS status of the project). In the meantime, I very briefly looked at the Protractor source code and wanted to capture my findings (for future reference). Currently, protractor adds this.executeScriptWithDescription(
'window.name = "' + DEFER_LABEL + '" + window.name;' +
'window.location.replace("' + destination + '");', I experimented with moving updating the window name after the browser has navigated to the new URL (as shown below) and (while it worked fine for most tests) it did cause some test (from the Modified protractor/lib/browser.ts: .then(() => {
- // Set defer label and navigate
+ // Navigate
return this.executeScriptWithDescription(
- 'window.name = "' + DEFER_LABEL + '" + window.name;' +
- 'window.location.replace("' + destination + '");',
+ 'window.location.replace("' + destination + '");',
msg('reset url'));
})
.then(() => {
// We need to make sure the new url has loaded before
// we try to execute any asynchronous scripts.
return this.driver.wait(() => {
return this.executeScriptWithDescription('return window.location.href;', msg('get url'))
.then(
(url: any) => {
return url !== this.resetUrl;
},
(err: IError) => {
if (err.code == 13 || err.name === 'JavascriptError') {
// Ignore the error, and continue trying. This is
// because IE driver sometimes (~1%) will throw an
// unknown error from this execution. See
// https://github.com/angular/protractor/issues/841
// This shouldn't mask errors because it will fail
// with the timeout anyway.
return false;
} else {
throw err;
}
});
}, timeout, 'waiting for page to load for ' + timeout + 'ms');
})
+ .then(() => {
+ // Set defer label
+ return this.executeScriptWithDescription(
+ 'window.name = "' + DEFER_LABEL + '" + window.name;',
+ msg('set defer label'));
+ }) |
We discussed this a bit more yesterday and there doesn't seem to be a great way to fix this (i.e. one that is straight-forward and does not involve the risk of breaking other usecases) 🤔 For now, using the work-around mentioned in #17117 (comment) should be sufficient. We might re-visit this if something changes in the future (for example, if the work-around stops being available). I'll keep the issue open for tracking purposes. |
I've filed https://bugzilla.mozilla.org/show_bug.cgi?id=1700931 for Firefox (Marionette). |
I wanted to find a Protractor issue for this, since it's more of an issue there than with AngularJS itself. Related issues
|
From the Firefox bug tracker:
So it looks like that work-around may only work for another couple of months. |
Yeah, I don't know exactly when we'll remove the preference and we at Firefox are happy to collaborate on making that as painless as possible for everyone involved (please chime in at https://bugzilla.mozilla.org/show_bug.cgi?id=1704469), but this |
@annevk Thanks for the update.
EDIT: I found Chromium's Intent to Ship in a separate issue: https://groups.google.com/a/chromium.org/g/blink-dev/c/86VeIi5sZzc/m/fPOAbuRHCAAJ |
Chromium is getting closer to shipping this in a beta: Do we have any plans for working around this? |
Special thanks to: angular/angular.js#17117
Just a quick not that the current workaround for this issue is to disable the new privacy firefox feature via webdriver configuration:
An alternative (and likely worse) option is to keep on using older version of Firefox in these e2e tests. We currently can't think of a permanent solution to this problem that wouldn't require modification of the test suite and/or the AngularJS application bootstrap code. |
I'm submitting a ...
Current behavior:
Firefox 86 resets window.name to an empty string. This breaks the Deferred Bootstrap(*) feature used by Protractor.
(*): https://docs.angularjs.org/guide/bootstrap#deferred-bootstrap
I've filed this here, because even if Protractor could change something, some other way of triggering Deferred Bootstrap is probably needed.
Expected / new behavior:
Test runners like Protractor can successfully use Deferred Bootstrap with Firefox.
Minimal reproduction of the problem with instructions:
Should happen with any minimal project and a minimal protractor setup.
AngularJS version: 1.8.2
Browser: Firefox 86
Anything else:
Protractor reports:
Firefox is the one browser thats relatively convenient to run in CI. Would be nice if that could be fixed even during LTS.
The text was updated successfully, but these errors were encountered: