There are a number of places in the dom code where we handle Observe events that contain wrapped JSObjects that get converted to Dictionaries. With a better cx-less WrappedJSToDictionary function in bug 1107673 all of that code should be cleaned up to use the new function.

fixes up all of the uses of WrappedJSToDictionary in the dom code.

try pass: http://mzl.la/1x218Va

The try pass is clean. The only failures are failures in the test framework. Re-running to see if they go away.

Comment on attachment 8541032 [details] [diff] [review] Bug_1107681.patch Review of attachment 8541032 [details] [diff] [review]: ----------------------------------------------------------------- This looks great! However, I have a few ergonomic changes I'd like to see: * I don't want to define a local called "cx" here, because that might lead somebody to accidentally using it for non-rooting purposes. So the call should be inlined directly into the rooting constructor. * Given that this seems to be used frequently, I think it's worth introducing a call that indicates that this is explicitly for rooting, and shaves off a few characters to boot. How about nsContentUtils::RootingCX() and nsContent::ThreadsafeRootingCX() which would be inline aliases for GetSafeJSContext and ThreadsafeGetSafeJSContext respectively? nsContentUtils::Root

updated to make review changes. try pass looks clean: http://mzl.la/1zJ6fvW

Comment on attachment 8541861 [details] [diff] [review] Bug_1107681.patch Review of attachment 8541861 [details] [diff] [review]: ----------------------------------------------------------------- This looks great, assuming all of this stuff runs on the main thread. r=me Thanks for doing this!

hold off for a moment while I double check that all of these are on the main thread.

Alright, I went through and every place I couldn't guarantee that we were executing on the main thread, I changed the call to RootingCxForThread(). The try pass is here: http://mzl.la/1BfsxCD If that comes back clean, then this can be checked in. FYI, RootingCxForThread is an alias for GetDefaultJSContextForThread which returns RootingCx if on the main thread otherwise it returns the current thread JS context.

Ok, the try pass is clean. This can go in.

Dave, stuff generally only gets resolved when it merges to mozilla-central (and it happens automatically).

k thanks.