1206312 - IDBKeyRange binding missing IndexedDatabaseManager include

Status: RESOLVED FIXED

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

Description

[(not currently active) Ted Mielczarek]

IDBKeyRange has:
Func="mozilla::dom::indexedDB::IndexedDatabaseManager::ExperimentalFeaturesEnabled" attribute on the interface here:
https://dxr.mozilla.org/mozilla-central/rev/9ed17db42e3e46f1c712e4dffd62d54e915e0fac/dom/webidl/IDBKeyRange.webidl#33

so it needs to #include IndexedDatabaseManager.h, but IDBKeyRange has a headerFile in Bindings.conf, so the codegen doesn't auto-generate headers:
https://dxr.mozilla.org/mozilla-central/rev/9ed17db42e3e46f1c712e4dffd62d54e915e0fac/dom/bindings/Bindings.conf#636

This is breaking my iOS builds because the set of unified files is slightly different for dom/bindings.

Comment 1

[(not currently active) Ted Mielczarek]

Attachment: MozReview Request: bug 1206312 - add IndexedDatbaseManager include to IDBKeyRange. r?smaug

bug 1206312 - add IndexedDatbaseManager include to IDBKeyRange. r?smaug

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8663209 [details]
MozReview Request: bug 1206312 - add IndexedDatbaseManager include to IDBKeyRange. r?smaug

https://reviewboard.mozilla.org/r/19747/#review17763

r=me (stealing from smaug)

Comment 3

[(not currently active) Ted Mielczarek]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d955b3538d39ef74c38f49e80965656c94a05adb
bug 1206312 - add IndexedDatabaseManager include to IDBKeyRange. r=bz

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/d955b3538d39