Move setMediaPlaybackRequiresUserGesture to the cross-origin frame.

Move setMediaPlaybackRequiresUserGesture to the cross-origin frame.

Before this CL the http/tests/webaudio/autoplay-crossorigin.html layout
test would fail if the main frame and the cross-origin frame are in
different renderer processes.  After this CL, the internal settings are
set from the same frame that hosts AudioContext object - this ensures
that setting the settings and getting the settings happens from the same
renderer process (and consequently ensures that the test passes both
with and without --site-per-process flag).

BUG=652366
Committed: https://crrev.com/f33a8c06201766c37344768aaf99846f78857ff3
Cr-Commit-Position: refs/heads/master@{#422746}

[Łukasz Anforowicz, 2016-10-03 19:40:05]

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

[commit-bot: I haz the power, 2016-10-03 19:40:36]

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

[Łukasz Anforowicz, 2016-10-03 20:11:27]

lukasza@chromium.org changed reviewers:
+ alexmos@chromium.org, mlamouri@chromium.org

[Łukasz Anforowicz, 2016-10-03 20:11:28]

Alex, can you take a look please?  I think simply moving where
setMediaPlaybackRequiresUserGesture gets called is okay for this single test. 
In the long-term we need to discuss if/how we might want to replicate
WebSettings across OOPIFs (IMO it is not settled yet whether we want to do the
replication - maybe tweaking individual tests *is* okay, because right now there
is no way for components/test_runner to get notifications that WebSettings got
changed OR to serialize/replicate them over IPC).

[alexmos, 2016-10-03 20:23:12]

LGTM.  Thanks for proactively fixing the test!  I agree that it makes sense to
tweak the test here, and not replicate the settings across OOPIFs until we find
a really good reason for doing it.

[commit-bot: I haz the power, 2016-10-03 20:33:08]

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

[commit-bot: I haz the power, 2016-10-03 20:33:09]

Dry run: This issue passed the CQ dry run.

[Łukasz Anforowicz, 2016-10-03 20:39:35]

mlamouri@, can you also take a quick look please (since you've authored the test
:-).

[mlamouri (slow - plz ping), 2016-10-04 10:45:11]

The CQ bit was checked by mlamouri@chromium.org

[mlamouri (slow - plz ping), 2016-10-04 10:45:11]

lgtm

[commit-bot: I haz the power, 2016-10-04 10:45:28]

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, 2016-10-04 10:50:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-04 10:51:57]

Description was changed from

==========
Move setMediaPlaybackRequiresUserGesture to the cross-origin frame.

Before this CL the http/tests/webaudio/autoplay-crossorigin.html layout
test would fail if the main frame and the cross-origin frame are in
different renderer processes.  After this CL, the internal settings are
set from the same frame that hosts AudioContext object - this ensures
that setting the settings and getting the settings happens from the same
renderer process (and consequently ensures that the test passes both
with and without --site-per-process flag).

BUG=652366
==========

to

==========
Move setMediaPlaybackRequiresUserGesture to the cross-origin frame.

Before this CL the http/tests/webaudio/autoplay-crossorigin.html layout
test would fail if the main frame and the cross-origin frame are in
different renderer processes.  After this CL, the internal settings are
set from the same frame that hosts AudioContext object - this ensures
that setting the settings and getting the settings happens from the same
renderer process (and consequently ensures that the test passes both
with and without --site-per-process flag).

BUG=652366

Committed: https://crrev.com/f33a8c06201766c37344768aaf99846f78857ff3
Cr-Commit-Position: refs/heads/master@{#422746}
==========

[commit-bot: I haz the power, 2016-10-04 10:51:59]

Patchset 1 (id:??) landed as
https://crrev.com/f33a8c06201766c37344768aaf99846f78857ff3
Cr-Commit-Position: refs/heads/master@{#422746}