1678349 - Additional blur and focus event get fired while calling iframe.focus() on OOP iframe

Status: RESOLVED FIXED

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

Description

[Edgar Chen [:edgar]]

This is another case that we trigger additional blur and focus events on the OOP iframe.
I will attach test script later.

Comment 1

[Edgar Chen [:edgar]]

Attachment: test.html

STR:

1. enable Fission
2. load test.html
3. wait for a few seconds.

Expected result:
No duplicated focus or blur event

Actual Result:
Duplicated focus and blur event on the iframe and framer.

Comment 2

[Chris Peterson [:cpeterson]]

Tracking for Fission Beta milestone (M7).

Comment 3

[Edgar Chen [:edgar]]

Attachment: Bug 1678349 - Part 1: Compute focus-in-other-content-process flag in nsFocusManager::Focus;

This simplify the code a bit and also fix FocusIsOutOfProcess isn't set correctly
for a certain case, e.g. framer calls iframe.focus() on a OOP iframe.

Comment 4

[Edgar Chen [:edgar]]

Attachment: Bug 1678349 - Part 2: Suppress additional blur/focus event on OOP iframe;

for the case that framer calls iframe.focus() on OOP iframe and the OOP iframe
already has the focus.

Comment 5

[Edgar Chen [:edgar]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=579f82c789a36a2655481e7f5d159c51422bf466

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ef077201b2a4
Part 1: Compute focus-in-other-content-process flag in nsFocusManager::Focus; r=hsivonen
https://hg.mozilla.org/integration/autoland/rev/c8f03513e3a5
Part 2: Suppress additional blur/focus event on OOP iframe; r=hsivonen

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/26717 for changes under testing/web-platform/tests

Comment 8

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/ef077201b2a4
https://hg.mozilla.org/mozilla-central/rev/c8f03513e3a5

Comment 9

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot