501656 - selected state is not exposed on selected HTML table cell

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

selected state is not exposed on selected HTML table cell

Comment 1

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 2

[Marco Zehe (:MarcoZ)]

Comment on attachment 386660 [details] [diff] [review]
patch

r=me

Comment 3

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 386660 [details] [diff] [review]
patch

1 nit:

>+  nsCOMPtr<nsIContent> content = do_QueryInterface(mDOMNode);
>+  nsCOMPtr<nsIPresShell> presShell = do_QueryReferent(mWeakShell);
>+  nsIFrame *frame = presShell->GetPrimaryFrameFor(content);
>+  NS_ASSERTION(frame, "No frame for valid cell accessible!");

We should probably avoid using NS_ASSERTION in new code, because there is so much console output already it is almost useless. Do you think NS_ABORT_IF_FALSE is maybe too strong here?

Either way r=me (the rest looks good).

Comment 4

[alexander :surkov (:asurkov)]

(In reply to comment #3)
> (From update of attachment 386660 [details] [diff] [review])
> 1 nit:
> 
> >+  nsCOMPtr<nsIContent> content = do_QueryInterface(mDOMNode);
> >+  nsCOMPtr<nsIPresShell> presShell = do_QueryReferent(mWeakShell);
> >+  nsIFrame *frame = presShell->GetPrimaryFrameFor(content);
> >+  NS_ASSERTION(frame, "No frame for valid cell accessible!");
> 
> We should probably avoid using NS_ASSERTION in new code, because there is so
> much console output already it is almost useless. Do you think
> NS_ABORT_IF_FALSE is maybe too strong here?

It might be. Though I use assertion here because I'm sure it shouldn't be happen. Nevertheless I would avoid to use ns_abort :). The fact we have heavy output in console isn't related with assertion usage :) But true we should fix these things.

Comment 5

[alexander :surkov (:asurkov)]

checked in on 1.9.2, http://hg.mozilla.org/mozilla-central/rev/7db7d2d7dbac