Closed
Bug 1985060
Opened 9 months ago
Closed 9 months ago
Bring back navigate-push-javascript-url.html
Categories
(Core :: DOM: Navigation, task)
Core
DOM: Navigation
Tracking
()
RESOLVED
INVALID
People
(Reporter: tschuster, Assigned: tschuster)
References
Details
Attachments
(1 obsolete file)
There are some concerns about test coverage / interop with the new combined navigate-javascript-url.html test. Bring back the old file.
| Assignee | ||
Comment 1•9 months ago
|
||
Trying to deal with the issues that are highlighted in https://github.com/web-platform-tests/wpt/pull/54437.
Updated•9 months ago
|
Assignee: nobody → tschuster
Status: NEW → ASSIGNED
Updated•9 months ago
|
Attachment #9509255 -
Attachment description: Bug 1985060 - Bring back original navigate-push-javascript-url.html and don't cause harness errors. r?farre → Bug 1985060 - Bring back navigate-push-javascript-url.html and don't cause harness errors. r?farre
| Assignee | ||
Updated•9 months ago
|
Summary: Bring back original navigate-push-javascript-url.html → Bring back navigate-push-javascript-url.html
Pushed by tschuster@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/bdf79c6d7c57
https://hg.mozilla.org/integration/autoland/rev/d9c03d4190e7
Bring back navigate-push-javascript-url.html and don't cause harness errors. r=farre
Failed to create upstream wpt PR due to merge conflicts. This requires fixup from a wpt sync admin.
Flags: needinfo?(james)
Flags: needinfo?(aborovova)
| Assignee | ||
Comment 4•9 months ago
|
||
Apparently the WPT PR from bug 1981104 never actually landed in the upstream WPT repo and I just didn't realize that. I guess it might be easier to do this change upstream directly.
Pushed by csabou@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/fc26cf0fb6af
https://hg.mozilla.org/integration/autoland/rev/58901b728664
Revert "Bug 1985060 - Bring back navigate-push-javascript-url.html and don't cause harness errors. r=farre" as requested by the dev.
Updated•9 months ago
|
Attachment #9509255 -
Attachment is obsolete: true
| Assignee | ||
Updated•9 months ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Flags: needinfo?(james)
Flags: needinfo?(aborovova)
Resolution: --- → INVALID
Flags: needinfo?(james)
Flags: needinfo?(aborovova)
Updated•9 months ago
|
Flags: needinfo?(james)
Flags: needinfo?(aborovova)
You need to log in
before you can comment on or make changes to this bug.
Description
•