93016 - Active Accessibility: Support STATE_ANIMATED for images

Status: RESOLVED FIXED

(SeaMonkey :: General, defect)

Description

[Aaron Leventhal]

A screen reader vendor asked us to support the animated state for images.
I need a checkin from Pavlov for this to work (no known bug #).
Pavlov's checkin added GetImageRequest to nsIImageFrame.

Here's the untested code I have so far, that should work once we get the checkin:

  nsCOMPtr<nsIImageFrame> imageFrame;
  if (frame) 
    imageFrame = do_QueryInterface(frame));

  nsCOMPtr<imgIRequest> imageRequest;
  if (imageFrame) {
    imageFrame->GetImageRequest(getter_AddRefs(imageRequest));
  }
  
  nsCOMPtr<imgIContainer> imgContainer;
  if (imageRequest) 
    imageRequest->GetImage(getter_AddRefs(imgContainer));

  if (imgContainer && imgContainer->GetNumFrames()>1)
    *_retval |= STATE_ANIMATED;

Comment 1

[Aaron Leventhal]

Attachment: Patch fixes STATE_ANIMATED

Comment 2

[John Gaunt (redfive)]

r=jgaunt

Comment 3

[Aaron Leventhal]

Attachment: Patch #2 - gets rid of unnecessary if before QI (can QI null and get null)

Comment 4

[Scott Collins]

sr=scc

Comment 5

[Aaron Leventhal]

fix checked in