Revert of [DMC #17] Make TextMatchMarkers constructible in single step

Revert of [DMC #17] Make TextMatchMarkers constructible in single step (patchset #4 id:60001 of https://codereview.chromium.org/2905753002/ )

Reason for revert:
This broke the build.

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac%20Builder%20%28dbg%29/builds/201575

Original issue's description:
> Make TextMatchMarkers constructible in single step
> 
> Currently, we only construct instances of TextMatchMarker (previously
> RenderedDocumentMarker) inside TextMatchMarkerListImpl, by calling
> TextMatchMarker::Create() on an instance of DocumentMarker. This CL changes this
> so callers directly construct instances of TextMatchMarker. This sets us up to
> eventually move the storage and accessor methods for whether or not the marker
> is active out of DocumentMarker and into TextMatchMarker.
> 
> BUG=707867
> 
> Review-Url: https://codereview.chromium.org/2905753002
> Cr-Commit-Position: refs/heads/master@{#475724}
> Committed: https://chromium.googlesource.com/chromium/src/+/6b0a494122ebe2fc48099a9549b0acbf0b28571d

TBR=yosin@chromium.org,xiaochengh@chromium.org,rlanday@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=707867
Review-Url: https://codereview.chromium.org/2912133003
Cr-Commit-Position: refs/heads/master@{#475732}
Committed: https://chromium.googlesource.com/chromium/src/+/182740235efc11d585d6710485d66fff4fcf84ae

[tyoshino (SeeGerritForStatus), 2017-05-31 01:48:21]

The CQ bit was checked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2017-05-31 01:48:22]

Created Revert of [DMC #17] Make TextMatchMarkers constructible in single step

[commit-bot: I haz the power, 2017-05-31 01:49:07]

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-31 01:49:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496195301738220, "parent_rev":
"fcbbb5f35592867b2c274348c9c83807c58001c0", "commit_rev":
"d07a21a4769d02db309f55c4efba1480ecc9a134"}

[commit-bot: I haz the power, 2017-05-31 01:49:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496195301738220, "parent_rev":
"77c59a87000dc29ebe28488b9302590081463f6e", "commit_rev":
"182740235efc11d585d6710485d66fff4fcf84ae"}

[commit-bot: I haz the power, 2017-05-31 01:49:55]

Description was changed from

==========
Revert of [DMC #17] Make TextMatchMarkers constructible in single step (patchset
#4 id:60001 of https://codereview.chromium.org/2905753002/ )

Reason for revert:
This broke the build.

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac%20Builder%...

Original issue's description:
> Make TextMatchMarkers constructible in single step
> 
> Currently, we only construct instances of TextMatchMarker (previously
> RenderedDocumentMarker) inside TextMatchMarkerListImpl, by calling
> TextMatchMarker::Create() on an instance of DocumentMarker. This CL changes
this
> so callers directly construct instances of TextMatchMarker. This sets us up to
> eventually move the storage and accessor methods for whether or not the marker
> is active out of DocumentMarker and into TextMatchMarker.
> 
> BUG=707867
> 
> Review-Url: https://codereview.chromium.org/2905753002
> Cr-Commit-Position: refs/heads/master@{#475724}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6b0a494122ebe2fc48099a9549b0...

TBR=yosin@chromium.org,xiaochengh@chromium.org,rlanday@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=707867
==========

to

==========
Revert of [DMC #17] Make TextMatchMarkers constructible in single step (patchset
#4 id:60001 of https://codereview.chromium.org/2905753002/ )

Reason for revert:
This broke the build.

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac%20Builder%...

Original issue's description:
> Make TextMatchMarkers constructible in single step
> 
> Currently, we only construct instances of TextMatchMarker (previously
> RenderedDocumentMarker) inside TextMatchMarkerListImpl, by calling
> TextMatchMarker::Create() on an instance of DocumentMarker. This CL changes
this
> so callers directly construct instances of TextMatchMarker. This sets us up to
> eventually move the storage and accessor methods for whether or not the marker
> is active out of DocumentMarker and into TextMatchMarker.
> 
> BUG=707867
> 
> Review-Url: https://codereview.chromium.org/2905753002
> Cr-Commit-Position: refs/heads/master@{#475724}
> Committed:
https://chromium.googlesource.com/chromium/src/+/6b0a494122ebe2fc48099a9549b0...

TBR=yosin@chromium.org,xiaochengh@chromium.org,rlanday@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=707867

Review-Url: https://codereview.chromium.org/2912133003
Cr-Commit-Position: refs/heads/master@{#475732}
Committed:
https://chromium.googlesource.com/chromium/src/+/182740235efc11d585d6710485d6...
==========

[commit-bot: I haz the power, 2017-05-31 01:49:57]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/182740235efc11d585d6710485d6...