Move test_bug1290951.html to WPT

RESOLVED FIXED in Firefox 65

Status

()

P2
normal
RESOLVED FIXED
2 years ago
5 months ago

People

(Reporter: bhsu, Assigned: perry)

Tracking

unspecified
mozilla65
Points:
---

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
When extending byte-check to imported scripts before launching a service worker, there comes a corner case that an imported script might be no longer available after its hosting site revises it. Thus, when practising byte-check on imported scripts, we tolerate network failures of imported scripts, as suggested by the Patch 3.2 for bug 1290951 (https://bug1290951.bmoattachments.org/attachment.cgi?id=8841928). Since this behaviour should be shared among different browsers, the testcase should be moved to web-plateform-tests. 

Also, it would be great that we give the testcase a more understandable name :)
(Reporter)

Updated

2 years ago
Depends on: 1290951
(Reporter)

Updated

2 years ago
Assignee: nobody → bhsu
Priority: -- → P3
Hi Eden, could you please put this into your queue and help this? Ho-Pang has some ideas that you might want to discuss with him. Thank you.
Assignee: bhsu → nobody
Flags: needinfo?(echuang)
Priority: P3 → P2
OK, I will discuss with him.
Assignee: nobody → echuang
Status: NEW → ASSIGNED
Flags: needinfo?(echuang)
Assignee: echuang → nobody
Status: ASSIGNED → NEW
Priority: P2 → P3
Assignee: nobody → perry
Status: NEW → ASSIGNED

Updated

6 months ago
Priority: P3 → P2
(Assignee)

Comment 4

5 months ago
Eden, would you be able to review this?
Flags: needinfo?(echuang)
Sure, send a review request to me.
Flags: needinfo?(echuang)
Attachment #9010789 - Attachment description: Bug 1357290 - move test_update_missing_imported_script.html to WPT r?asuth → Bug 1357290 - move test_update_missing_imported_script.html to WPT r?edenchuang
(Assignee)

Updated

5 months ago
Keywords: checkin-needed
Hi, when trying to land this bug I received this:
Error: We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again. (255, 'applying /tmp/tmpMSD_J4\npatching file testing/web-platform/meta/MANIFEST.json\nHunk #3 FAILED at 578416\nHunk #4 FAILED at 578808\nHunk #5 FAILED at 579220\nHunk #8 FAILED at 656480\nHunk #9 FAILED at 656516\nHunk #10 FAILED at 656616\n6 out of 10 hunks FAILED -- saving rejects to file testing/web-platform/meta/MANIFEST.json.rej\nabort: patch failed to apply', '') 

Please take a look.
Flags: needinfo?(perry)
Keywords: checkin-needed
(Assignee)

Comment 7

5 months ago
My bad, just updated the changes to be rebaseable. Removed the MANIFEST.json changes due to bug 1473915.
Flags: needinfo?(perry)
(Assignee)

Updated

5 months ago
Keywords: checkin-needed

Comment 8

5 months ago
Pushed by aciure@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9f3a74845bdb
move test_update_missing_imported_script.html to WPT r=edenchuang
Keywords: checkin-needed
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/13728 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 11

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9f3a74845bdb
Status: ASSIGNED → RESOLVED
Last Resolved: 5 months ago
status-firefox65: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Upstream PR merged
You need to log in before you can comment on or make changes to this bug.