Closed
Bug 1280471
Opened 8 years ago
Closed 8 years ago
Turn on highlight for WebAssembly text format
Categories
(DevTools :: Debugger, defect)
DevTools
Debugger
Tracking
(firefox50 fixed)
RESOLVED
FIXED
Firefox 50
Tracking | Status | |
---|---|---|
firefox50 | --- | fixed |
People
(Reporter: yury, Assigned: yury)
References
Details
Attachments
(1 file)
Bug 1274618 landed to display text for WebAssembly code (improved in bug 1274618). It will be nice to add highlight to CodeMirror for this format. Grammar is currently can be found at https://github.com/mbebenita/was/blob/master/was_lexer.l
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/59436/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/59436/
Attachment #8763108 -
Flags: review?(jlong)
Assignee | ||
Comment 2•8 years ago
|
||
James, I tentatively set you as a reviewer -- let me know if you cannot review the CodeMirror integration piece.
Comment 3•8 years ago
|
||
Comment on attachment 8763108 [details] Bug 1280471 - Turn on highlight for WebAssembly text format. https://reviewboard.mozilla.org/r/59436/#review57266 Looks good to me!
Attachment #8763108 -
Flags: review?(jlong) → review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/fx-team/rev/87b183a92a93 Turn on highlight for WebAssembly text format. r=jlongster
Keywords: checkin-needed
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/87b183a92a93
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•