Created attachment 647892 [details] [diff] [review] Patch The imgStatusTracker field here is used as a PRUint32 by all callers that I see. It's evidently meant to contain the values of an anonymous enum member of imgIRequest, so PRUint32 is the right type until someone gives it a name. I'm not totally sure that the nsUrlClassifierStore change doesn't need module peer review -- it might be unintentional that it returns NS_OK.
Comment on attachment 647892 [details] [diff] [review] Patch (No part 2 intended here.)
5 years ago