Tighten SVGAnimationElement::shouldApplyAnimation

Tighten SVGAnimationElement::shouldApplyAnimation

Fold the targetIsUsable(...) helper from SVGAnimateElement into said
method, and then replace the uses of the former with the corresponding
'should apply' predicate.

BUG=640676
Committed: https://crrev.com/fe3a12df154f8aa3d439f5ed275f0eb85ab2fc9a
Cr-Commit-Position: refs/heads/master@{#415253}

[fs, 2016-08-28 18:02:13]

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

[commit-bot: I haz the power, 2016-08-28 18:02:22]

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

[fs, 2016-08-28 19:22:33]

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

[fs, 2016-08-28 19:22:33]

Follow-up to suggestions from https://codereview.chromium.org/2285473002

[commit-bot: I haz the power, 2016-08-28 19:23:48]

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

[commit-bot: I haz the power, 2016-08-28 19:23:49]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[pdr., 2016-08-29 00:38:54]

On 2016/08/28 at 19:22:33, fs wrote:
> Follow-up to suggestions from https://codereview.chromium.org/2285473002

The !parentNode() change is tricky but as far as I can tell correct (and cleaner
now).

LGTM

[fs, 2016-08-29 08:15:27]

On 2016/08/29 at 00:38:54, pdr wrote:
> On 2016/08/28 at 19:22:33, fs wrote:
> > Follow-up to suggestions from https://codereview.chromium.org/2285473002
> 
> The !parentNode() change is tricky but as far as I can tell correct (and
cleaner now).

...and quite possibly redundant =)

[fs, 2016-08-29 08:15:37]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-29 08:15:59]

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-08-29 09:33:17]

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

[commit-bot: I haz the power, 2016-08-29 09:33:19]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[fs, 2016-08-29 17:46:39]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-29 17:52:20]

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-08-30 03:41:36]

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

[commit-bot: I haz the power, 2016-08-30 03:41:37]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[fs, 2016-08-30 07:54:16]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-30 07:54:44]

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-08-30 09:21:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 09:22:18]

Description was changed from

==========
Tighten SVGAnimationElement::shouldApplyAnimation

Fold the targetIsUsable(...) helper from SVGAnimateElement into said
method, and then replace the uses of the former with the corresponding
'should apply' predicate.

BUG=640676
==========

to

==========
Tighten SVGAnimationElement::shouldApplyAnimation

Fold the targetIsUsable(...) helper from SVGAnimateElement into said
method, and then replace the uses of the former with the corresponding
'should apply' predicate.

BUG=640676

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

[commit-bot: I haz the power, 2016-08-30 09:22:19]

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