Debugged this a little with :asurkov today (thank you!) and found we're not calling `SelectionManager::NotifySelectionChanged` even though we are calling `EditorBase::NotifySelectionChanged` meaning after we set our document listener it's being cleared. In my testcase, I see two selection listeners when I get here (https://searchfox.org/mozilla-central/rev/be7d1f2d52dd9474ca2df145190a817614c924e4/dom/base/Selection.cpp#3156), but in the "working" case where we move focus from the address bar and come back to it, there are three listeners. I'm having a hard time telling what the listeners _are_ because of all our generics, but I think the missing listener is the problem.
Bug 1590216 Comment 16 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Debugged this a little with :asurkov today (thank you!) and found we're not calling `SelectionManager::NotifySelectionChanged` even though we are calling `EditorBase::NotifySelectionChanged` meaning after we set our document listener it's being cleared. In my testcase, I see two selection listeners when I get here (https://searchfox.org/mozilla-central/rev/be7d1f2d52dd9474ca2df145190a817614c924e4/dom/base/Selection.cpp#3156), but in the "working" case where we move focus from the address bar and come back to it, there are three listeners. Looks like both versions register a caret and text editor as a listener, but the working version also registers a `nsAccessibilityService`.