timeless@mozdev.org
|
74b458a796
|
Bug 421303 Crash [@ jsds_ScriptHookProc] r=caillon a=dsicore If we reach ~jsdService, that means our client doesn't care about us, so we can (and should) drop all references to any callbacks (if they cared, they'd have kept us alive!*). I think jsdService::Off should clear all the hooks, the strange magic of not clearing it isn't really a great idea. So for Off, we'll now clear the ScriptHook too (consumers who use off should really drop any references they have to our objects...). I'm still on the fence on this point, I suspect we can actually move it from ::Off to ~jsdService (it must be cleared at some point, otherwise if jsd_xpc's library manages to get unloaded, the function pointer would be invalid, which would be *BAD*). jsds_NotifyPendingDeadScripts needs to clear gDeadScripts whether or not there's a service or hooks, so it does. Because it's a static callback and because of the scary way GC works, I'd rather ensure (deathgrip) that jsds is available (and consistent!) for the duration of the function call. The code already handles the lack of a hook, so there's no reason to do magical returns.... The real problem which mayhemer found was that jsdService::Off was returning early (failure) because gGCStatus wasn't JSGC_END when called from ~jsdService from JS_GC from the cyclecollector, so we make sure that ~jsdService forces ::Off to act as if it is JSGC_END (after ensuring that there are no callbacks available). * a pure javascript (xpcom component, not DOM hosted!) version of a jsdService consumer means that jsdService will need to talk to the CycleCollector eventually (this is another bug for the future).
|
2008-03-10 17:13:48 -07:00 |
|
timeless@mozdev.org
|
230fba1b62
|
Bug 282660 Crash [@ jsds_NotifyPendingDeadScripts] ds->script is null r=jst a=beltzner
|
2008-03-05 13:10:01 -08:00 |
|
timeless@mozdev.org
|
a7cd3c0106
|
Bug 394114 Interfaces missing from various QI implementations. jsdContext r=caillon sr=jag a=dsicore
|
2008-02-26 07:10:19 -08:00 |
|
timeless@mozdev.org
|
3be3d64ce2
|
Bug 405025 ASSERT_VALID_LOCK failed r=gijs a=dsicore
|
2008-02-26 07:07:05 -08:00 |
|
timeless@mozdev.org
|
9fce33202b
|
Bug 405025 ASSERT_VALID_LOCK failed r=gijs a=dsicore
|
2008-02-26 07:04:13 -08:00 |
|
timeless@mozdev.org
|
b6c5d2862f
|
Bug 416293 unbalanced locking in jsd_SetExecutionHook r=crowder a=mtschrep
|
2008-02-09 20:16:54 -08:00 |
|
timeless@mozdev.org
|
ebdf2bd20b
|
Bug 411249 Top crash [@ jsds_ScriptHookProc][@jsds_ScriptHookProc(JSDContext*, JSDScript*, int, void*)] on shutdown with Firebug 1.1.0b10 installed r=gijs a=beltzner
|
2008-01-31 11:12:47 -08:00 |
|
benjamin@smedbergs.us
|
dfc4cee45d
|
Bug 411327 - nsIXPCNativeCallContext should not inherit from nsISupports, r=mrbkap, a=schrep
|
2008-01-15 07:50:57 -08:00 |
|
timeless@mozdev.org
|
33d48a27ec
|
Bug 336991 Useless null check of jsdscript in _destroyJSDScript r=gijs sr=bz a=dsicore
|
2007-12-23 15:47:36 -08:00 |
|
jwalden@mit.edu
|
e3c4baccae
|
Bug 348748 - Replace all instances of NS_STATIC_CAST and friends with C++ casts (and simultaneously bitrot nearly every patch in existence). r=bsmedberg on the script that did this. Tune in next time for Macro Wars: Episode II: Attack on the LL_* Macros.
|
2007-07-08 00:08:04 -07:00 |
|
timeless@mozdev.org
|
90c5be45ec
|
Bug 385765 [@ jsdASObserver::Observe] You can't dereference a NULL nsCOMPtr with operator->()
r=dveditz sr=dveditz
|
2007-07-01 12:25:10 -07:00 |
|
timeless@mozdev.org
|
3da1de5e09
|
Bug 365363 Return value for GetJSDValue ignored leading to death
r=silver sr=biesi
|
2007-06-19 22:56:13 -07:00 |
|
igor@mir2.org
|
d69de1f698
|
Bug 379165: Simplifing JS_DimpHeap while fixing BeOS build problems. r=brendan
|
2007-04-29 14:49:00 -07:00 |
|
igor@mir2.org
|
68d7b74a96
|
Bug 378261: Replacing GC_MARK_DEBUG by DumpHeap. r=brendan
|
2007-04-25 06:43:18 -07:00 |
|
hg@mozilla.com
|
465265d0d4
|
Free the (distributed) Lizard! Automatic merge from CVS: Module mozilla: tag HG_REPO_INITIAL_IMPORT at 22 Mar 2007 10:30 PDT,
|
2007-03-22 10:30:00 -07:00 |
|