Adjust the timestamp for VRDisplay.requestAnimationFrame calls

Adjust the timestamp for VRDisplay.requestAnimationFrame calls

Previously timestamps had their origin incorrectly adjusted and were multiplied
by 1000.

BUG=669594
Committed: https://crrev.com/769c85525581f5f39cdc1d207b2254d35cb106f3
Cr-Commit-Position: refs/heads/master@{#435143}

[bajones, 2016-11-29 19:42:18]

bajones@chromium.org changed reviewers:
+ klausw@chromium.org

[bajones, 2016-11-29 19:42:19]

klausw@: Care to give me a sanity check here? Testing shows that the timestamps
appear to be consistent now with the ones returned from window.rAF()

[cjgrant, 2016-11-29 20:02:02]

cjgrant@chromium.org changed reviewers:
+ cjgrant@chromium.org

[cjgrant, 2016-11-29 20:02:02]

https://codereview.chromium.org/2533383002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/vr/VRDisplay.cpp (right):

https://codereview.chromium.org/2533383002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/vr/VRDisplay.cpp:53: // Need to devide by 1000
here because serviceScriptedAnimations expects
divide

[klausw, 2016-11-29 20:50:27]

LGTM with minor clarification request.

FWIW, I'm not convinced that replacing m_navigatorVR->document() with
m_vrDisplay->document() is much of an improvement, but it's fine if you think
it's clearer.

https://codereview.chromium.org/2533383002/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/vr/VRDisplay.cpp (right):

https://codereview.chromium.org/2533383002/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/vr/VRDisplay.cpp:199:
callback->m_useLegacyTimeBase = true;
Why are you changing this too? Add a comment?

[klausw, 2016-11-29 20:52:02]

On 2016/11/29 20:50:27, klausw wrote:
> LGTM with minor clarification request.
> 
> FWIW, I'm not convinced that replacing m_navigatorVR->document() with
> m_vrDisplay->document() is much of an improvement, but it's fine if you think
> it's clearer.

Nevermind, I just realized you need this because you don't have m_navigatorVR in
the helper class for the callback. Add a comment?

[bajones, 2016-11-29 22:47:22]

The CQ bit was checked by bajones@chromium.org

[bajones, 2016-11-29 22:47:23]

The patchset sent to the CQ was uploaded after l-g-t-m from klausw@chromium.org
Link to the patchset: https://codereview.chromium.org/2533383002/#ps20001
(title: "Addressed feedback")

[commit-bot: I haz the power, 2016-11-29 22:47:58]

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-11-30 00:50:06]

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

[commit-bot: I haz the power, 2016-11-30 00:50:07]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[bajones, 2016-11-30 07:24:05]

The CQ bit was checked by bajones@chromium.org

[commit-bot: I haz the power, 2016-11-30 07:24:49]

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-11-30 07:53:30]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480490645646970,
"parent_rev": "370504175ab320dd7b79ce3bf4c263b03644798e", "commit_rev":
"d7e14971361af51e56489b940e7dcacb8c4e432c"}

[commit-bot: I haz the power, 2016-11-30 07:53:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-30 07:56:21]

Description was changed from

==========
Adjust the timestamp for VRDisplay.requestAnimationFrame calls

Previously timestamps had their origin incorrectly adjusted and were multiplied
by 1000.

BUG=669594
==========

to

==========
Adjust the timestamp for VRDisplay.requestAnimationFrame calls

Previously timestamps had their origin incorrectly adjusted and were multiplied
by 1000.

BUG=669594

Committed: https://crrev.com/769c85525581f5f39cdc1d207b2254d35cb106f3
Cr-Commit-Position: refs/heads/master@{#435143}
==========

[commit-bot: I haz the power, 2016-11-30 07:56:22]

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