Send orientation change events to sub frames even if parent isn't active.

Send orientation change events to sub frames even if parent isn't active.

When a frame receive an orientatio change notification, it only carry
the notification over if it is active but the iframe might be listening
to the event and not the frame.

Unfortunately, the test checking this was subtly broken because it
initialised itself by looking for the current orientation, thus
initialising the top frame controller.

BUG=682441
Review-Url: https://codereview.chromium.org/2661663003
Cr-Commit-Position: refs/heads/master@{#446834}
Committed: https://chromium.googlesource.com/chromium/src/+/1c17214f719b1e715aa6c64848c87176c2ac90b0

[mlamouri (slow - plz ping), 2017-01-27 20:19:11]

The CQ bit was checked by mlamouri@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-27 20:20:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mlamouri (slow - plz ping), 2017-01-27 20:20:31]

mlamouri@chromium.org changed reviewers:
+ eae@chromium.org

[mlamouri (slow - plz ping), 2017-01-27 20:20:32]

PTAL :)

[eae, 2017-01-27 20:21:12]

Thanks for fixing this
LGTM

[commit-bot: I haz the power, 2017-01-27 23:24:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-27 23:24:47]

Dry run: This issue passed the CQ dry run.

[mlamouri (slow - plz ping), 2017-01-27 23:35:57]

The CQ bit was checked by mlamouri@chromium.org

[commit-bot: I haz the power, 2017-01-27 23:36:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-27 23:45:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485560157617130, "parent_rev":
"f8206ba1cdddee202d83e2c468aef524da8ede4b", "commit_rev":
"1c17214f719b1e715aa6c64848c87176c2ac90b0"}

[commit-bot: I haz the power, 2017-01-27 23:46:10]

Description was changed from

==========
Send orientation change events to sub frames even if parent isn't active.

When a frame receive an orientatio change notification, it only carry
the notification over if it is active but the iframe might be listening
to the event and not the frame.

Unfortunately, the test checking this was subtly broken because it
initialised itself by looking for the current orientation, thus
initialising the top frame controller.

BUG=682441
==========

to

==========
Send orientation change events to sub frames even if parent isn't active.

When a frame receive an orientatio change notification, it only carry
the notification over if it is active but the iframe might be listening
to the event and not the frame.

Unfortunately, the test checking this was subtly broken because it
initialised itself by looking for the current orientation, thus
initialising the top frame controller.

BUG=682441

Review-Url: https://codereview.chromium.org/2661663003
Cr-Commit-Position: refs/heads/master@{#446834}
Committed:
https://chromium.googlesource.com/chromium/src/+/1c17214f719b1e715aa6c64848c8...
==========

[commit-bot: I haz the power, 2017-01-27 23:46:11]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/1c17214f719b1e715aa6c64848c8...