Rename nodes/elements traversal function names.

Rename nodes/elements traversal function names.

Rename the following functions:

  NodeTraversal::startsFrom(const Node*) (and Traversal<ElementType>::startsFrom(const ElementType*)
  NodeTraversal::startsFromNextOf(const Node&) (and Traversal<ElementType>::startsFromNextOf(const ElementType&)

to:

  NodeTraversal::startsAt(const Node*) (and Traversal<ElementType>::startsAt(const ElementType*)
  NodeTraversal::startsAfter(const Node&) (and Traversal<ElementType>::startsAfter(const ElementType&)

so that they have more meaningful names.

This patch also removes an unnecessary underlying iterator, TraversalInclusiveNextIterator.

BUG=None
TEST=no layout test failure

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184287

[hayato, 2014-10-22 07:15:50]

hayato@chromium.org changed reviewers:
+ dglazkov@chromium.org, esprehn@chromium.org

[hayato, 2014-10-22 07:15:51]

PTAL.

A follow-up patch for the comment,
https://codereview.chromium.org/642973003/#msg18

[dglazkov, 2014-10-22 15:13:18]

What do you think of startAt, startAfter? /bikeshed

[hayato, 2014-10-23 07:32:22]

On 2014/10/22 15:13:18, dglazkov wrote:
> What do you think of startAt, startAfter? /bikeshed

Sounds better. Let me update the patch.

[hayato, 2014-10-23 07:59:21]

I've renamed. PTAL

[dglazkov, 2014-10-23 16:48:41]

The CQ bit was checked by dglazkov@chromium.org

[dglazkov, 2014-10-23 16:48:41]

lgtm

[commit-bot: I haz the power, 2014-10-23 16:49:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/667403002/20001

[commit-bot: I haz the power, 2014-10-23 16:53:59]

Committed patchset #2 (id:20001) as 184287