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

Unified Diff: third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp

Issue 2001083002: Explicit management of FrameSelection availability (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 2016-06-08T18:08:39 Created 4 years, 6 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/FrameSelectionTest.cpp
diff --git a/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp b/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
index 8af22adb147e20698180d31f99a74417b32b65d1..190500163a856aec66976048dd04c1b49a9df2ea 100644
--- a/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
+++ b/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
@@ -69,25 +69,6 @@ TEST_F(FrameSelectionTest, SetValidSelection)
EXPECT_FALSE(selection().isNone());
}
-TEST_F(FrameSelectionTest, SetInvalidSelection)
-{
- // Create a new document without frame by using DOMImplementation.
- DocumentInit dummy;
- Document* documentWithoutFrame = Document::create();
- Element* body = HTMLBodyElement::create(*documentWithoutFrame);
- documentWithoutFrame->appendChild(body);
- Text* anotherText = documentWithoutFrame->createTextNode("Hello, another world");
- body->appendChild(anotherText);
-
- // Create a new VisibleSelection for the new document without frame and
- // update FrameSelection with the selection.
- VisibleSelection invalidSelection;
- invalidSelection.setWithoutValidation(Position(anotherText, 0), Position(anotherText, 5));
- setSelection(invalidSelection);
-
- EXPECT_TRUE(selection().isNone());
-}
-
TEST_F(FrameSelectionTest, InvalidateCaretRect)
{
Text* text = appendTextNode("Hello, World!");

Powered by Google App Engine
This is Rietveld 408576698