Open Bug 1263564 Opened 8 years ago Updated 2 years ago

ImageCacheKey::GetControlledDocumentToken ignores an ErrorResult

Categories

(Core :: Graphics: ImageLib, defect, P3)

Product:
Core
Component:
Graphics: ImageLib
Type:
defect

Tracking

()

People

(Reporter: Ms2ger, Unassigned)

Details

(Whiteboard: [gfx-noted]) 

I think that's kinda bad, though IsControlled only throws plain nsresults. Paging bz to confirm.
Flags: needinfo?
Paging for real
Flags: needinfo? → needinfo?(bzbarsky)
Well, if we only throw plain nsresults then we won't leak on failure, at least.  We'll still assert.

This should either explicitly ignore (via IgnoredErrorResult) or handle somehow.
Flags: needinfo?(bzbarsky)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.