170554 - HTMLIFrameElement does not have a focus() method

Status: RESOLVED WORKSFORME

(Core :: DOM: Core & HTML, defect)

Description

[Mason]

When I call focus() on an iFrame an exception is thrown.  This doesn't seem like
it should occur as the underlying XUL layer can call focus and blur on iframes.
 When I add focus and blur functions, they too are never called.  Sounds like a
bug with iframes.  I will attach a sample

Comment 1

[Mason]

Attachment: Test Case

This works in IE and throws an Exception in Mozilla 1.1

Comment 2

[Boris Zbarsky [:bzbarsky]]

See http://www.w3.org/TR/2002/CR-DOM-Level-2-HTML-20020605/html.html#ID-50708718
-- iframe elements do not have a focus() method in the standard DOM.

Over to DOM HTML to decide whether the W3C cares to adjust the spec here or not.

Comment 3

[Mason]

Workaround:
instead of calling 'select.add( option )', call 'select[select.length] = option'.
Seems to work for the time being.

Comment 4

[Mason]

Ignore last comment, wrong bug number

Comment 5

[Boris Zbarsky [:bzbarsky]]

How about undoing the Severity change you made too, eh?

Comment 6

[Mike Kaply [:mkaply]]

I don't believe you are supposed to be focusing the iframe, you are supposed to
be focusing the iframe contentWindow.

So:

document.getElementById("iframename").contentWindow.focus();

Of course, this doesn't work either, and it has serious implications for Midas
(editor within Mozilla).

You can't focus an iframe when it is in edit mode.

Comment 7

[Mike Kaply [:mkaply]]

Attachment: New testcase

This testcase is timing based and easier to see.

After five seconds, focus should move to the iframe. It doesn't.

Works in IE and should work in Mozilla.

You get no errors in the Javascript console.

Note the fact that the IFRAME is in designMode has nothing to do with this. I
did that so it would be more obvious that focus switched to the IFRAME.

Comment 8

[Mike Kaply [:mkaply]]

Attachment: Sorry, attaching the 5 second version to be more clear

Ooops - previous test was 10 second, not 5 so text in testcase didn't match.

Comment 9

[Johnny Stenback (:jst)]

Mass-reassigning bugs.

Comment 10

[Anne (:annevk)]

The test case worksforme in Deer Park Alpha 2. Marking so.