No description provided.
Comment on attachment 823035 [details] [diff] [review] Patch v1 Everything looks good to me, but I definitely recommend a try run. Alphabetizing the include list always makes me worry that we'll be changing which headers get picked up. As a side note, the size of LOCAL_INCLUDES in dom/bindings is frightening :)
Attachment #823035 - Flags: review?(mshal) → review+
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
You need to log in before you can comment on or make changes to this bug.