Created attachment 367053 [details] [diff] [review] patch 1 We should not use NS_ENSURE_STATE in methods that return PRBool. (Caught by Taras Glek's analysis tool)
Attachment #367053 - Flags: review?(marco.zehe) → review+
Is this the only spot where we do this, or are there other places in our code that do this and need fixing?
We should get automatic emails from Taras' tool if we do this elsewhere.
Attachment #367053 - Flags: review?(surkov.alexander) → review+
Pushed on Davidb's behalf in changeset: http://hg.mozilla.org/mozilla-central/rev/9bb687517490
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.