[DMC #1.914] Rename DocumentMarkerController::RemoveMarkers() to RemoveMarkersForNode()

Rename DocumentMarkerController::RemoveMarkers() to RemoveMarkersForNode()
yosin pointed out that it's confusing to have so many methods named
RemoveMarkers():
https://codereview.chromium.org/2820633002#msg30

This is the fourth of several renaming CLs I'm going to upload

BUG=707867
Review-Url: https://codereview.chromium.org/2830233002
Cr-Commit-Position: refs/heads/master@{#466534}
Committed: https://chromium.googlesource.com/chromium/src/+/bda3d4871655a1cb5121cde488845d9b968eda6c

[rlanday, 2017-04-21 03:49:39]

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

[rlanday, 2017-04-21 03:49:50]

rlanday@chromium.org changed reviewers:
+ xiaochengh@chromium.org, yosin@chromium.org

[commit-bot: I haz the power, 2017-04-21 03:49:59]

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

[rlanday, 2017-04-21 03:53:44]

Description was changed from

==========
Rename DocumentMarkerController::RemoveMarkers() to RemoveMarkersForNode()
yosin pointed out that it's confusing to have so many methods named
RemoveMarkers():
https://codereview.chromium.org/2820633002#msg30

This is the second of several renaming CLs I'm going to upload

BUG=707867
==========

to

==========
Rename DocumentMarkerController::RemoveMarkers() to RemoveMarkersForNode()
yosin pointed out that it's confusing to have so many methods named
RemoveMarkers():
https://codereview.chromium.org/2820633002#msg30

This is the fourth of several renaming CLs I'm going to upload

BUG=707867
==========

[Xiaocheng, 2017-04-21 05:26:29]

lgtm

[rlanday, 2017-04-21 05:42:41]

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

[commit-bot: I haz the power, 2017-04-21 05:43:04]

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-04-21 07:29:56]

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

[commit-bot: I haz the power, 2017-04-21 07:29:57]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2017-04-21 08:27:32]

lgtm

[yosin_UTC9, 2017-04-21 08:32:58]

yosin@chromium.org changed reviewers:
+ tkent@chromium.org

[yosin_UTC9, 2017-04-21 08:32:59]

+tkent@ for core/dom/Node.cpp chagnes

[tkent, 2017-04-21 08:35:30]

lgtm

[rlanday, 2017-04-21 08:39:55]

The CQ bit was checked by rlanday@chromium.org

[rlanday, 2017-04-21 08:39:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
xiaochengh@chromium.org, yosin@chromium.org
Link to the patchset: https://codereview.chromium.org/2830233002/#ps40001
(title: "Remove dependency")

[commit-bot: I haz the power, 2017-04-21 08:40:07]

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

[commit-bot: I haz the power, 2017-04-21 08:40:08]

This CL has an open dependency (Issue 2836563002 Patch 40001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[rlanday, 2017-04-21 23:37:49]

The CQ bit was checked by rlanday@chromium.org

[rlanday, 2017-04-21 23:37:50]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
yosin@chromium.org, xiaochengh@chromium.org
Link to the patchset: https://codereview.chromium.org/2830233002/#ps60001
(title: "Rebase, remove dep")

[commit-bot: I haz the power, 2017-04-21 23:38:40]

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-04-22 03:39:14]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492817869515030,
"parent_rev": "f2300e8e61e7d70eea4f7039b8d275c1a9978a97", "commit_rev":
"bda3d4871655a1cb5121cde488845d9b968eda6c"}

[commit-bot: I haz the power, 2017-04-22 03:39:53]

Description was changed from

==========
Rename DocumentMarkerController::RemoveMarkers() to RemoveMarkersForNode()
yosin pointed out that it's confusing to have so many methods named
RemoveMarkers():
https://codereview.chromium.org/2820633002#msg30

This is the fourth of several renaming CLs I'm going to upload

BUG=707867
==========

to

==========
Rename DocumentMarkerController::RemoveMarkers() to RemoveMarkersForNode()
yosin pointed out that it's confusing to have so many methods named
RemoveMarkers():
https://codereview.chromium.org/2820633002#msg30

This is the fourth of several renaming CLs I'm going to upload

BUG=707867

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

[commit-bot: I haz the power, 2017-04-22 03:39:54]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/bda3d4871655a1cb5121cde48884...