All users were logged out of Bugzilla on October 13th, 2018

[jsd] Change jsdIFilter and jsdIScript to work off 'window' rather than 'url'

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
10 years ago
10 years ago

People

(Reporter: johnjbarton, Unassigned)

Tracking

Trunk
x86
Windows XP

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

10 years ago
The current jsdIDebugggerService infrastructure works off of URLs (filename). This is the right API for the user interface, eg Firebug, which needs to express actions in the way that users will (eg by URL). But its the wrong interface for jsdIDebuggerService because multiple windows can have the same URL. One operation Firebug has to do a lot is map the URL into the window (and Firebug's state object for the window called 'context') because the state is what the script/error/net data will be used with. If the mapping fails, then the debugger fails. 

This would be a pervasive change, so we need to go at this a bit at a time. To start I propose we add window refs to jsdIScript (and nsIConsoleMessage but that's another bug) and jsdIFilter.  This will probably have to be done with derived interfaces, jsdIScript2 and jsdIFilter2.
Component: General → JavaScript Debugger
Product: Firefox → Other Applications
Version: unspecified → Trunk

Comment 1

10 years ago
John, jsdIFilter has  'globalObject' property which is intended to filter by 'window' object.
(Reporter)

Comment 2

10 years ago
Thanks, I missed that, I've never used jsdIFilter. I don't really get how jsdIFilter works, it seems like you have to know the line range of the target. I guess you just these to 1 and maximum integer value.

While having the window on jsdIScript would be handy, knowing that its on jsdIFilter plus fixes 449454 and 449458 would make the proposal in this bug report low priority.

Comment 3

10 years ago
for now to simplify my life, i'm going to resolve this as wontfix. if you already have access to what you need, then i don't see any reason to pollute another interface w/ another way of reaching it.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.