Closed
Bug 668826
Opened 14 years ago
Closed 14 years ago
Make reflectString able to test reflections with different IDL attribute and content attribute names
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: mounir, Assigned: mounir)
References
Details
Attachments
(1 file)
6.46 KB,
patch
|
Ms2ger
:
review+
|
Details | Diff | Splinter Review |
With bug 668820, I think we could allow things like this:
reflectString({
element: myElement,
attribute: "myAttribute",
});
and:
reflectString({
element: myElement,
attribute: { idl: "myIDL", content: "myContent", },
});
That would require to check the type of the 'attribute' attribute on the methods. Would you approve such a solution Ms2ger?
Assignee | ||
Comment 1•14 years ago
|
||
Only doing that for reflectString for the moment. Should be easy to do the same thing for other methods if needed.
Assignee | ||
Updated•14 years ago
|
Whiteboard: [needs review]
Comment 2•14 years ago
|
||
Comment on attachment 544093 [details] [diff] [review]
Patch v1
>--- a/content/html/content/test/reflect.js
>+++ b/content/html/content/test/reflect.js
> function reflectString(aParameters)
> {
> var element = aParameters.element;
>- var attr = aParameters.attribute;
>+ var contentAttr = typeof aParameters.attribute == "string"
>+ ? aParameters.attribute : aParameters.attribute.content;
>+ var idlAttr = typeof aParameters.attribute == "string"
>+ ? aParameters.attribute : aParameters.attribute.idl;
===, to match the !== below.
Looks good. r=me
Attachment #544093 -
Flags: review?(Ms2ger) → review+
Assignee | ||
Comment 3•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [needs review]
Target Milestone: --- → mozilla8
Assignee | ||
Updated•14 years ago
|
Status: RESOLVED → UNCONFIRMED
Ever confirmed: false
Resolution: FIXED → ---
Summary: Make reflects.js methods able to tests reflections with different IDL attribute and content attribute names → Make reflectString able to test reflections with different IDL attribute and content attribute names
Target Milestone: mozilla8 → ---
Updated•14 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in
before you can comment on or make changes to this bug.
Description
•