Use findScripts to retrieve the list of scripts known to the debugger

RESOLVED FIXED in Firefox 14

Status

()

Firefox
Developer Tools: Debugger
P2
normal
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: past, Assigned: past)

Tracking

Trunk
Firefox 14
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

When bug 676281 lands, the debugger server should use Debugger.prototype.getAllScripts to retrieve the list of all scripts known to the debugger. Currently we rely on onNewScript notifications to generate the list, which doesn't work when the client takes a while to attach to the server, thus missing the initial notifications.
See bug 697040 for a manifestation of the problem with the current solution.
We are implementing a workaround in bug 697040, so the first thing to do in this bug is revert that. I'll upload the necessary patch once 697040 lands.
Created attachment 599914 [details] [diff] [review]
Backout patch

This is the backout patch for the workaround in bug 697040.
Assignee: nobody → past
Status: NEW → ASSIGNED
Priority: P3 → P2
Summary: Use getAllScripts to retrieve the list of scripts known to the debugger → Use findScripts to retrieve the list of scripts known to the debugger
Created attachment 605843 [details] [diff] [review]
WIP

This is almost there, but I'm having issues with getting the client to send the request. Might be related to bug 731537.
Attachment #599914 - Attachment is obsolete: true
Created attachment 606271 [details] [diff] [review]
Working patch

Fixed the tests by adding an extra event. This may come in handy with some oranges as well. Also contains the typo fix from bug 736000.
Attachment #605843 - Attachment is obsolete: true
Attachment #606271 - Flags: review?(rcampbell)
Blocks: 728244
Comment on attachment 606271 [details] [diff] [review]
Working patch

very nice. I wish I had comments for you.
Attachment #606271 - Flags: review?(rcampbell) → review+
if it fixes orange, I'll be super happy. :)
Whiteboard: [land-in-fx-team]
I sneaked in another one-liner to fix the gcli breakpoint test in the same way as the others:

https://hg.mozilla.org/integration/fx-team/rev/794ff67c7a9a
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Depends on: 739995
Backed out due to new randomorange:

https://hg.mozilla.org/integration/fx-team/rev/4b87cffba4d5
Whiteboard: [fixed-in-fx-team]
Relanded:

https://hg.mozilla.org/integration/fx-team/rev/401804642db1
Whiteboard: [fixed-in-fx-team]
Blocks: 736000
https://hg.mozilla.org/mozilla-central/rev/401804642db1
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 14
You need to log in before you can comment on or make changes to this bug.