don't leave gDeadScripts pointing at garbage. May or may not fix the problem at hand, but is an improvement in either case.
git-svn-id: svn://10.0.0.236/trunk@125124 18797224-902f-48f8-a5cc-f745e15eee43
Stop caching jsdIValues, it doesn't seem to be worth the lookup cost. This has the side effect of plugging the leak, as described in the bug.
also...
* Convert a few raw pointers to nsCOMPtrs
* Fix a bug where removing the last filter did not null out the list head,
causing a crash the next time filters were used.
* Track live jsdStackFrames, so we can invalidate them all when execution
continues. Without this, only the top frame is properly invalidated, and any
other frame accessed after a continue will do Bad Things.
* Add some debugging prints to GetInitAtService, which seems to be failing at
random times.
git-svn-id: svn://10.0.0.236/trunk@123613 18797224-902f-48f8-a5cc-f745e15eee43
bug 139883, "ErrorHookProc can be tricked into thinking it never finished running", r=peterv
fixes bad early return in jsds_ErrorHookProc
test for correct max params (12, not 8) in CreatePPLineMap
use nsnull instead of 0 for null pointer return value
git-svn-id: svn://10.0.0.236/trunk@120000 18797224-902f-48f8-a5cc-f745e15eee43
sr=shaver, r=jband
add JSD_IsStackFrameNative, IsStackFrameDebugger, and IsStackFrameConstructing
add similar attributes to jsdIStackFrame
tweak return values in jsds_FilterHook
don't include dummy stack frames in threadstates
bug 110387, "Crash on exiting venkman"
check to see if the debugger was turned off before going through with an unPause.
git-svn-id: svn://10.0.0.236/trunk@114496 18797224-902f-48f8-a5cc-f745e15eee43
adds jsdIValue::getWrappedValue() and jsdIService::wrapValue() methods to provid
e js scripts a way to convert between jsdIValues and "real" js values.
Also includes fix to not call breakpoint hook while paused, and changes jsdIFilt
er::glob attribute to jsdIFilter::globalObject, to match jsdIContext::globalObject
git-svn-id: svn://10.0.0.236/trunk@113854 18797224-902f-48f8-a5cc-f745e15eee43
implement the jsdIContext interface.
create only one jsdValue wrapper per unique jsval.
add executionContext property to jsdIStackFrame.
add enumerateContexts to jsdIDebuggerService.
add jsdIContextEnumerator interface.
git-svn-id: svn://10.0.0.236/trunk@113111 18797224-902f-48f8-a5cc-f745e15eee43
adds jsdIDebugHook interface, change the meaning of jsdIErrorHook.
error hook is called when an error occurs, and is given information about the error report.
debug hook is called when/if the error hook returns false, and is given information about the ececution state.
git-svn-id: svn://10.0.0.236/trunk@113100 18797224-902f-48f8-a5cc-f745e15eee43
use nsCOMPtr to avoid releasing a final reference via jsdScript::Invalidate()
git-svn-id: svn://10.0.0.236/trunk@112686 18797224-902f-48f8-a5cc-f745e15eee43
large addition to the jsd_xpc component allows arbitrary filtering of debug hooks by url pattern, line range, and global object. also adds ability to begin instrumenting jsscripts at app startup.
git-svn-id: svn://10.0.0.236/trunk@106689 18797224-902f-48f8-a5cc-f745e15eee43
remove venkman startup listener so we don't have to load the dll at startup.
checking into 0.9.5 branch also.
git-svn-id: svn://10.0.0.236/trunk@104808 18797224-902f-48f8-a5cc-f745e15eee43
init xpconnect on the debugger's default context so that xpconnect can use it when it has to. r=jband
git-svn-id: svn://10.0.0.236/trunk@102366 18797224-902f-48f8-a5cc-f745e15eee43
turn off verbose debugging for me
push a thread event queue for necko before we enter a nested event loop
git-svn-id: svn://10.0.0.236/trunk@102328 18797224-902f-48f8-a5cc-f745e15eee43
don't allow gc's during script hooks if CAUTIOUS_SCRIPTHOOK is defined (which it is, by default.) Should help with stability until we can fix the real problems.
Use JSVAL_ macros instead of JSD_* calls in jsdValue::GetJSType method, avoiding two c++ frames per call.
git-svn-id: svn://10.0.0.236/trunk@101912 18797224-902f-48f8-a5cc-f745e15eee43
remove isActive attribute from jsdScript (it's the same thing as isValid)
fix bug in jsdValue::GetDoubleValue()
add jsdService::GC()
git-svn-id: svn://10.0.0.236/trunk@100262 18797224-902f-48f8-a5cc-f745e15eee43
Large changes to improve the way we deal with our wrappers around js engine structures. jsdIScript, jsdIStackFrame, jsdIValue, and jsdIProperty interfaces now inherit from a new interface "jsdIEphemeral". This interface is used to invalidate the wrapper. Once the wrapper is invalidated, *most* methods throw NS_ERROR_NOT_AVAILABLE, some interfaces, such as jsdIScript, cache important information so that the wrapper isn't utterly useless once it has been invalidated. The boolean isValid attribute can be used to see if the wrapper is still valid.
factor debug object counters into some simple macros
add new velid assertion macros for the new ephemeral objects
add utility functions for dealing with PR_CLISTs full of ephemeral objects.
invalidate the jsdIFrame after the execution hook completes
move some c/dtors from jsd_xpc.h over here to avoid exposing debug object counters, and repeating some macros
fix incorrectly set out parameter in getValue::GetDoubleValue
git-svn-id: svn://10.0.0.236/trunk@98576 18797224-902f-48f8-a5cc-f745e15eee43
large changes to fix the following bugs:
82684, crash manually clearing breakpoint
*actually* clearing mValid in jsdScript::Invalidate fixed this one.
85636, assertions on quiting venkman
jsdService::Off now disconnects the hooks into JSD, to avoid calling back into js after that. It also processes any pending script delete events that occurred during the last GC. The code to process the gPendingScripts list has been factored out of the gc callback. Processing the dead script list allows us to properly finalize all of the jsdIScript object, which seems to clear up the "gc roots exist at shutdown" assertions. In effect, these changes get rid of *all* of the jsd related assertions on exit.
Added isOn attribute to jsdIService.
Added isValid attribute to jsdIScript. We now prefetch appropriate properties from the underlying JSDScript, so that it's available after the script is Invalidate()d
moved jsdService constructor to jsd_xpc.h
Save the runtime passed to OnForRuntime so we can use it to clear the GC Gallback in Off().
git-svn-id: svn://10.0.0.236/trunk@98152 18797224-902f-48f8-a5cc-f745e15eee43
add enumerateScripts() method to jsdIDebugger service
don't create a new debugger context if the user trys to call on[ForRuntime]() multiple times
git-svn-id: svn://10.0.0.236/trunk@95587 18797224-902f-48f8-a5cc-f745e15eee43
add onForRuntime() method to jsdIDebuggerService to let native code turn on the deubgger (on() can only be called from js)
register an app-start observer so we can turn on the debugger at startup if "js.debugger.autostart" pref is true.
r=peterv, bug 81840
git-svn-id: svn://10.0.0.236/trunk@95583 18797224-902f-48f8-a5cc-f745e15eee43
check for valid PC in ClearBreakpoint
Lock script subsystem while clearing all breakpoints
git-svn-id: svn://10.0.0.236/trunk@94823 18797224-902f-48f8-a5cc-f745e15eee43
remove jsdThreadState and jsdContext objects.
consolidate ExecutionHook and BreakpointHook callbacks
remove return value checking from all methods (xpconnect does this for us.)
git-svn-id: svn://10.0.0.236/trunk@93975 18797224-902f-48f8-a5cc-f745e15eee43
validate integrity of jsdScript data to guard against calling into a destroyed script.
queue up script deletes that happen during the JS GC cycle, call them when GC finishes (bug 76979.)
don't NS_IF_ADDREF objects that we get using *::FromPtr()
add jsdScript::Invalidate()
git-svn-id: svn://10.0.0.236/trunk@93560 18797224-902f-48f8-a5cc-f745e15eee43
register for gc callbacks
explicitly create a singleton factory
only call OnScriptLoaded() if |creating| is true
git-svn-id: svn://10.0.0.236/trunk@93089 18797224-902f-48f8-a5cc-f745e15eee43
add global service for the breakpoint callback
add breakpoint callback
move c callbacks to top of source
add creatorURL, creatorLine, constructorURL, constructorLine, and value attribut
es to jsdObject
move from pc as a ulong to pc as an object wrapped around a uword
move lineToPc and pcToLine from ThreadState to Script (where it belongs)
add setBreakpoint(), clearBreakpoint(), and clearAllBreakpoints() to jsdScript
relocate jsdService constructor from jsd_xpp.h in order to initialize the global
service
rename init() to on() on jsdService
add off(), clearAllBreakpoints(), and breakpointHook attribute to jsdService
git-svn-id: svn://10.0.0.236/trunk@93076 18797224-902f-48f8-a5cc-f745e15eee43
add line attribute to jsdStackFrame
add pcToLine and lineToPc methods to jsdStackFrame
remove isFunction attribute from jsdValue (already covered by jsType attribute)
add propertyCount attribute to jsdValue so you can get the property cound without forcing a bunch of property wrappers to be created (as in GetProperties())
git-svn-id: svn://10.0.0.236/trunk@92934 18797224-902f-48f8-a5cc-f745e15eee43
