Unify "contributes to" and "requires mask" for clip-path child iteration

Unify "contributes to" and "requires mask" for clip-path child iteration

Since contributesToClip(...) and requiresMask(...) have a lot of overlap,
refactor them into a new (set of) function(s) that return an enumeration
based on the requirements for the element in question.

Committed: https://crrev.com/ce8c0ceacc0595d6fbc0ea52f07fd4b2132353ef
Cr-Commit-Position: refs/heads/master@{#437248}

[fs, 2016-12-08 08:50:56]

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

[commit-bot: I haz the power, 2016-12-08 08:51:04]

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

[fs, 2016-12-08 09:42:08]

Description was changed from

==========
Unify "contributes" and "requires mask" for clip-path child iteration

Since contributesToClip(...) and requiresMask(...) have a lot of overlap,
refactor them into a new (set of) function(s) that return an enumeration
based on the requirements for the element in question.
==========

to

==========
Unify "contributes to" and "requires mask" for clip-path child iteration

Since contributesToClip(...) and requiresMask(...) have a lot of overlap,
refactor them into a new (set of) function(s) that return an enumeration
based on the requirements for the element in question.
==========

[commit-bot: I haz the power, 2016-12-08 10:00:19]

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

[commit-bot: I haz the power, 2016-12-08 10:00:19]

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

[fs, 2016-12-08 10:13:24]

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

[fs, 2016-12-08 10:13:24]

This should be somewhere along the lines of your suggestion I think. Not less
code, but less code-paths at least.

[f(malita), 2016-12-08 14:13:36]

On 2016/12/08 10:13:24, fs wrote:
> This should be somewhere along the lines of your suggestion I think. Not less
> code, but less code-paths at least.

Thanks for following up, LGTM.

[fs, 2016-12-08 14:19:36]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-12-08 14:19:49]

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-08 15:35:00]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481206776671760, "parent_rev":
"a91eacba810ffabe313356b5626298b3950c70b0", "commit_rev":
"b1c5935183d492324b272feaece4d247f7df2e46"}

[commit-bot: I haz the power, 2016-12-08 15:35:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-08 15:38:26]

Description was changed from

==========
Unify "contributes to" and "requires mask" for clip-path child iteration

Since contributesToClip(...) and requiresMask(...) have a lot of overlap,
refactor them into a new (set of) function(s) that return an enumeration
based on the requirements for the element in question.
==========

to

==========
Unify "contributes to" and "requires mask" for clip-path child iteration

Since contributesToClip(...) and requiresMask(...) have a lot of overlap,
refactor them into a new (set of) function(s) that return an enumeration
based on the requirements for the element in question.

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

[commit-bot: I haz the power, 2016-12-08 15:38:27]

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