Closed Bug 1673738 Opened 4 years ago Closed 3 years ago

Some .cpp files do not include all the header files they require

Categories

(Core :: DOM: Service Workers, task)

task

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: ytausky, Assigned: ytausky)

Details

Attachments

(1 file)

Due to the unified build, it's possible for a file to not include declarations for all the names it uses and not break the build. It does throw off clangd, though, so this should be fixed.

Assignee: nobody → ytausky
Status: NEW → ASSIGNED
Pushed by ytausky@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a3e402c60e55
Add missing includes to ServiceWorkerOp.cpp r=dom-workers-and-storage-reviewers,sg
Keywords: leave-open

The leave-open keyword is there and there is no activity for 6 months.
:ytausky, maybe it's time to close this bug?

Flags: needinfo?(ytausky)

The leave-open keyword is there and there is no activity for 6 months.
:ytausky, maybe it's time to close this bug?

Flags: needinfo?(ytausky)
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME
Flags: needinfo?(ytausky)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: