Comment on attachment 409915 [details] [diff] [review] v1 At the very least, this will clean up your console some :-).
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.
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.