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]
This is the backout patch for the workaround in bug 697040.
Created attachment 605843 [details] [diff] [review]
This is almost there, but I'm having issues with getting the client to send the request. Might be related to bug 731537.
Created attachment 606271 [details] [diff] [review]
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.
Comment on attachment 606271 [details] [diff] [review]
very nice. I wish I had comments for you.
if it fixes orange, I'll be super happy. :)
I sneaked in another one-liner to fix the gcli breakpoint test in the same way as the others:
Backed out due to new randomorange: