Consolidate *Element::SetFocus()

RESOLVED WORKSFORME

Status

()

Core
Event Handling
--
enhancement
RESOLVED WORKSFORME
13 years ago
a month ago

People

(Reporter: mats, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

13 years ago
Just so it's not lost... in bug 303620 comment 13, Aaron wrote:

I wonder why we really need separate impls for nsHTMLAnchorElement::SetFocus()
and nsHTMLAreaElement::SetFocus() apart from nsGenericHTMLElement::SetFocus(). I
see that they FlushPendingNotifications(Flush_Layout) which seems like it should
be done for everything or not at all. The anchor impl has checks for a link
handler but the generic SetFocus() already does a better IsFocusable().
Assignee: events → nobody
QA Contact: ian → events
This seems can be resolved at Bug 778654.

Comment 2

a month ago
Not relevant anymore.
Status: NEW → RESOLVED
Last Resolved: a month ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.