Closed Bug 909563 Opened 7 years ago Closed 7 years ago

Bindings.conf cleanups

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: khuey, Assigned: khuey)

Details

Attachments

(3 files)

We have an interesting collection of 2 and 3 space indentation.
Attachment #795718 - Flags: review?(bzbarsky)
Comment on attachment 795718 [details] [diff] [review]
Part 1: Fix whitespace

r=me
Attachment #795718 - Flags: review?(bzbarsky) → review+
Comment on attachment 795719 [details] [diff] [review]
Part 2: Remove some easily removed annotations

r=me
Attachment #795719 - Flags: review?(bzbarsky) → review+
Comment on attachment 795756 [details] [diff] [review]
Rename nsDOMParser and drop annotations

>+  rv = DOMParser::ParseFromBuffer(aBuf.Elements(), aBufLen,
>                                     StringFromSupportedType(aType),

Fix indent?  This happens twice.

>+  rv = DOMParser::ParseFromStream(aStream,
>                                     NS_ConvertUTF16toUTF8(aCharset).get(),

And here.

r=me
Attachment #795756 - Attachment is patch: true
Attachment #795756 - Flags: review?(bzbarsky) → review+
There's a few places in Bindings.conf where there's a space before a colon, if you feel like fixing that.
I'll file new bugs for other stuff.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [leave open]
Target Milestone: --- → mozilla26
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.