From 0f082bced5686eff8b3cb1de679a870f9138cb16 Mon Sep 17 00:00:00 2001 From: Seth Fowler Date: Tue, 25 Nov 2014 00:00:00 -0800 Subject: [PATCH] Bug 1103328 - Tweak wording of comment for Decoder::PostHasTransparency. r=me DONTBUILD --- image/src/Decoder.h | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/image/src/Decoder.h b/image/src/Decoder.h index c29f26eef02b..0ae05af5408b 100644 --- a/image/src/Decoder.h +++ b/image/src/Decoder.h @@ -217,11 +217,13 @@ protected: // Called by decoders if they determine that the image has transparency. // // This should be fired as early as possible to allow observers to do things - // that affect content, so it's necessarily pessimistic. PostFrameStop's - // aFrameAlpha argument, on the other hand, is only used internally to - // ImageLib. Because PostFrameStop isn't delivered until the entire frame has - // been decoded, decoders may take into account the actual contents of the - // frame. + // that affect content, so it's necessarily pessimistic - if there's a + // possibility that the image has transparency, for example because its header + // specifies that it has an alpha channel, we fire PostHasTransparency + // immediately. PostFrameStop's aFrameAlpha argument, on the other hand, is + // only used internally to ImageLib. Because PostFrameStop isn't delivered + // until the entire frame has been decoded, decoders may take into account the + // actual contents of the frame and give a more accurate result. void PostHasTransparency(); // Called by decoders when they begin a frame. Informs the image, sends