Closed
Bug 1109278
Opened 10 years ago
Closed 9 years ago
undocumented Debugger.Script method
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: tromey, Assigned: ejpbruel)
References
Details
Attachments
(1 file)
2.69 KB,
patch
|
jimb
:
review+
|
Details | Diff | Splinter Review |
The Debugger.Script methods "getAllColumnOffsets" and "isInCatchScope" are not documented in Debugger.Script.md. Also, the "strictMode" attribute is documented, but does not seem to be implemented.
Reporter | ||
Comment 1•10 years ago
|
||
"decompile" also seems to be documented but not implemented.
Comment 3•9 years ago
|
||
Eddy, these are both yours. Will you have any time to take a shot at documenting them?
Flags: needinfo?(ejpbruel)
Assignee | ||
Comment 4•9 years ago
|
||
Sure Jim! I'll put it on my todo list for next week.
Assignee: nobody → ejpbruel
Flags: needinfo?(ejpbruel)
Reporter | ||
Comment 5•9 years ago
|
||
Source.displayURL is also missing
Comment 6•9 years ago
|
||
(In reply to Tom Tromey :tromey from comment #5) > Source.displayURL is also missing I think that one's fitzgen (and my bad review).
Comment 7•9 years ago
|
||
(In reply to Eddy Bruel [:ejpbruel] from comment #4) > Sure Jim! I'll put it on my todo list for next week. Oh, great! Thanks very much.
Assignee | ||
Comment 8•9 years ago
|
||
Jim, what do you use to view these markdown files? I use a Firefox plugin, but I'm seeing some weird things such as part of the description of getAllOffsets being preformatted while its clearly not intended to be. The markdown I wrote just mimics what we do everywhere else, but that seems wrong somehow.
Attachment #8548829 -
Flags: review?(jimb)
Comment 9•9 years ago
|
||
Comment on attachment 8548829 [details] [diff] [review] Document getAllColumnOffsets and isInCatchScope Review of attachment 8548829 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me, with the point below clarified. ::: js/src/doc/Debugger/Debugger.Script.md @@ +303,5 @@ > error. > > +<code>isInCatchScope([<i>offset</i>])</code> > +: This is `true` if this offset falls within the scope of a try/catch block, > + and `false` otherwise. Only if it's in the 'try' clause, not the 'catch' clause, right?
Attachment #8548829 -
Flags: review?(jimb) → review+
Comment 10•9 years ago
|
||
(In reply to Eddy Bruel [:ejpbruel] from comment #8) > Created attachment 8548829 [details] [diff] [review] > Document getAllColumnOffsets and isInCatchScope > > Jim, what do you use to view these markdown files? I use a Firefox plugin, > but I'm seeing some weird things such as part of the description of > getAllOffsets being preformatted while its clearly not intended to be. The > markdown I wrote just mimics what we do everywhere else, but that seems > wrong somehow. See the description of 'format.sh' in js/src/doc/README.md. It explains how I check the appearance.
Assignee | ||
Comment 11•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/dfaf13cfe206
https://hg.mozilla.org/mozilla-central/rev/dfaf13cfe206
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in
before you can comment on or make changes to this bug.
Description
•