1090032 - input field in OOP mozbrowser iframes can't receive focus in Fx Desktop mochitest

Status: NEW

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

Description

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: imapi-test-oop.patch

STR:

1. Apply attachment and turn on oop perfs (prefs copied from dom/browser-element/mochitest/)
2. Try to |./mach mochitest-plain dom/inputmothod/mochitest/

Expected:

1. The same tests should work

Actual:

1. By observing |ps ax| and dump I can see the remote frame loads, forms.js loaded too but when the test frame script call input1.focus() it doesn't not generate a focus event.

Note:

The remote frame has no rendering, see bug 1085217.

Comment 1

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Hi :smaug,

I need some pointer here and maybe some opinion on if it actually make sense to fix the issue in the mochitest environment. 

We can always protect the API with Gaia int tests but those tests are depend on Gaia user interfaces, therefore I think having a complete mochitest test suite is better than that, but I don't know how much effort it would take.

Thanks!

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

I don't understand the changes to inputmethod_common.js.
Where is that run? And when.

And are we trying to run oop iframes within oop tabs?
(we have too many prefs to enable various ipc stuff.)

Anyhow, it sounds like we should support focus() taking focus to oop iframe assuming focus isn't currently in browser chrome.

Comment 3

[Tim Guan-tin Chien [:timdream] (please needinfo)]

(In reply to Olli Pettay [:smaug] from comment #2)
> I don't understand the changes to inputmethod_common.js.
> Where is that run? And when.

inputmethod_common.js is run in every mochitest page, for example
http://dxr.mozilla.org/mozilla-central/source/dom/inputmethod/mochitest/test_two_inputs.html#18-20

> And are we trying to run oop iframes within oop tabs?
> (we have too many prefs to enable various ipc stuff.)

No. e10s is not enabled in those tests.

> Anyhow, it sounds like we should support focus() taking focus to oop iframe
> assuming focus isn't currently in browser chrome.

Hum. That probably make sense. I suppose when the mochitests are run the focus should be on the mochitest page. I did try to write a test that place the focus on an input in the oop mozbrowser iframe and move the focus of the main page away from the iframe and back, but the input is not re-focused. Maybe this quirk of mozbrowser is related.

Comment 4

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.