Closed
Bug 832900
Opened 11 years ago
Closed 11 years ago
Make test layout/mathml/tests/stretchy-and-large-operators.html work
Categories
(Core :: MathML, defect)
Core
MathML
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: fredw, Assigned: darkowlzz)
References
()
Details
(Keywords: helpwanted, Whiteboard: [mentor=fredw][lang=js][good first bug])
Attachments
(1 file, 2 obsolete files)
2.96 KB,
patch
|
fredw
:
review+
|
Details | Diff | Splinter Review |
layout/mathml/tests/stretchy-and-large-operators.html is an exhaustive test for the stretchy and large operators from our operator dictionary. However is does not work at the moment, because it incorrectly creates MathML elements with the createElement() function. The function createElementNS() (with the MathML namespace) should be used instead. (See bug 755541 and 832713 comment 3)
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #704958 -
Flags: review?
Reporter | ||
Comment 2•11 years ago
|
||
Thanks, that look good. I would put the MathML namespace string in a global variable rather than repeating it everywhere, but that's OK. Be sure to try to conform to Mozilla's style guides: 80-character lines and a space after the commas. (https://developer.mozilla.org/En/Mozilla_Coding_Style_Guide) For the commit message, see a model here: https://developer.mozilla.org/en-US/docs/Developer_Guide/Committing_Rules_and_Responsibilities#Commit_message_restrictions
Assignee: nobody → indiasuny000
Assignee | ||
Comment 3•11 years ago
|
||
Made some changes. Created a global variable to supply namespace to all the creation code.
Attachment #704958 -
Attachment is obsolete: true
Attachment #704958 -
Flags: review?
Attachment #705008 -
Flags: review?(fred.wang)
Assignee | ||
Updated•11 years ago
|
Attachment #705008 -
Attachment is obsolete: true
Attachment #705008 -
Flags: review?(fred.wang)
Assignee | ||
Comment 4•11 years ago
|
||
Added Bug number in commit message
Attachment #705011 -
Flags: review?(fred.wang)
Assignee | ||
Updated•11 years ago
|
Whiteboard: [mentor=fredw][lang=c++][good first bug] → [mentor=fredw][lang=js][good first bug]
Reporter | ||
Comment 5•11 years ago
|
||
Comment on attachment 705011 [details] [diff] [review] Made changes as per the comment That looks good, thanks!
Attachment #705011 -
Flags: review?(fred.wang) → review+
Assignee | ||
Updated•11 years ago
|
Attachment #705011 -
Flags: checkin?(fred.wang)
Reporter | ||
Updated•11 years ago
|
Keywords: checkin-needed
Reporter | ||
Comment 6•11 years ago
|
||
Comment on attachment 705011 [details] [diff] [review] Made changes as per the comment I don't have commit access. But I put the checkin-needed keyword, so someone will take care of that relatively soon.
Attachment #705011 -
Flags: checkin?(fred.wang)
Comment 7•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/36477d961bf2
Flags: in-testsuite+
Keywords: checkin-needed
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/36477d961bf2
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in
before you can comment on or make changes to this bug.
Description
•