Eliminate DocumentMarker and TextMatchMarker copy constructors

We decided DocumentMarkers shouldn't ever be copied. After
https://codereview.chromium.org/2905753002, we no longer need to copy
DocumentMarkers to create TextMatchMarkers, so we can remove the copy
constructor. We can also remove the TextMatchMarker constructor that takes a
DocumentMarker.

We need to remove the copy constructor anyway because we're turning
DocumentMarker into a polymorphic class hierarchy, and copying the base class
will no longer produce correct behavior.

BUG=707867

Review-Url: https://codereview.chromium.org/2902393003
Cr-Commit-Position: refs/heads/master@{#475824}
4 files changed