Convert SMILTimeContainer timers to UnspecedTimer frame-specific timer.

Convert SMILTimeContainer timers to UnspecedTimer frame-specific timer.

BUG=624694
Review-Url: https://codereview.chromium.org/2709103002
Cr-Commit-Position: refs/heads/master@{#451930}
Committed: https://chromium.googlesource.com/chromium/src/+/b74914599dbcf571dbe4e9b3c25c3d085ad2611e

[joelhockey, 2017-02-22 06:19:00]

joelhockey@chromium.org changed reviewers:
+ haraken@chromium.org, slangley@chromium.org

[haraken, 2017-02-22 06:24:42]

Would you add a link to the spec to the CL description?

Is it saying that SMIL tasks should be posted to the Presentation task source?

[joelhockey, 2017-02-22 06:26:37]

On 2017/02/22 at 06:24:42, haraken wrote:
> Would you add a link to the spec to the CL description?
> 
> Is it saying that SMIL tasks should be posted to the Presentation task source?

I will have a look through the spec, but I don't know what TaskType to assign. 
I should have made the question clear, but I was hoping that you would give some
direction.

[haraken, 2017-02-22 06:29:57]

On 2017/02/22 06:26:37, joelhockey wrote:
> On 2017/02/22 at 06:24:42, haraken wrote:
> > Would you add a link to the spec to the CL description?
> > 
> > Is it saying that SMIL tasks should be posted to the Presentation task
source?
> 
> I will have a look through the spec, but I don't know what TaskType to assign.

> I should have made the question clear, but I was hoping that you would give
some
> direction.

I don't think this is a speced task, so shall we use TaskType::UnspecedTimer?

[joelhockey, 2017-02-22 06:37:46]

Description was changed from

==========
Convert SMILTimeContainer timers to frame-specific Presentation timer.

BUG=624694
==========

to

==========
Convert SMILTimeContainer timers to Unspeced frame-specific timer.

BUG=624694
==========

[joelhockey, 2017-02-22 06:38:02]

On 2017/02/22 at 06:29:57, haraken wrote:
> On 2017/02/22 06:26:37, joelhockey wrote:
> > On 2017/02/22 at 06:24:42, haraken wrote:
> > > Would you add a link to the spec to the CL description?
> > > 
> > > Is it saying that SMIL tasks should be posted to the Presentation task
source?
> > 
> > I will have a look through the spec, but I don't know what TaskType to
assign. 
> > I should have made the question clear, but I was hoping that you would give
some
> > direction.
> 
> I don't think this is a speced task, so shall we use TaskType::UnspecedTimer?

Done.

[haraken, 2017-02-22 06:42:00]

LGTM

[joelhockey, 2017-02-22 06:42:45]

Description was changed from

==========
Convert SMILTimeContainer timers to Unspeced frame-specific timer.

BUG=624694
==========

to

==========
Convert SMILTimeContainer timers to UnspecedTimer frame-specific timer.

BUG=624694
==========

[joelhockey, 2017-02-22 06:43:43]

The CQ bit was checked by joelhockey@chromium.org

[joelhockey, 2017-02-22 06:43:44]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2709103002/#ps40001
(title: "Convert SMILTimeContainer timers to Unspeced frame-specific timer.")

[commit-bot: I haz the power, 2017-02-22 06:45:05]

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-02-22 08:30:44]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487745823829340,
"parent_rev": "68ca487c1e84a61cbcf3995637376d45bc1ff01b", "commit_rev":
"b74914599dbcf571dbe4e9b3c25c3d085ad2611e"}

[commit-bot: I haz the power, 2017-02-22 08:31:25]

Description was changed from

==========
Convert SMILTimeContainer timers to UnspecedTimer frame-specific timer.

BUG=624694
==========

to

==========
Convert SMILTimeContainer timers to UnspecedTimer frame-specific timer.

BUG=624694

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

[commit-bot: I haz the power, 2017-02-22 08:31:27]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/b74914599dbcf571dbe4e9b3c25c...