Update API to latest spec : Remove VRFrameData#timestamp

Update API to latest spec : Remove VRFrameData#timestamp
According to latest WebVR Spec, there are several changes. Remove VRFrameData#timestamp is one of them.

BUG=697167
Review-Url: https://codereview.chromium.org/2736593002
Cr-Commit-Position: refs/heads/master@{#455619}
Committed: https://chromium.googlesource.com/chromium/src/+/f4b1ca9d2d930fad5daaf6291a7a41c2eb141465

[shaobo.yan, 2017-03-06 02:27:13]

shaobo.yan@intel.com changed reviewers:
+ bajones@chromium.org, mthiesse@chromium.org

[shaobo.yan, 2017-03-06 02:27:14]

This patch is the first patch to Update API based on latest WebVR Spec (2,
March). PTAL.

[shaobo.yan, 2017-03-06 03:34:20]

Description was changed from

==========
Update API to latest spec : Remove VRFrameData#timestamp
According to latest WebVR Spec, there are several changes. Remove
VRFrameData#timestamp is one of them.

BUG=697167
==========

to

==========
Update API to latest spec : Remove VRFrameData#timestamp
According to latest WebVR Spec, there are several changes. Remove
VRFrameData#timestamp is one of them.

BUG=697167
==========

[mthiesse, 2017-03-06 15:28:23]

vr_shell/ lgtm

[bajones, 2017-03-06 18:59:43]

modules/vr LGTM

[bsheedy, 2017-03-06 19:03:11]

I think this will cause the IDL test to fail unless you update the expectations
file.
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/external/...

[bsheedy, 2017-03-06 19:05:24]

On 2017/03/06 19:03:11, bsheedy wrote:
> I think this will cause the IDL test to fail unless you update the
expectations
> file.
>
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/external/...

I think you'll also need to update the IDL that's in idlharness.html.

[shaobo.yan, 2017-03-07 00:15:24]

The CQ bit was checked by shaobo.yan@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-07 00:16:05]

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

[shaobo.yan, 2017-03-07 00:35:07]

On 2017/03/06 19:05:24, bsheedy wrote:
> On 2017/03/06 19:03:11, bsheedy wrote:
> > I think this will cause the IDL test to fail unless you update the
> expectations
> > file.
> >
>
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/external/...
> 
> I think you'll also need to update the IDL that's in idlharness.html.

Thx for your suggestion.
bsheedy@ PTAL.

[bsheedy, 2017-03-07 00:36:42]

On 2017/03/07 00:35:07, shaobo.yan wrote:
> On 2017/03/06 19:05:24, bsheedy wrote:
> > On 2017/03/06 19:03:11, bsheedy wrote:
> > > I think this will cause the IDL test to fail unless you update the
> > expectations
> > > file.
> > >
> >
>
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/external/...
> > 
> > I think you'll also need to update the IDL that's in idlharness.html.
> 
> Thx for your suggestion.
> bsheedy@ PTAL.

IDL changes LGTM

[shaobo.yan, 2017-03-07 01:46:12]

The CQ bit was checked by shaobo.yan@intel.com

[shaobo.yan, 2017-03-07 01:46:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
mthiesse@chromium.org, bajones@chromium.org
Link to the patchset: https://codereview.chromium.org/2736593002/#ps20001
(title: "Update wpt test related file")

[commit-bot: I haz the power, 2017-03-07 01:47:09]

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-07 01:55:58]

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

[commit-bot: I haz the power, 2017-03-07 01:55:59]

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...)

[shaobo.yan, 2017-03-07 02:09:36]

shaobo.yan@intel.com changed reviewers:
+ dcheng@chromium.org

[shaobo.yan, 2017-03-07 02:10:59]

dcheng@: PTAL

[dcheng, 2017-03-07 08:17:29]

rs lgtm for mojom changes

[shaobo.yan, 2017-03-07 08:37:02]

The CQ bit was checked by shaobo.yan@intel.com

[commit-bot: I haz the power, 2017-03-07 08:37: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-07 09:44:52]

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

[commit-bot: I haz the power, 2017-03-07 09:44:54]

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

[shaobo.yan, 2017-03-07 10:22:16]

According to bots result, delete getFrameData_timestamp_update.html test. 
bsheedy@: PTAL

[shaobo.yan, 2017-03-07 10:26:16]

The CQ bit was checked by shaobo.yan@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-07 10:26:33]

Dry run: 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-07 11:25:04]

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

[commit-bot: I haz the power, 2017-03-07 11:25:04]

Dry run: 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_...)

[shaobo.yan, 2017-03-07 11:35:40]

The CQ bit was checked by shaobo.yan@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-07 11:35:52]

Dry run: 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-07 12:41:20]

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

[commit-bot: I haz the power, 2017-03-07 12:41:21]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[shaobo.yan, 2017-03-07 13:04:25]

The CQ bit was checked by shaobo.yan@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-07 13:04:44]

Dry run: 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-07 16:40:05]

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

[commit-bot: I haz the power, 2017-03-07 16:40:07]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[bsheedy, 2017-03-07 17:14:51]

On 2017/03/07 10:22:16, shaobo.yan wrote:
> According to bots result, delete getFrameData_timestamp_update.html test. 
> bsheedy@: PTAL

LGTM, but I think you'll also need to modify
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/vr/getFra...
and

[shaobo.yan, 2017-03-08 00:48:51]

The CQ bit was checked by shaobo.yan@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-08 00:50:04]

Dry run: 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-08 01:38:55]

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

[commit-bot: I haz the power, 2017-03-08 01:38:56]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[shaobo.yan, 2017-03-08 01:44:33]

The CQ bit was checked by shaobo.yan@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-08 01:45:33]

Dry run: 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-08 02:28:53]

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

[commit-bot: I haz the power, 2017-03-08 02:28:55]

Dry run: 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_...)

[shaobo.yan, 2017-03-08 05:49:39]

The CQ bit was checked by shaobo.yan@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-08 05:49:52]

Dry run: 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-08 06:46:16]

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

[commit-bot: I haz the power, 2017-03-08 06:46:17]

Dry run: 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_...)

[shaobo.yan, 2017-03-08 07:03:12]

The CQ bit was checked by shaobo.yan@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-08 07:03:42]

Dry run: 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-08 08:06:19]

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

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

Dry run: This issue passed the CQ dry run.

[shaobo.yan, 2017-03-08 08:12:04]

On 2017/03/07 17:14:51, bsheedy wrote:
> On 2017/03/07 10:22:16, shaobo.yan wrote:
> > According to bots result, delete getFrameData_timestamp_update.html test. 
> > bsheedy@: PTAL
> 
> LGTM, but I think you'll also need to modify
>
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/vr/getFra...
> and

bsheedy@
Thx for your guidance about test related modification.
I've updated some test related file and dry bots seems finally passed.
PTAL.

[bsheedy, 2017-03-08 17:17:47]

On 2017/03/08 08:12:04, shaobo.yan wrote:
> On 2017/03/07 17:14:51, bsheedy wrote:
> > On 2017/03/07 10:22:16, shaobo.yan wrote:
> > > According to bots result, delete getFrameData_timestamp_update.html test. 
> > > bsheedy@: PTAL
> > 
> > LGTM, but I think you'll also need to modify
> >
>
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/vr/getFra...
> > and
> 
> bsheedy@
> Thx for your guidance about test related modification.
> I've updated some test related file and dry bots seems finally passed.
> PTAL.

third_party/WebKit/LayoutTests/vr/ lgtm

[shaobo.yan, 2017-03-09 00:43:23]

The CQ bit was checked by shaobo.yan@intel.com

[shaobo.yan, 2017-03-09 00:43:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
mthiesse@chromium.org, bajones@chromium.org, dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2736593002/#ps120001
(title: "modify mock-vr-service.js")

[commit-bot: I haz the power, 2017-03-09 00:44:10]

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-09 01:13:37]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1489020203280630,
"parent_rev": "10253f303a50e68dd30f5b1f372d30646366cc2e", "commit_rev":
"f4b1ca9d2d930fad5daaf6291a7a41c2eb141465"}

[commit-bot: I haz the power, 2017-03-09 01:15:03]

Description was changed from

==========
Update API to latest spec : Remove VRFrameData#timestamp
According to latest WebVR Spec, there are several changes. Remove
VRFrameData#timestamp is one of them.

BUG=697167
==========

to

==========
Update API to latest spec : Remove VRFrameData#timestamp
According to latest WebVR Spec, there are several changes. Remove
VRFrameData#timestamp is one of them.

BUG=697167

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

[commit-bot: I haz the power, 2017-03-09 01:15:05]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/chromium/src/+/f4b1ca9d2d930fad5daaf6291a7a...