Closed
Bug 589053
Opened 14 years ago
Closed 14 years ago
Capitalization mistake in nsIDOMHTMLMediaElement.idl
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: cpearce, Assigned: cpearce)
Details
Attachments
(1 file)
|
1.48 KB,
patch
|
Details | Diff | Splinter Review |
In bug 548523 I added the attribute 'preload', but when it was declared in the IDL, it was capitalized as 'DomString', but it's supposed to be 'DOMString'. Apparently IDL is supposed to be case sensitive and so my IDL changes were not even supposed to work at all, despite them having worked locally, and on trunk for the past 6 hours, and building and passing tests on TryServer on all platforms previously as well...
|
Assignee | |
Comment 1•14 years ago
|
||
Assignee: nobody → chris
Attachment #467675 -
Flags: review?(roc)
Attachment #467675 -
Flags: approval2.0?
|
|
Assignee | |
Comment 2•14 years ago
|
||
hmm, it seems the reason this didn't cause bustage is that the build machines didn't do a clobber build. If I delete my object directory and rebuild, I get a build failure, so I'll just check in this patch as a=bustage-fix. Surprising it didn't cause bustage on TryServer.
|
|
Assignee | |
Updated•14 years ago
|
Attachment #467675 -
Flags: review?(roc)
Attachment #467675 -
Flags: approval2.0?
|
|
Assignee | |
Comment 3•14 years ago
|
||
Pushed as a bustage-fix: http://hg.mozilla.org/mozilla-central/rev/79f2ae861bf1
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•