Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(551)

Unified Diff: third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp

Issue 1605863002: Restart search in page when new text is found. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: small clean up Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
diff --git a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
index 199c7b934a8ad47a9344f05dad7e5dd4d64e4228..c37f4e2c2f9e2e0f6459af79cfe47c60e9d0a294 100644
--- a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
+++ b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerControllerTest.cpp
@@ -252,4 +252,20 @@ TEST_F(DocumentMarkerControllerTest, CompositionMarkersNotMerged)
EXPECT_EQ(2u, markerController().markers().size());
}
+TEST_F(DocumentMarkerControllerTest, SetMarkerActiveTest)
+{
+ setBodyInnerHTML("<b>foo</b>");
+ RefPtrWillBeRawPtr<Element> bElement = toElement(document().body()->firstChild());
+ EphemeralRange ephemeralRange = EphemeralRange::rangeOfContents(*bElement);
+ RefPtrWillBeRawPtr<Range> range = Range::create(document(), toPositionInDOMTree(ephemeralRange.startPosition()), toPositionInDOMTree(ephemeralRange.endPosition()));
Finnur 2016/01/29 17:47:49 nit: Break up into two lines for readability?
dvadym 2016/02/01 12:41:10 Done.
+
+ // Try to make active a marker that doesn't exist.
+ EXPECT_FALSE(markerController().setMarkersActive(range, true));
+
+ // Add a marker and try it once more.
+ markerController().addTextMatchMarker(range, false);
+ EXPECT_EQ(1u, markerController().markers().size());
+ EXPECT_TRUE(markerController().setMarkersActive(range, true));
+}
+
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698