Closed Bug 981728 Opened 6 years ago Closed 6 years ago

Spidermonkey reports a syntax error in browser.js

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 30

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(1 file, 1 obsolete file)

I have a thing in my local build scripts that checks for JS syntax errors. And it found one!
Attachment #8388681 - Flags: review?(wjohnston) → review+
Comment on attachment 8388681 [details] [diff] [review]
1-doubletap-dc9d7be

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

Hmm.. Its weird that we're not seeing this in our local consoles...
Yeah, it is. For the record, the way I'm checking is using a locally-built standalone spidermonkey binary (instructions at [1]; I tested with the latest m-c code) and the script at https://github.com/staktrace/moz-scripts/blob/master/jscheck where "js.$(uname)" points to the spidermonkey binary. The first line of the jscheck script deals with preprocessor goop, and AFAICT doesn't interfere with this particular bug.

[1] https://developer.mozilla.org/en-US/docs/SpiderMonkey/Build_Documentation
Attachment #8388681 - Flags: review?(wjohnston)
https://hg.mozilla.org/mozilla-central/rev/e6b3d4a09ea6
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 30
You need to log in before you can comment on or make changes to this bug.