Improve separation between "SMIL times" and timestamps

Improve separation between "SMIL times" and timestamps

The 'elapsed' time does not need to be a SMILTime in general - it can
only be non-finite in the case where the document is not active, and
then the timeline should not be running/animations updated.
Thread the double value further from SMILTimeContainer down into
SVGSMILElement. Simplify some computations.

BUG=641437, 676137
Committed: https://crrev.com/d6b6ebf86773fcafd3ce6cdc6fe7ef7d14bae91e
Cr-Commit-Position: refs/heads/master@{#440171}

[fs, 2016-12-21 14:31:38]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-12-21 14:31:50]

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, 2016-12-21 16:43:03]

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

[commit-bot: I haz the power, 2016-12-21 16:43:04]

Dry run: This issue passed the CQ dry run.

[fs, 2016-12-21 17:24:58]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[pdr., 2016-12-21 18:58:15]

On 2016/12/21 at 17:24:58, fs wrote:
> 

LGTM for the cleanup, but I don't see how this fixes
https://bugs.chromium.org/p/chromium/issues/detail?id=676137 since we're still
doing these calculations with doubles.

[fs, 2016-12-21 18:59:31]

On 2016/12/21 at 18:58:15, pdr wrote:
> On 2016/12/21 at 17:24:58, fs wrote:
> > 
> 
> LGTM for the cleanup, but I don't see how this fixes
https://bugs.chromium.org/p/chromium/issues/detail?id=676137 since we're still
doing these calculations with doubles.

I just happened to (re)notice this while looking at that bug, so I made a fix
that depends on this change.

[pdr., 2016-12-21 19:00:48]

On 2016/12/21 at 18:59:31, fs wrote:
> On 2016/12/21 at 18:58:15, pdr wrote:
> > On 2016/12/21 at 17:24:58, fs wrote:
> > > 
> > 
> > LGTM for the cleanup, but I don't see how this fixes
https://bugs.chromium.org/p/chromium/issues/detail?id=676137 since we're still
doing these calculations with doubles.
> 
> I just happened to (re)notice this while looking at that bug, so I made a fix
that depends on this change.

ahh ok, full steam ahead!

[fs, 2016-12-21 19:04:48]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-12-21 19:05: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, 2016-12-21 19:11:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482347088553830, "parent_rev":
"75c7dadf8015ba1195923d1a83609485923bdc87", "commit_rev":
"8dbb2be423803a19d83575e05e8036319afbad80"}

[commit-bot: I haz the power, 2016-12-21 19:12:29]

Description was changed from

==========
Improve separation between "SMIL times" and timestamps

The 'elapsed' time does not need to be a SMILTime in general - it can
only be non-finite in the case where the document is not active, and
then the timeline should not be running/animations updated.
Thread the double value further from SMILTimeContainer down into
SVGSMILElement. Simplify some computations.

BUG=641437,676137
==========

to

==========
Improve separation between "SMIL times" and timestamps

The 'elapsed' time does not need to be a SMILTime in general - it can
only be non-finite in the case where the document is not active, and
then the timeline should not be running/animations updated.
Thread the double value further from SMILTimeContainer down into
SVGSMILElement. Simplify some computations.

BUG=641437,676137

Review-Url: https://codereview.chromium.org/2592103002
==========

[commit-bot: I haz the power, 2016-12-21 19:12:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-21 19:15:46]

Description was changed from

==========
Improve separation between "SMIL times" and timestamps

The 'elapsed' time does not need to be a SMILTime in general - it can
only be non-finite in the case where the document is not active, and
then the timeline should not be running/animations updated.
Thread the double value further from SMILTimeContainer down into
SVGSMILElement. Simplify some computations.

BUG=641437,676137

Review-Url: https://codereview.chromium.org/2592103002
==========

to

==========
Improve separation between "SMIL times" and timestamps

The 'elapsed' time does not need to be a SMILTime in general - it can
only be non-finite in the case where the document is not active, and
then the timeline should not be running/animations updated.
Thread the double value further from SMILTimeContainer down into
SVGSMILElement. Simplify some computations.

BUG=641437,676137

Committed: https://crrev.com/d6b6ebf86773fcafd3ce6cdc6fe7ef7d14bae91e
Cr-Commit-Position: refs/heads/master@{#440171}
==========

[commit-bot: I haz the power, 2016-12-21 19:15:47]

Patchset 1 (id:??) landed as
https://crrev.com/d6b6ebf86773fcafd3ce6cdc6fe7ef7d14bae91e
Cr-Commit-Position: refs/heads/master@{#440171}