Closed
Bug 1272551
Opened 9 years ago
Closed 8 years ago
Fix the failure of "interfaces.worker" in IDB web-platform test
Categories
(Core :: Storage: IndexedDB, defect)
Core
Storage: IndexedDB
Tracking
()
RESOLVED
FIXED
mozilla49
Tracking | Status | |
---|---|---|
firefox49 | --- | fixed |
People
(Reporter: bevis, Assigned: bevis)
References
(Blocks 1 open bug)
Details
(Whiteboard: [tw-dom] btpp-active)
Attachments
(1 file)
After further investigation, it seems that |WorkerUtils| in [1] is deprecated in the latest spec of WebWorkers [1] and is covered by WorkerGlobalScope.
The test case should be modified to meet the new spec instead.
[1] https://www.w3.org/TR/2009/WD-workers-20091029/#apis-available-to-workers
[2] https://www.w3.org/TR/workers/#apis-available-to-workers
Assignee | ||
Updated•9 years ago
|
Status: NEW → ASSIGNED
Summary: Fix Failure of "interfaces.worker" in IDB web-platform test → Fix the failure of "interfaces.worker" in IDB web-platform test
Whiteboard: [tw-dom] btpp-active
Assignee | ||
Comment 1•9 years ago
|
||
(In reply to Bevis Tseng[:bevistseng][:btseng] from comment #0)
> deprecated in the latest spec of WebWorkers [1] and is covered by WorkerGlobalScope.
^^^ typo of [2]
> [2] https://www.w3.org/TR/workers/#apis-available-to-workers
Assignee | ||
Comment 2•9 years ago
|
||
|WorkerUtils| in [1] is replaced with |WorkerGlobalScope| in the latest spec of WebWorkers [2].
Modify the test case to meet this spec change.
[1] https://www.w3.org/TR/2009/WD-workers-20091029/#apis-available-to-workers
[2] https://www.w3.org/TR/workers/#apis-available-to-workers
Attachment #8752145 -
Flags: review?(khuey)
Attachment #8752145 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 3•8 years ago
|
||
treeherder result looks fine:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=92fb681433bd
Keywords: checkin-needed
Assignee | ||
Updated•8 years ago
|
Attachment #8752145 -
Attachment description: (v1) Patch: Modify the Test Case to Run the Test on |WorkerGlobalScope| instead of |WorkerUtils|. → (v1) Patch: Modify the Test Case to Run the Test on |WorkerGlobalScope| instead of |WorkerUtils|. r=khuey
Comment 4•8 years ago
|
||
(In reply to Bevis Tseng[:bevistseng][:btseng] from comment #0)
> [1] https://www.w3.org/TR/2009/WD-workers-20091029/#apis-available-to-workers
> [2] https://www.w3.org/TR/workers/#apis-available-to-workers
Please don't pay attention to either of those documents; the relevant specification is <https://html.spec.whatwg.org/multipage/#workers>.
Keywords: checkin-needed
Assignee | ||
Comment 6•8 years ago
|
||
(In reply to :Ms2ger from comment #4)
> (In reply to Bevis Tseng[:bevistseng][:btseng] from comment #0)
> > [1] https://www.w3.org/TR/2009/WD-workers-20091029/#apis-available-to-workers
> > [2] https://www.w3.org/TR/workers/#apis-available-to-workers
>
> Please don't pay attention to either of those documents; the relevant
> specification is <https://html.spec.whatwg.org/multipage/#workers>.
Thanks for pointing out this!
Comment 7•8 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in
before you can comment on or make changes to this bug.
Description
•