Created attachment 651694 [details] [diff] [review]
mBroken is an nsresult, and IsBroken() returns an nsresult. Testing them as though they were booleans won't compile when nsresult is an enum class, even though it will work fine as long as you don't use success codes other than NS_OK. So NS_FAILED/NS_SUCCEEDED need to be used here.
Green try: https://tbpl.mozilla.org/?tree=Try&rev=da4047717ef7