User:GijsKruitbosch/JS Debugging/Tasks: Difference between revisions
Jump to navigation
Jump to search
No edit summary |
|||
| Line 109: | Line 109: | ||
<td>Anonymous functions should not be lumped together in profiling.</td> | <td>Anonymous functions should not be lumped together in profiling.</td> | ||
<td>Presumably not</td> | <td>Presumably not</td> | ||
<td> | <td>{{bug|347888}}</td> | ||
<td>Understand problem, fix wanted</td> | |||
<td> | |||
</tr> | </tr> | ||
<tr> | <tr> | ||
| Line 118: | Line 117: | ||
<td>No</td> | <td>No</td> | ||
<td>-</td> | <td>-</td> | ||
<td>FIXED</td> | <td>FIXED</td> | ||
</tr> | </tr> | ||
| Line 126: | Line 124: | ||
<td>14</td> | <td>14</td> | ||
<td>Figure out why I'm crashing on GMail with firebug enabled (hack/sneak/workaround?) and on exit when using two debuggers</td> | <td>Figure out why I'm crashing on GMail with firebug enabled (hack/sneak/workaround?) and on exit when using two debuggers</td> | ||
<td>Hopefully not.</td> | <td>Hopefully not.</td> | ||
<td>{{bug|342074}}</td> | <td>{{bug|342074}}</td> | ||
| Line 134: | Line 131: | ||
<td>15</td> | <td>15</td> | ||
<td>Fix wrapping of variable/value tree</td> | <td>Fix wrapping of variable/value tree</td> | ||
<td>No</td> | <td>No</td> | ||
<td>-</td> | <td>-</td> | ||
| Line 143: | Line 139: | ||
<td>JS 1.7 improvements</td> | <td>JS 1.7 improvements</td> | ||
<td>Yes?</td> | <td>Yes?</td> | ||
<td>-</td> | <td>-</td> | ||
<td>None yet</td> | <td>None yet</td> | ||
| Line 150: | Line 145: | ||
<td>17</td> | <td>17</td> | ||
<td>Debug mutation events (watch DOM elements)</td> | <td>Debug mutation events (watch DOM elements)</td> | ||
<td>Yes</td> | <td>Yes</td> | ||
<td>-</td> | <td>-</td> | ||
| Line 158: | Line 152: | ||
<td>18</td> | <td>18</td> | ||
<td>Getters / Setters improvements</td> | <td>Getters / Setters improvements</td> | ||
<td>Yes?</td> | <td>Yes?</td> | ||
<td>-</td> | <td>-</td> | ||
| Line 166: | Line 159: | ||
<td>19</td> | <td>19</td> | ||
<td>'real' stack list (not a dropdown)</td> | <td>'real' stack list (not a dropdown)</td> | ||
<td>No</td> | <td>No</td> | ||
<td>-</td> | <td>-</td> | ||
<td>WONTFIX | <td>WONTFIX</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td>20</td> | <td>20</td> | ||
<td>Inline editing of functions etc.</td> | <td>Inline editing of functions etc.</td> | ||
<td>No, hopefully.</td> | <td>No, hopefully.</td> | ||
<td>-</td> | <td>-</td> | ||
| Line 182: | Line 173: | ||
<td>21</td> | <td>21</td> | ||
<td>Pretty print</td> | <td>Pretty print</td> | ||
<td>No</td> | <td>No</td> | ||
<td>-</td> | <td>-</td> | ||
Revision as of 19:37, 8 August 2006
Importance-ordered Task List
A small note about this list: importance != working order, given that some of these items need other items, and that some require back-end changes which will need to happen fast, if at all, on the 1.8.1 branch (which is what I'm trying to cater to).
Front end work is (mostly) being done on: Firebug
| N | Task | Needs backend changes | Bug number | Result |
| 0 | Security Issues | Yes | - | Out of my hands |
| 1 | Implement list of Breakpoints | No | - | Patch |
| 2 | Implement Watches | No | - | Patch |
| 3 | Make the originating document accessible from any error in the console | No | None | Patch |
| 4 | Frontend adapted to keep breakpoints and such separate on a per-document basis (for Firebug's console) (see above) | No | None | Patch |
| 5 | Be able to go from a var referencing a DOM element to that DOM element in DOMI | No | bug 342651 | FIXED |
| 6 | Be able to debug from the slow script warning dialog. | Yes | bug 341764 | FIXED |
| 7 | Newlines are lost in Firebug's watch pane. | No | - | WORKSFORME |
| 8 | Venkman should try to load line-maps lazily (on-demand, and not demand them immediately like it does now). | No | bug 341758 | None yet |
| 9 | Changing variable values / evaluating js shouldn't require changing tabs away from the debugger (in Firebug) | No | - | Patch |
| 10 | Evaluate js in different windows/tabs/iframes | No | - | None yet |
| 11 | Provide immediate access to profiling data | No | bug 341767 | FIXED |
| 12 | Anonymous functions should not be lumped together in profiling. | Presumably not | bug 347888 | Understand problem, fix wanted |
| 13 | Double-check handling of about:blank and (related) running/stopped state tracking in Firebug. | No | - | FIXED |
| 14 | Figure out why I'm crashing on GMail with firebug enabled (hack/sneak/workaround?) and on exit when using two debuggers | Hopefully not. | bug 342074 | FIXED |
| 15 | Fix wrapping of variable/value tree | No | - | FIXED |
| 16 | JS 1.7 improvements | Yes? | - | None yet |
| 17 | Debug mutation events (watch DOM elements) | Yes | - | Is Hewitt fixing this himself? |
| 18 | Getters / Setters improvements | Yes? | - | None yet |
| 19 | 'real' stack list (not a dropdown) | No | - | WONTFIX |
| 20 | Inline editing of functions etc. | No, hopefully. | - | None yet |
| 21 | Pretty print | No | - | None yet |
Getting to the document that error X originated from
Discussion
I would probably order these things higher:
- immediate profiling
- newlines in watches view (but there's no watch support yet, so?)
- eval JS in tab/frame/etc.
And these things lower:
- JS 1.7
- inline function editing
But I think you're generally on the right track. I need to write here how we can track what document an error came from without biting off everything that's in the current bug 228205, too. Basically involves finding the current document (via cx->contextPrivate->scriptOwner->frameElement or some such) from the handler called for exceptions and errors, and keeping track of it. We could also "just" have the console error object QI to nsIException, but I bet that would make some people API-sad, so I'm interested in us keeping the hacks to our side of the line if we can.
-- shaver
- Done, I think. GijsKruitbosch 07:20, 16 June 2006 (PDT)