Closed Bug 1415829 Opened 7 years ago Closed 7 years ago

Files under dom/clients/manager are not covered by tests

Categories

(Core :: DOM: Core & HTML, defect, P2)

Product:
Core
Component:
DOM: Core & HTML
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox58 --- affected

People

(Reporter: marco, Unassigned)

References

(Blocks 1 open bug)

Details

Bug 1413606 recently added several files to dom/clients/manager, but they are all not covered by tests.
Flags: needinfo?(bkelly)
Blocks: 1413606
I am landing code incrementally from a mega patch in bug 1293277. It's definitely tested in the try runs there, but yes the initial code is not tested. It's also not actually executed anywhere. If we can land things incrementally as reviews happen it will make getting something this large implemented extremely difficult.
No longer blocks: 1413606
Depends on: 1293277
Flags: needinfo?(bkelly)
Last comment should read "if we cannot land things incrementally".
Priority: -- → P2
Now that the dom/clients code is fully hooked into our bindings its covered by our pre-existing Clients API mochitests and WPT tests.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.