Jesse Ruderman Reporter
	Description • 10 years ago

Assertion failure: editingHost == mEditableNode (found editing host should be mEditableNode), at dom/events/IMEContentObserver.cpp:122

This assertion was added in http://hg.mozilla.org/mozilla-central/rev/2cfc400dd0ad (bug 496360 part 2)

	Masayuki Nakano [:masayuki] (he/him)(JST, +0900) Assignee
	Comment 2 • 10 years ago

I'm trying to fix this...

	Masayuki Nakano [:masayuki] (he/him)(JST, +0900) Assignee
	Comment 4 • 10 years ago

This bug tells us another bug. Even if an <input> element or a <textarea> element in contenteditable element is focused, IMEContentObserver observes its ancestor (i.e., editing host of the contenteditable editor). Of course, when <input> or <textarea> has focus, IMEContentObserver should observe the focused editor.

Strictly speaking, this patch is NOT correct. This patch checks if found editable node in GetRootEditableNode() has independent selection. Currently, looks like <input>, <textarea> and <select> has independent selection. However, <select> returns false for IsEditable(). Therefore, this currently works.

I think that we should create stabler API for this. Perhaps, in nsIEditor? Then, focused editor should return its editable root content or node.

If you'd like me to fix this strictly here, I'll try to do it.

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 5 • 10 years ago

Comment on attachment 8494923 [details] [diff] [review]
part.1 Stop searching root editable node if a node has independent selection


>+      // If the node has independent selection like <input type="text"> or
>+      // <textarea>, the node should be the root editable node for aContent.
>+      // FYI: <select> element also has independent selection but IsEditable()
>+      //      returns false.
>+      // XXX: If somebody will add new editable element which has independent
>+      //      selection but doesn't own editor, we need more check here.
if somebody adds ...we'll need more checks here.



I think I can live with this patch for now :)

	Masayuki Nakano [:masayuki] (he/him)(JST, +0900) Assignee
	Comment 6 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/2bfce4ae91e6
https://hg.mozilla.org/integration/mozilla-inbound/rev/049f2e6b043d

Thanks!

	Carsten Book [:Tomcat]
	Comment 7 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/2bfce4ae91e6
https://hg.mozilla.org/mozilla-central/rev/049f2e6b043d