Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2025-03-01 13:57:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 878958: Use a Map to map actor names to reply handlers, not an object. r=dcamp

Browse Source
This commit is contained in:
Jim Blandy 2013-06-14 13:14:30 -07:00
parent 296fb31c55
commit 29e5bf70c0
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
toolkit/devtools/client/dbg-client.jsm
View File

@ -217,7 +217,7 @@ this.DebuggerClient = function DebuggerClient(aTransport)
this._consoleClients = {};
this._pendingRequests = [];
this._activeRequests = {};
this._activeRequests = new Map;
this._eventsEnabled = true;
/* The root actor for this client's main connection. */
@ -544,11 +544,11 @@ DebuggerClient.prototype = {
*/
_sendRequests: function DC_sendRequests() {
this._pendingRequests = this._pendingRequests.filter((request) => {
if (request.to in this._activeRequests) {
if (this._activeRequests.has(request.to)) {
return true;
}
this._activeRequests[request.to] = request;
this._activeRequests.set(request.to, request.onResponse);
this._transport.send(request.request);
return false;
@ -591,12 +591,12 @@ DebuggerClient.prototype = {
let onResponse;
// Don't count unsolicited notifications or pauses as responses.
if (aPacket.from in this._activeRequests &&
if (this._activeRequests.has(aPacket.from) &&
!(aPacket.type in UnsolicitedNotifications) &&
!(aPacket.type == ThreadStateTypes.paused &&
aPacket.why.type in UnsolicitedPauses)) {
onResponse = this._activeRequests[aPacket.from].onResponse;
delete this._activeRequests[aPacket.from];
onResponse = this._activeRequests.get(aPacket.from);
this._activeRequests.delete(aPacket.from);
}
// Packets that indicate thread state changes get special treatment.