Implement DocumentTimeline constructor

Implement DocumentTimeline constructor

This patch adds a constructor to DocumentTimeline to match the spec at
http://w3c.github.io/web-animations/#the-documenttimeline-interface. The
constructor enables the zero time of the timeline to be set at
something other than the navigationStart moment.

Changing the playback rate of the timeline (a functionality provided for
DevTools support) still modifies this zero time. The initial value for
the zero time is saved in the origin_time_ member.

BUG=674514, 624639, 600248
Review-Url: https://codereview.chromium.org/2960733004
Cr-Commit-Position: refs/heads/master@{#483594}
Committed: https://chromium.googlesource.com/chromium/src/+/7ee5b6192d46b7a25c3670c1aa8b10547e1057ef

[suzyh_UTC10 (ex-contributor), 2017-06-28 05:11:05]

suzyh@chromium.org changed reviewers:
+ ericwilligers@chromium.org

[suzyh_UTC10 (ex-contributor), 2017-06-28 05:11:11]

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

[commit-bot: I haz the power, 2017-06-28 05:11:28]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[suzyh_UTC10 (ex-contributor), 2017-06-28 05:13:50]

Description was changed from

==========
Implement DocumentTimeline constructor

This patch adds a constructor to DocumentTimeline to match the spec at
http://w3c.github.io/web-animations/#the-documenttimeline-interface. The
constructor enables the zero time of the timeline to be set at
something other than the navigationStart moment.

Changing the playback rate of the timeline (a functionality provided for
DevTools support) still modifies this zero time. The initial value for
the zero time is saved in the origin_time_ member.

BUG=674514
==========

to

==========
Implement DocumentTimeline constructor

This patch adds a constructor to DocumentTimeline to match the spec at
http://w3c.github.io/web-animations/#the-documenttimeline-interface. The
constructor enables the zero time of the timeline to be set at
something other than the navigationStart moment.

Changing the playback rate of the timeline (a functionality provided for
DevTools support) still modifies this zero time. The initial value for
the zero time is saved in the origin_time_ member.

BUG=674514,624639,600248
==========

[commit-bot: I haz the power, 2017-06-28 07:07:56]

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

[commit-bot: I haz the power, 2017-06-28 07:07:57]

Dry run: This issue passed the CQ dry run.

[Eric Willigers, 2017-06-29 00:31:38]

lgtm

[alancutter (OOO until 2018), 2017-06-29 03:16:18]

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

[alancutter (OOO until 2018), 2017-06-29 03:16:19]

lgtm with nit.

https://codereview.chromium.org/2960733004/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/animation/DocumentTimeline.h (right):

https://codereview.chromium.org/2960733004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/animation/DocumentTimeline.h:71: double
origin_time = 0.0,
Let's use origin_time_in_seconds here.

[alancutter (OOO until 2018), 2017-06-29 03:29:00]

On 2017/06/29 at 03:16:19, alancutter wrote:
> lgtm with nit.
> 
>
https://codereview.chromium.org/2960733004/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/animation/DocumentTimeline.h (right):
> 
>
https://codereview.chromium.org/2960733004/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/animation/DocumentTimeline.h:71: double
origin_time = 0.0,
> Let's use origin_time_in_seconds here.

*origin_time_in_milliseconds :P

[suzyh_UTC10 (ex-contributor), 2017-06-29 04:12:22]

On 2017/06/29 at 03:29:00, alancutter wrote:
> On 2017/06/29 at 03:16:19, alancutter wrote:
> > lgtm with nit.
> > 
> >
https://codereview.chromium.org/2960733004/diff/1/third_party/WebKit/Source/c...
> > File third_party/WebKit/Source/core/animation/DocumentTimeline.h (right):
> > 
> >
https://codereview.chromium.org/2960733004/diff/1/third_party/WebKit/Source/c...
> > third_party/WebKit/Source/core/animation/DocumentTimeline.h:71: double
origin_time = 0.0,
> > Let's use origin_time_in_seconds here.
> 
> *origin_time_in_milliseconds :P

Done.

[suzyh_UTC10 (ex-contributor), 2017-06-29 04:12:49]

The CQ bit was checked by suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2017-06-29 04:12:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org, ericwilligers@chromium.org
Link to the patchset: https://codereview.chromium.org/2960733004/#ps40001
(title: "Extend rename into test file")

[commit-bot: I haz the power, 2017-06-29 04:13:02]

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-06-29 06:16:41]

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

[commit-bot: I haz the power, 2017-06-29 06:16:42]

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

[suzyh_UTC10 (ex-contributor), 2017-06-29 06:59:31]

The CQ bit was checked by suzyh@chromium.org

[commit-bot: I haz the power, 2017-06-29 06:59:47]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[suzyh_UTC10 (ex-contributor), 2017-06-29 08:44:28]

The CQ bit was checked by suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2017-06-29 08:44:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
alancutter@chromium.org, ericwilligers@chromium.org
Link to the patchset: https://codereview.chromium.org/2960733004/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2017-06-29 08:44:47]

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-06-29 09:01:32]

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

[commit-bot: I haz the power, 2017-06-29 09:01:32]

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

[suzyh_UTC10 (ex-contributor), 2017-06-29 23:25:33]

The CQ bit was checked by suzyh@chromium.org

[commit-bot: I haz the power, 2017-06-29 23:25: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, 2017-06-30 02:19:50]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1498778733516820,
"parent_rev": "f2703bb4a656cdd8b1c058040fac0eac101643c0", "commit_rev":
"7ee5b6192d46b7a25c3670c1aa8b10547e1057ef"}

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

Description was changed from

==========
Implement DocumentTimeline constructor

This patch adds a constructor to DocumentTimeline to match the spec at
http://w3c.github.io/web-animations/#the-documenttimeline-interface. The
constructor enables the zero time of the timeline to be set at
something other than the navigationStart moment.

Changing the playback rate of the timeline (a functionality provided for
DevTools support) still modifies this zero time. The initial value for
the zero time is saved in the origin_time_ member.

BUG=674514,624639,600248
==========

to

==========
Implement DocumentTimeline constructor

This patch adds a constructor to DocumentTimeline to match the spec at
http://w3c.github.io/web-animations/#the-documenttimeline-interface. The
constructor enables the zero time of the timeline to be set at
something other than the navigationStart moment.

Changing the playback rate of the timeline (a functionality provided for
DevTools support) still modifies this zero time. The initial value for
the zero time is saved in the origin_time_ member.

BUG=674514,624639,600248

Review-Url: https://codereview.chromium.org/2960733004
Cr-Commit-Position: refs/heads/master@{#483594}
Committed:
https://chromium.googlesource.com/chromium/src/+/7ee5b6192d46b7a25c3670c1aa8b...
==========

[commit-bot: I haz the power, 2017-06-30 02:20:05]

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