Closed Bug 1409060 Opened 5 years ago Closed 5 years ago
Decoder .h includes necko-config .h unnecessarily
59 bytes, text/x-review-board-request
MediaDecoder.h is including necko-config.h, but I can't see any reason why it needs to. necko-config.h just #undefs NECKO_WIFI and NECKO_COOKIES, and we don't depend on those, so I don't see why we need this.
Comment on attachment 8918914 [details] Bug 1409060 - Don't include necko-config.h in MediaDecoder.h. https://reviewboard.mozilla.org/r/189812/#review195206
Attachment #8918914 - Flags: review?(jwwang) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/88383205b1d4 Don't include necko-config.h in MediaDecoder.h. r=jwwang
You need to log in before you can comment on or make changes to this bug.