Tidy up instance time handling in SVGSMILElement

Tidy up instance time handling in SVGSMILElement

Merge the addBeginTime and addEndTime methods into a single method
addInstanceTime (that takes a BeginOrEnd argument.) Migrate callers to
the new method, getting rid of SVGSMILElement::handleConditionEvent in
the process.
Also inline some trivial functions in SVGAnimationElement.

BUG=641437
Review-Url: https://codereview.chromium.org/2712343004
Cr-Commit-Position: refs/heads/master@{#453233}
Committed: https://chromium.googlesource.com/chromium/src/+/901d1e45d6a0a49407a7e8936f6edf9b6124c260

[fs, 2017-02-27 11:54:16]

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

[commit-bot: I haz the power, 2017-02-27 11:54:23]

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

[fs, 2017-02-27 11:55:13]

Description was changed from

==========
Tidy up instance time handling in SVGSMILElement

Merge the addBeginTime and addEndTime methods into a single method
addInstanceTime (that takes a BeginOrEnd argument.) Migrate callers to
the new method.
Also inline some trivial functions in SVGAnimationElement and fold away
SVGSMILElement::handleConditionEvent.

BUG=641437
==========

to

==========
Tidy up instance time handling in SVGSMILElement

Merge the addBeginTime and addEndTime methods into a single method
addInstanceTime (that takes a BeginOrEnd argument.) Migrate callers to
the new method, getting rid of SVGSMILElement::handleConditionEvent in
the process.
Also inline some trivial functions in SVGAnimationElement and fold away
SVGSMILElement::handleConditionEvent.

BUG=641437
==========

[fs, 2017-02-27 12:09:25]

Description was changed from

==========
Tidy up instance time handling in SVGSMILElement

Merge the addBeginTime and addEndTime methods into a single method
addInstanceTime (that takes a BeginOrEnd argument.) Migrate callers to
the new method, getting rid of SVGSMILElement::handleConditionEvent in
the process.
Also inline some trivial functions in SVGAnimationElement and fold away
SVGSMILElement::handleConditionEvent.

BUG=641437
==========

to

==========
Tidy up instance time handling in SVGSMILElement

Merge the addBeginTime and addEndTime methods into a single method
addInstanceTime (that takes a BeginOrEnd argument.) Migrate callers to
the new method, getting rid of SVGSMILElement::handleConditionEvent in
the process.
Also inline some trivial functions in SVGAnimationElement.

BUG=641437
==========

[fs, 2017-02-27 12:59:03]

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

[commit-bot: I haz the power, 2017-02-27 13:25:06]

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

[commit-bot: I haz the power, 2017-02-27 13:25:07]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-02-27 15:39:17]

lgtm

[fs, 2017-02-27 16:27:37]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2017-02-27 16:28:04]

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-27 16:33:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488212857230630, "parent_rev":
"0d2ad84ffb7dfb632cbfe9f847daa931ae128344", "commit_rev":
"901d1e45d6a0a49407a7e8936f6edf9b6124c260"}

[commit-bot: I haz the power, 2017-02-27 16:33:59]

Description was changed from

==========
Tidy up instance time handling in SVGSMILElement

Merge the addBeginTime and addEndTime methods into a single method
addInstanceTime (that takes a BeginOrEnd argument.) Migrate callers to
the new method, getting rid of SVGSMILElement::handleConditionEvent in
the process.
Also inline some trivial functions in SVGAnimationElement.

BUG=641437
==========

to

==========
Tidy up instance time handling in SVGSMILElement

Merge the addBeginTime and addEndTime methods into a single method
addInstanceTime (that takes a BeginOrEnd argument.) Migrate callers to
the new method, getting rid of SVGSMILElement::handleConditionEvent in
the process.
Also inline some trivial functions in SVGAnimationElement.

BUG=641437

Review-Url: https://codereview.chromium.org/2712343004
Cr-Commit-Position: refs/heads/master@{#453233}
Committed:
https://chromium.googlesource.com/chromium/src/+/901d1e45d6a0a49407a7e8936f6e...
==========

[commit-bot: I haz the power, 2017-02-27 16:34:00]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/901d1e45d6a0a49407a7e8936f6e...