Add a requestFullscreen variant with a default (prefixed) type

Add a requestFullscreen variant with a default (prefixed) type

This is refactoring only, with the exception of VRDisplay.cpp, which
changes from unprefixed to prefixed.

As per the TODO, the idea is to make the default type depend on
RuntimeEnabledFeatures::fullscreenUnprefixedEnabled().

Also make RequestType an enum class while in the area.

BUG=402376
Committed: https://crrev.com/e0f53cb2304b26339df19a65b40137651a3346b1
Cr-Commit-Position: refs/heads/master@{#437856}

[foolip, 2016-12-11 19:01:53]

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

[commit-bot: I haz the power, 2016-12-11 19:02:09]

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

[foolip, 2016-12-11 19:48:49]

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

[foolip, 2016-12-11 19:48:50]

PTAL and CQ

[commit-bot: I haz the power, 2016-12-11 20:54:01]

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

[commit-bot: I haz the power, 2016-12-11 20:54:02]

Dry run: This issue passed the CQ dry run.

[mlamouri (slow - plz ping), 2016-12-12 10:40:44]

Leave a TODO(foolip), get things fix the next week <3

lgtm

[foolip, 2016-12-12 12:42:37]

The CQ bit was checked by foolip@chromium.org

[commit-bot: I haz the power, 2016-12-12 12:42:52]

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-12-12 12:46:55]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481546557011030, "parent_rev":
"dd5116124a08ed4a13a53ff2f75a8c61788e592d", "commit_rev":
"71fa8dbbfb6d8b64ea860ce3356c5507d96dd329"}

[commit-bot: I haz the power, 2016-12-12 12:47:21]

Description was changed from

==========
Add a requestFullscreen variant with a default (prefixed) type

This is refactoring only, with the exception of VRDisplay.cpp, which
changes from unprefixed to prefixed.

As per the TODO, the idea is to make the default type depend on
RuntimeEnabledFeatures::fullscreenUnprefixedEnabled().

Also make RequestType an enum class while in the area.

BUG=402376
==========

to

==========
Add a requestFullscreen variant with a default (prefixed) type

This is refactoring only, with the exception of VRDisplay.cpp, which
changes from unprefixed to prefixed.

As per the TODO, the idea is to make the default type depend on
RuntimeEnabledFeatures::fullscreenUnprefixedEnabled().

Also make RequestType an enum class while in the area.

BUG=402376

Review-Url: https://codereview.chromium.org/2565203002
==========

[commit-bot: I haz the power, 2016-12-12 12:47:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-12 15:11:46]

Description was changed from

==========
Add a requestFullscreen variant with a default (prefixed) type

This is refactoring only, with the exception of VRDisplay.cpp, which
changes from unprefixed to prefixed.

As per the TODO, the idea is to make the default type depend on
RuntimeEnabledFeatures::fullscreenUnprefixedEnabled().

Also make RequestType an enum class while in the area.

BUG=402376

Review-Url: https://codereview.chromium.org/2565203002
==========

to

==========
Add a requestFullscreen variant with a default (prefixed) type

This is refactoring only, with the exception of VRDisplay.cpp, which
changes from unprefixed to prefixed.

As per the TODO, the idea is to make the default type depend on
RuntimeEnabledFeatures::fullscreenUnprefixedEnabled().

Also make RequestType an enum class while in the area.

BUG=402376

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

[commit-bot: I haz the power, 2016-12-12 15:11:47]

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