Comment on attachment 409915 [details] [diff] [review] v1 At the very least, this will clean up your console some :-).
Attachment #409915 - Flags: review?(mrbkap) → review+
Created attachment 410194 [details] [diff] [review] v1.1 Bah, that wasn't enough because we call wrapper->NeedsChromeWrapper() further down. If a wrapper is a slim wrapper and we morph then NeedsChromeWrapper would return false anyway, so checking for a non-null wrapper first should be fine.
Attachment #410194 - Flags: review?(mrbkap) → review+
Comment on attachment 410194 [details] [diff] [review] v1.1 I can't wait until my patch that renames NeedsChromeWrapper -> WantsSOWs lands. Sorry for missing this before.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
You need to log in before you can comment on or make changes to this bug.