Reuse executeForTraverseRoot to avoid a manual loop in SelectorQuery::findTraverseRootsAndExecute.

Reuse executeForTraverseRoot to avoid a manual loop in SelectorQuery::findTraverseRootsAndExecute.

This lets us remove a for loop inside SelectorQuery::findTraverseRootsAndExecute,
we can also make executeForTraverseRoot take a reference since only one caller
could have passed null before.

BUG=703900
Review-Url: https://codereview.chromium.org/2785253002
Cr-Commit-Position: refs/heads/master@{#460880}
Committed: https://chromium.googlesource.com/chromium/src/+/9a91d5107bc7f45f238c52466449a9cea12e8493

[esprehn, 2017-03-30 18:53:25]

The CQ bit was checked by esprehn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-30 18:54:02]

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

[esprehn, 2017-03-30 20:26:23]

esprehn@chromium.org changed reviewers:
+ dglazkov@chromium.org, meade@chromium.org, rune@opera.com

[dglazkov, 2017-03-30 20:53:07]

lgtm

[esprehn, 2017-03-30 20:55:36]

The CQ bit was unchecked by esprehn@chromium.org

[esprehn, 2017-03-30 20:55:38]

The CQ bit was checked by esprehn@chromium.org

[esprehn, 2017-03-30 20:56:20]

Description was changed from

==========
Reuse executeForTraverseRoot to avoid a manual loop in
SelectorQuery::findTraverseRootsAndExecute.

BUG=703900
==========

to

==========
Reuse executeForTraverseRoot to avoid a manual loop in
SelectorQuery::findTraverseRootsAndExecute.

This lets us remove a for loop inside
SelectorQuery::findTraverseRootsAndExecute,
we can also make executeForTraverseRoot take a reference since only one caller
could have passed null before.

BUG=703900
==========

[commit-bot: I haz the power, 2017-03-30 20:56:37]

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-03-30 21:18:34]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490907338201680,
"parent_rev": "a46872ca96d862816f993c16d6c270af26290247", "commit_rev":
"9a91d5107bc7f45f238c52466449a9cea12e8493"}

[commit-bot: I haz the power, 2017-03-30 21:19:24]

Description was changed from

==========
Reuse executeForTraverseRoot to avoid a manual loop in
SelectorQuery::findTraverseRootsAndExecute.

This lets us remove a for loop inside
SelectorQuery::findTraverseRootsAndExecute,
we can also make executeForTraverseRoot take a reference since only one caller
could have passed null before.

BUG=703900
==========

to

==========
Reuse executeForTraverseRoot to avoid a manual loop in
SelectorQuery::findTraverseRootsAndExecute.

This lets us remove a for loop inside
SelectorQuery::findTraverseRootsAndExecute,
we can also make executeForTraverseRoot take a reference since only one caller
could have passed null before.

BUG=703900

Review-Url: https://codereview.chromium.org/2785253002
Cr-Commit-Position: refs/heads/master@{#460880}
Committed:
https://chromium.googlesource.com/chromium/src/+/9a91d5107bc7f45f238c52466449...
==========

[commit-bot: I haz the power, 2017-03-30 21:19:25]

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