[DMC #16] Rename RenderedDocumentMarker to TextMatchMarker

Rename RenderedDocumentMarker to TextMatchMarker

RenderedDocumentMarker used to be created for all DocumentMarker types. Now it's
only created for TextMatchMarkers. Renaming it to TextMatchMarker sets us up to
move all of the TextMatch-specific functionality out of DocumentMarker into this
class. I eventually want to do this for all the other marker types as well.

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

[rlanday, 2017-05-25 00:24:47]

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

[rlanday, 2017-05-25 00:30:10]

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

[commit-bot: I haz the power, 2017-05-25 00:30:59]

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-05-25 04:20:24]

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

[commit-bot: I haz the power, 2017-05-25 04:20:25]

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

[yosin_UTC9, 2017-05-25 05:27:52]

lgtm

[rlanday, 2017-05-25 20:22:19]

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

[commit-bot: I haz the power, 2017-05-25 20:22:59]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xiaocheng, 2017-05-25 21:28:16]

lgtm

[commit-bot: I haz the power, 2017-05-25 23:59:00]

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

[commit-bot: I haz the power, 2017-05-25 23:59:01]

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

[rlanday, 2017-05-26 05:27:16]

The CQ bit was checked by rlanday@chromium.org

[rlanday, 2017-05-26 05:27:17]

The patchset sent to the CQ was uploaded after l-g-t-m from yosin@chromium.org
Link to the patchset: https://codereview.chromium.org/2898953005/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2017-05-26 05:28:17]

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-05-26 07:03:00]

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

[commit-bot: I haz the power, 2017-05-26 07:03:01]

Failed to apply patch for
third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp:
While running git apply --index -3 -p1;
  error: patch failed:
third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp:37
  error: repository lacks the necessary blob to fall back on 3-way merge.
  error:
third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp:
patch does not apply

Patch:      
third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
Index:
third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
diff --git
a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
index
cc2824902add2e1672da0609fa7ac24c03d9638a..cf2be572e4bd24cec84531d65063ea0b1a82bc5c
100644
---
a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
+++
b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
@@ -37,8 +37,8 @@
 #include "core/editing/markers/CompositionMarkerListImpl.h"
 #include "core/editing/markers/DocumentMarkerListEditor.h"
 #include "core/editing/markers/GrammarMarkerListImpl.h"
-#include "core/editing/markers/RenderedDocumentMarker.h"
 #include "core/editing/markers/SpellingMarkerListImpl.h"
+#include "core/editing/markers/TextMatchMarker.h"
 #include "core/editing/markers/TextMatchMarkerListImpl.h"
 #include "core/frame/FrameView.h"
 #include "core/layout/LayoutObject.h"
@@ -446,7 +446,7 @@ void
DocumentMarkerController::InvalidateRectsForTextMatchMarkersInNode(
   const HeapVector<Member<DocumentMarker>>& markers_in_list =
       marker_list->GetMarkers();
   for (auto& marker : markers_in_list)
-    ToRenderedDocumentMarker(marker)->Invalidate();
+    ToTextMatchMarker(marker)->Invalidate();
 
   InvalidatePaintForTickmarks(node);
 }

[rlanday, 2017-05-26 16:31:51]

The CQ bit was checked by rlanday@chromium.org

[rlanday, 2017-05-26 16:31:53]

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

[commit-bot: I haz the power, 2017-05-26 16:32:43]

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-05-26 18:35:39]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495816311475810,
"parent_rev": "66cf3e9f6bb5ad99f0364f80e3469e6912c27f31", "commit_rev":
"802826d9bacfad322a4ec415373d1ed3f317a2a2"}

[commit-bot: I haz the power, 2017-05-26 18:35:51]

Description was changed from

==========
Rename RenderedDocumentMarker to TextMatchMarker

RenderedDocumentMarker used to be created for all DocumentMarker types. Now it's
only created for TextMatchMarkers. Renaming it to TextMatchMarker sets us up to
move all of the TextMatch-specific functionality out of DocumentMarker into this
class. I eventually want to do this for all the other marker types as well.

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

to

==========
Rename RenderedDocumentMarker to TextMatchMarker

RenderedDocumentMarker used to be created for all DocumentMarker types. Now it's
only created for TextMatchMarkers. Renaming it to TextMatchMarker sets us up to
move all of the TextMatch-specific functionality out of DocumentMarker into this
class. I eventually want to do this for all the other marker types as well.

BUG=707867

Review-Url: https://codereview.chromium.org/2898953005
Cr-Commit-Position: refs/heads/master@{#475070}
Committed:
https://chromium.googlesource.com/chromium/src/+/802826d9bacfad322a4ec415373d...
==========

[commit-bot: I haz the power, 2017-05-26 18:35:52]

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