Closed Bug 914260 Opened 6 years ago Closed 6 years ago

Add review requirement to the error message in test_interfaces

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: mccr8, Assigned: mccr8)

Details

Attachments

(1 file)

I'll get to this in the next day or two, but if somebody else wants to grab it before then, feel free.
Other ideas discussed in IRC were adding the review requirement to every line, and possibly include the name of the DOM peer who approved that line.  Possibly with cryptographic signing of each line. ;)
Attached patch 914260.patchSplinter Review
Attachment #801914 - Flags: review?(bzbarsky)
Comment on attachment 801914 [details] [diff] [review]
914260.patch

r=me
Attachment #801914 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/mozilla-central/rev/2b76eb674066
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.