Variables that indicate whether a node was script-created or parser-created (and how exactly parser-created) should use an enum instead of PRUint32 and these #defines: http://mxr.mozilla.org/mozilla-central/source/content/base/public/nsContentCreatorFunctions.h#59
Created attachment 483289 [details] [diff] [review] Patch v1
Comment on attachment 483289 [details] [diff] [review] Patch v1 r=hsivonen. I noticed that there were some clean-up bits that weren't strictly speaking fixing this bug, but all those clean-ups looked good to me. Thanks for fixing this! Since I'm not officially qualified to act as a reviewer here, I'm requesting sr.
Comment on attachment 483289 [details] [diff] [review] Patch v1 Looks fine.
Attachment #483289 - Flags: superreview?(bzbarsky) → superreview+
Comment on attachment 483289 [details] [diff] [review] Patch v1 Requesting approval for 2.0. This is a low-risk patch that'll bitrot extremely quickly. If we don't land this soon, it'll bitrot. OTOH, landing it soon would be harmless.
Landed with a=jonas as a dependency for bug 604924. http://hg.mozilla.org/mozilla-central/rev/6aad432efa02
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Doh. I meant bug 604660.
You need to log in before you can comment on or make changes to this bug.