Removed deprecated WebVR functionality

Removed deprecated WebVR functionality

BUG=706561
Review-Url: https://codereview.chromium.org/2777903008
Cr-Commit-Position: refs/heads/master@{#461914}
Committed: https://chromium.googlesource.com/chromium/src/+/c1d1aee5dcc8f13668fc418d5d670dc280759531

[bajones, 2017-03-29 21:43:10]

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

[commit-bot: I haz the power, 2017-03-29 21:44:08]

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

[bajones, 2017-03-29 22:09:01]

bajones@chromium.org changed reviewers:
+ bshe@chromium.org, bsheedy@chromium.org, dcheng@chromium.org,
mthiesse@chromium.org

[bajones, 2017-03-29 22:09:02]

dcheng@: Could you please review the use counter and .mojom changes?

bsheedy@: Please review changes to LayoutTests/vr

mthiesse@ and bshe@: Please review vr_shell, device/vr, and modules/vr

[bsheedy, 2017-03-29 22:19:56]

There's also the WPT IDL test in
third_party/WebKit/LayoutTests/external/wpt/webvr/ that needs to be changed.

Have you run any of these layout tests locally?

https://codereview.chromium.org/2777903008/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/vr/getVRDisplays_one_display.html (right):

https://codereview.chromium.org/2777903008/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/vr/getVRDisplays_one_display.html:18:
assert_true(pixel.capabilities.canPresent);
"display", not "pixel"

[bajones, 2017-03-29 23:05:36]

On 2017/03/29 22:19:56, bsheedy wrote:
> There's also the WPT IDL test in
> third_party/WebKit/LayoutTests/external/wpt/webvr/ that needs to be changed.

Oh, thanks for pointing that out!

> Have you run any of these layout tests locally?
> 
> "display", not "pixel"

*Sigh.* Yes, I ran them locally (at least I ran vr/*). I apparently also forgot
to actually, you know, save my correction to that file before uploading the CL.
Kinda important. :P Anyway, it's fixed now.

[bshe, 2017-03-30 00:25:40]

On 2017/03/29 23:05:36, bajones wrote:
> On 2017/03/29 22:19:56, bsheedy wrote:
> > There's also the WPT IDL test in
> > third_party/WebKit/LayoutTests/external/wpt/webvr/ that needs to be changed.
> 
> Oh, thanks for pointing that out!
> 
> > Have you run any of these layout tests locally?
> > 
> > "display", not "pixel"
> 
> *Sigh.* Yes, I ran them locally (at least I ran vr/*). I apparently also
forgot
> to actually, you know, save my correction to that file before uploading the
CL.
> Kinda important. :P Anyway, it's fixed now.

vr_shell lgtm

[dcheng, 2017-03-30 00:30:48]

rs lgtm for the mojo and usecounter changes

[bsheedy, 2017-03-30 00:32:19]

LayoutTests lgtm

[bajones, 2017-03-30 04:36:03]

The CQ bit was checked by bajones@chromium.org

[commit-bot: I haz the power, 2017-03-30 04:36:24]

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-03-30 04:44:53]

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

[commit-bot: I haz the power, 2017-03-30 04:44:54]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[haraken, 2017-03-30 05:07:09]

On 2017/03/30 04:44:54, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

owner LGTM

[bajones, 2017-03-30 15:03:24]

The CQ bit was checked by bajones@chromium.org

[commit-bot: I haz the power, 2017-03-30 15:03:51]

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-03-30 16:32:58]

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

[commit-bot: I haz the power, 2017-03-30 16:33:00]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[bajones, 2017-04-03 22:23:03]

The CQ bit was checked by bajones@chromium.org

[bajones, 2017-04-03 22:23:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
bsheedy@chromium.org, haraken@chromium.org, bshe@chromium.org,
dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2777903008/#ps40001
(title: "Rebase, fixed another test baseline")

[commit-bot: I haz the power, 2017-04-03 22:23:25]

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-04-04 00:41:18]

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

[commit-bot: I haz the power, 2017-04-04 00:41:21]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[bajones, 2017-04-04 20:05:47]

The CQ bit was checked by bajones@chromium.org

[bajones, 2017-04-04 20:05:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
bsheedy@chromium.org, haraken@chromium.org, bshe@chromium.org,
dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2777903008/#ps60001
(title: "Another test re-baseline")

[commit-bot: I haz the power, 2017-04-04 20:06:32]

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-04-05 00:28:34]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1491336347838220,
"parent_rev": "f23e01c5bf039c5a4c95b5448288e3b079ce7ddc", "commit_rev":
"c1d1aee5dcc8f13668fc418d5d670dc280759531"}

[commit-bot: I haz the power, 2017-04-05 00:30:01]

Description was changed from

==========
Removed deprecated WebVR functionality

BUG=706561
==========

to

==========
Removed deprecated WebVR functionality

BUG=706561

Review-Url: https://codereview.chromium.org/2777903008
Cr-Commit-Position: refs/heads/master@{#461914}
Committed:
https://chromium.googlesource.com/chromium/src/+/c1d1aee5dcc8f13668fc418d5d67...
==========

[commit-bot: I haz the power, 2017-04-05 00:30:02]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/c1d1aee5dcc8f13668fc418d5d67...