Closed Bug 723207 Opened 8 years ago Closed 8 years ago

info() is broken for mochitest/mochitest-chrome

Categories

(Testing :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla13

People

(Reporter: Gavin, Assigned: Gavin)

Details

Attachments

(1 file)

It's exposed as a "global symbol" directly, but uses "this" incorrectly. Simple fix...
The error:

3 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/docshell/test/chrome/test_principalInherit.xul | an unexpected uncaught JS exception reported through window.onerror - this._logResult is not a function at chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:287
Summary: info() is broken for mochitest-chrome → info() is broken for mochitest/mochitest-chrome
Attached patch patchSplinter Review
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #593535 - Flags: review?
Attachment #593535 - Flags: review? → review?(jmaher)
Comment on attachment 593535 [details] [diff] [review]
patch

Review of attachment 593535 [details] [diff] [review]:
-----------------------------------------------------------------

this looks good.  Does this pass on try?
Attachment #593535 - Flags: review?(jmaher) → review+
Flags: in-testsuite+
https://hg.mozilla.org/mozilla-central/rev/c831b760a443
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.