Closed Bug 1179081 Opened 11 years ago Closed 11 years ago

ContentEventHandler should use root content's primary frame to retrieve focused widget when there is no caret

Categories

(Core :: DOM: UI Events & Focus Handling, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

If widget dispatches WidgetQueryContentEvent when no editor has focus, ContentEventHandler tries to query whole contents of the document. However, in such case, there may not be caret. Then, it fails to initialize itself due to here: http://mxr.mozilla.org/mozilla-central/source/dom/events/ContentEventHandler.cpp?rev=4ffb0322905d&mark=123-124#105 In such case, we should use the root content's primary frame to retrieve the focused widget.
Attached patch PatchSplinter Review
See comment 0 for the detail.
Attachment #8628192 - Flags: review?(bugs)
Attachment #8628192 - Flags: review?(bugs) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: