Bug 1750959 Comment 6 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to Olli Pettay [:smaug] from comment #5)
> Are you talking about nsHTMLAnchorElement::SetFocus?

Oh, I'm sorry, I might have been confused. According to attachment 382498 [details] [diff] [review] (`nsFocusManager` creation), [`nsHTMLButtonElement::SetFocus` used `nsGenericHTMLElement::Focus`](https://bugzilla.mozilla.org/attachment.cgi?id=382498&action=diff#a/content/html/content/src/nsHTMLButtonElement.cpp_sec4). And [`nsGenericHTMLElement::Focus` still used `NS_PRESSHELL_SCROLL_IF_NOT_VISIBLE`](https://bugzilla.mozilla.org/attachment.cgi?id=257453&action=diff#content/base/src/nsGenericElement.cpp_sec2). So I need to check the history of `nsGenericElement::Focus` more.
Revision 1 by
on 
(In reply to Olli Pettay [:smaug] from comment #5)
> Are you talking about nsHTMLAnchorElement::SetFocus?

Oh, I'm sorry, I might have been confused. According to attachment 382498 [details] [diff] [review] (`nsFocusManager` creation), [`nsHTMLButtonElement::SetFocus` used `nsGenericHTMLElement::Focus`](https://bugzilla.mozilla.org/attachment.cgi?id=382498&action=diff#a/content/html/content/src/nsHTMLButtonElement.cpp_sec4). And [`nsGenericHTMLElement::Focus` still used `NS_PRESSHELL_SCROLL_IF_NOT_VISIBLE`](https://bugzilla.mozilla.org/attachment.cgi?id=257453&action=diff#content/base/src/nsGenericElement.cpp_sec2). So I need to check the history of `nsGenericHTMLElement::Focus` more.

Back to Bug 1750959 Comment 6