Active Accessibility: Support STATE_ANIMATED for images

RESOLVED FIXED

Status

SeaMonkey
General
RESOLVED FIXED
17 years ago
13 years ago

People

(Reporter: Aaron Leventhal, Assigned: Aaron Leventhal)

Tracking

({access})

Trunk
x86
Windows 2000
access
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: Looking for sr=)

Attachments

(2 attachments)

(Assignee)

Description

17 years ago
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;
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Keywords: access
(Assignee)

Updated

17 years ago
Depends on: 93044
(Assignee)

Comment 1

17 years ago
Created attachment 45710 [details] [diff] [review]
Patch fixes STATE_ANIMATED
(Assignee)

Updated

17 years ago
Whiteboard: Looking for r=/sr=

Comment 2

17 years ago
r=jgaunt
Whiteboard: Looking for r=/sr= → Looking for sr=
(Assignee)

Comment 3

17 years ago
Created attachment 45720 [details] [diff] [review]
Patch #2 - gets rid of unnecessary if before QI (can QI null and get null)

Comment 4

17 years ago
sr=scc
(Assignee)

Comment 5

17 years ago
fix checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Updated

17 years ago
Blocks: 95819
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.