315454 - Find as you Type is not disabled when an editable <iframe> is added to a document using the DOM

Status: RESOLVED WORKSFORME

(Core :: DOM: Editor, defect, P5)

Description

[Tommy Morgan]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.12) Gecko/20050915 Firefox/1.0.7
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.12) Gecko/20050915 Firefox/1.0.7

When you add an <iframe> using DOM methods, find as you type is not disabled.  If the <iframe> is manually written into the page and you simply use script to enable designMode, find as you type is correctly disabled.

Reproducible: Always

Steps to Reproduce:
1. Create a simple html document.
2. Add a script that will create an <iframe> using document.createElement
3. Insert the new node using someNode.appendChild(newNode)
4. Enable designMode on the new <iframe>

Actual Results:  
A somewhat functional editor is created, but find as you type is still enabled.

Expected Results:  
Find as you type should be disabled.

Comment 1

[Tommy Morgan]

Attachment: Simple testcase that illustrates the problem

Comment 2

[Tommy Morgan]

Attachment: Testcase that creates an editor without using the DOM and has no FAYT issues

Comment 3

[Martijn Wargers (dead)]

With the first testcase, designMode isn't working at all.
You probably need to write this code in a separate function:

function doe(){
editor.contentWindow.document.designMode = "on";
}
and then call doe in a timer inside init():
setTimeout(doe,500);

Then it should work.

The problem is that the iframe is not really 'ready' to set designMode on.

Comment 4

[Tommy Morgan]

(In reply to comment #3)
> With the first testcase, designMode isn't working at all.

I've noticed that sometimes when I load that testcase designMode isn't enabled.  I don't know why that is.  I threw the testcase together very quickly.  If you reload the page designMode will be enabled and you can see the faulty behavior I'm talking about.  I'll see if I can create a better testcase.

Comment 5

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=5 years untouched DOM/Storage bugs' priority.

If you have reason to believe this is wrong (especially for the severity), please write a comment and ni :jstutte.

Comment 6

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

See the testcase: https://jsfiddle.net/d_toybox/gq5v8wf7/show

designMode document does not steal focus. Therefore, FAYT should keep being ready until the sub-document gets focus.

-> WFM