Created attachment 622918 [details]
1. Load the testcase.
2. Quit Firefox.
Result: An assertion fails during cycle collection.
###!!! ASSERTION: mRequiredRadioCount about to wrap below 0!: 'radioGroup->mRequiredRadioCount != 0', file content/base/src/nsDocument.cpp, line 6617
This is a "regression" from bug 750370 (before which the assertion was a noop).
Created attachment 622919 [details]
Created attachment 623232 [details] [diff] [review]
I have no idea why |GetParent()| was checked before.
Unfortunately, we need to trigger the GC to write a test and the only tests that would fail with asserts are reftests/crashtests which don't allow GC to be triggered AFAIK.
Comment on attachment 623232 [details] [diff] [review]
This patch just rearranges the comments and moves the 'notify' init. Based on comment 2. shouldn't something happen here with GetParent() ?
Created attachment 623434 [details] [diff] [review]
Arg... I changed the patch to see what I could do to test it and I forgot to revert that change.
This is the real patch I wanted to attach ;)
Comment on attachment 623434 [details] [diff] [review]
That makes way more sense. r=me