[DMC #17] Make TextMatchMarkers constructible in single step

third_party/WebKit/Source/core/editing/markers/DocumentMarkerTest.cpp

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/editing/markers/DocumentMarker.h"
 
+#include "core/editing/markers/TextMatchMarker.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace blink {
 
 using MarkerOffsets = DocumentMarker::MarkerOffsets;
 
 class DocumentMarkerTest : public ::testing::Test {
  protected:
   DocumentMarker* CreateMarker(unsigned startOffset, unsigned endOffset) {
-    return new DocumentMarker(startOffset, endOffset,
-                              DocumentMarker::MatchStatus::kInactive);
+    return new TextMatchMarker(startOffset, endOffset,

[rlanday, 2017/05/31 02:03:30]

This is the fix: use the new TextMatchMarker constructor here instead of the
now-private DocumentMarker constructor

+                               DocumentMarker::MatchStatus::kInactive);
   }
 };
 
 TEST_F(DocumentMarkerTest, MarkerTypeIteratorEmpty) {
   DocumentMarker::MarkerTypes types(0);
   EXPECT_TRUE(types.begin() == types.end());
 }
 
 TEST_F(DocumentMarkerTest, MarkerTypeIteratorOne) {
   DocumentMarker::MarkerTypes types(DocumentMarker::kSpelling);
@@ skipping 181 matching lines @@
   EXPECT_EQ(WTF::nullopt, result);
 }
 
 TEST_F(DocumentMarkerTest, GetShiftedMarkerPosition_ReplaceBeforeAndEnd) {
   DocumentMarker* marker = CreateMarker(5, 10);
   Optional<MarkerOffsets> result = marker->ComputeOffsetsAfterShift(4, 6, 1);
   EXPECT_EQ(WTF::nullopt, result);
 }
 
 }  // namespace blink