The Seamonkey folks over in bug 677905 found two bugs in my recent changes which were checked in in bug 602718. My code calls document.getElementById on a potentially null value, and in one location where I copied and modified some code I forgot to change AllowHTML_menuitem to AllBodyParts_menuitem. D'oh! The attached patch fixes both issues.
Assignee: nobody → jik
Status: NEW → ASSIGNED
Comment on attachment 552436 [details] [diff] [review] patch to fix both minor coding issues looks good, thx.
Attachment #552436 - Flags: review?(dbienvenu) → review+
Comment on attachment 552436 [details] [diff] [review] patch to fix both minor coding issues I think this needs to go into the Aurora branch as well or it'll be broken in TB8 when it's released. I think I set approval-comm-aurora? to request that? Is that right?
Attachment #552436 - Flags: approval-comm-aurora?
Yes, now that 8.0 migrated to the aurora channel it needs to land there as well. Your requests are fine as set, someone should take care of both from here.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 9.0
Attachment #552436 - Flags: approval-comm-aurora? → approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.