Fix FullscreenController tests with simplified-fullscreen-ui flag.

Fix FullscreenController tests with simplified-fullscreen-ui flag.

Running browser_tests and interactive_ui_tests with
--enable-simplified-fullscreen-ui now passes. (This is not currently run
on any bots, but a flag flip will follow.)

Also fixed FullscreenOnFileURL to test fullscreen, not mouse lock
(previously this test was identical to MouseLockOnFileURL).

Note: Previously none of the tests were actually locking the mouse and
testing that the bubble was displayed (now we are forced to do that). It
turns out this never worked because calling Browser::RequestToLockMouse
is not sufficient to lock the mouse. Added a fake flag that tells the
mouse lock controller to simulate locking the mouse so we can test the
bubble.

BUG=352425
Committed: https://crrev.com/8a6b6824cc9d4095ef3ec5c8c01f937b9f3ccabf
Cr-Commit-Position: refs/heads/master@{#368283}

[Matt Giuca, 2016-01-08 02:44:51]

mgiuca@chromium.org changed reviewers:
+ scheib@gmail.com

[Matt Giuca, 2016-01-08 02:45:17]

mgiuca@chromium.org changed reviewers:
+ scheib@chromium.org
- scheib@gmail.com

[scheib, 2016-01-08 03:11:00]

LGTM

[Matt Giuca, 2016-01-08 04:59:12]

I had to fix some more tests after LGTM that were broken on Win/ChromeOS. The
fix is the same as the one already approved so going ahead and landing this.

[Matt Giuca, 2016-01-08 04:59:51]

The CQ bit was checked by mgiuca@chromium.org

[Matt Giuca, 2016-01-08 04:59:52]

The patchset sent to the CQ was uploaded after l-g-t-m from scheib@chromium.org
Link to the patchset: https://codereview.chromium.org/1570443008/#ps20001
(title: "FullscreenControllerInteractiveTest: Further fixes for ChromeOS-only
tests.")

[commit-bot: I haz the power, 2016-01-08 05:00:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1570443008/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1570443008/20001

[Matt Giuca, 2016-01-08 05:01:03]

On 2016/01/08 04:59:12, Matt Giuca wrote:
> I had to fix some more tests after LGTM that were broken on Win/ChromeOS. The
> fix is the same as the one already approved so going ahead and landing this.

(You can see these updated tests passing on
https://codereview.chromium.org/1564783004/#ps60001 where the flag is flipped.)

[commit-bot: I haz the power, 2016-01-08 05:55:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-08 05:55:56]

Description was changed from

==========
Fix FullscreenController tests with simplified-fullscreen-ui flag.

Running browser_tests and interactive_ui_tests with
--enable-simplified-fullscreen-ui now passes. (This is not currently run
on any bots, but a flag flip will follow.)

Also fixed FullscreenOnFileURL to test fullscreen, not mouse lock
(previously this test was identical to MouseLockOnFileURL).

Note: Previously none of the tests were actually locking the mouse and
testing that the bubble was displayed (now we are forced to do that). It
turns out this never worked because calling Browser::RequestToLockMouse
is not sufficient to lock the mouse. Added a fake flag that tells the
mouse lock controller to simulate locking the mouse so we can test the
bubble.

BUG=352425
==========

to

==========
Fix FullscreenController tests with simplified-fullscreen-ui flag.

Running browser_tests and interactive_ui_tests with
--enable-simplified-fullscreen-ui now passes. (This is not currently run
on any bots, but a flag flip will follow.)

Also fixed FullscreenOnFileURL to test fullscreen, not mouse lock
(previously this test was identical to MouseLockOnFileURL).

Note: Previously none of the tests were actually locking the mouse and
testing that the bubble was displayed (now we are forced to do that). It
turns out this never worked because calling Browser::RequestToLockMouse
is not sufficient to lock the mouse. Added a fake flag that tells the
mouse lock controller to simulate locking the mouse so we can test the
bubble.

BUG=352425

Committed: https://crrev.com/8a6b6824cc9d4095ef3ec5c8c01f937b9f3ccabf
Cr-Commit-Position: refs/heads/master@{#368283}
==========

[commit-bot: I haz the power, 2016-01-08 05:55:57]

Patchset 2 (id:??) landed as
https://crrev.com/8a6b6824cc9d4095ef3ec5c8c01f937b9f3ccabf
Cr-Commit-Position: refs/heads/master@{#368283}