Created attachment 647892 [details] [diff] [review]
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]
(No part 2 intended here.)