Move Node::lengthOfContents into Range, its only user.

Move Node::lengthOfContents into Range, its only user.

Node has so many methods, lets move this Range specific one to inside
Range.cpp. I also removed the Attr test which didn't really make sense.

Review-Url: https://codereview.chromium.org/2775003003
Cr-Commit-Position: refs/heads/master@{#459661}
Committed: https://chromium.googlesource.com/chromium/src/+/cb3466e6dad6e3cb829df8ebe4dc56d806168c84

[esprehn, 2017-03-25 01:14:49]

esprehn@chromium.org changed reviewers:
+ haraken@chromium.org, ojan@chromium.org

[esprehn, 2017-03-25 01:14:58]

Description was changed from

==========
Move Node::lengthOfContents into Range, it's only user.

Node has so many methods, lets move this Range specific one to inside
Range.cpp. I also removed the Attr test which didn't really make sense.
==========

to

==========
Move Node::lengthOfContents into Range, its only user.

Node has so many methods, lets move this Range specific one to inside
Range.cpp. I also removed the Attr test which didn't really make sense.
==========

[esprehn, 2017-03-25 01:15:02]

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

[commit-bot: I haz the power, 2017-03-25 01:15:28]

Dry run: 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-25 01:27:18]

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

[commit-bot: I haz the power, 2017-03-25 01:27:19]

Dry run: Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[esprehn, 2017-03-25 02:16:18]

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

[commit-bot: I haz the power, 2017-03-25 02:16:31]

Dry run: 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-25 03:39:59]

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

[commit-bot: I haz the power, 2017-03-25 03:40:00]

Dry run: This issue passed the CQ dry run.

[haraken, 2017-03-25 04:42:38]

LGTM

[esprehn, 2017-03-25 22:56:17]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2017-03-25 22:56: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, 2017-03-25 22:59:49]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490482577431780,
"parent_rev": "7596046bd145df1b871971eb65f3d97c5b752adc", "commit_rev":
"cb3466e6dad6e3cb829df8ebe4dc56d806168c84"}

[commit-bot: I haz the power, 2017-03-25 23:00:34]

Description was changed from

==========
Move Node::lengthOfContents into Range, its only user.

Node has so many methods, lets move this Range specific one to inside
Range.cpp. I also removed the Attr test which didn't really make sense.
==========

to

==========
Move Node::lengthOfContents into Range, its only user.

Node has so many methods, lets move this Range specific one to inside
Range.cpp. I also removed the Attr test which didn't really make sense.

Review-Url: https://codereview.chromium.org/2775003003
Cr-Commit-Position: refs/heads/master@{#459661}
Committed:
https://chromium.googlesource.com/chromium/src/+/cb3466e6dad6e3cb829df8ebe4dc...
==========

[commit-bot: I haz the power, 2017-03-25 23:00:35]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/cb3466e6dad6e3cb829df8ebe4dc...