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

Unified Diff: third_party/WebKit/Source/core/editing/iterators/TextIteratorTest.cpp

Issue 2397033002: Reflow comments in //third_party/WebKit/Source/core/editing/iterators (Closed)
Patch Set: Created 4 years, 2 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/iterators/TextIteratorTest.cpp
diff --git a/third_party/WebKit/Source/core/editing/iterators/TextIteratorTest.cpp b/third_party/WebKit/Source/core/editing/iterators/TextIteratorTest.cpp
index 6ea422df36aa3001cb36879d937b8dd6ea99fd60..947c2e089b8d51d2e37903eccde624e2f6b1be19 100644
--- a/third_party/WebKit/Source/core/editing/iterators/TextIteratorTest.cpp
+++ b/third_party/WebKit/Source/core/editing/iterators/TextIteratorTest.cpp
@@ -173,7 +173,8 @@ TEST_F(TextIteratorTest, NotEnteringShadowTree) {
setBodyContent(bodyContent);
createShadowRootForElementWithIDAndSetInnerHTML(document(), "host",
shadowContent);
- // TextIterator doesn't emit "text" since its layoutObject is not created. The shadow tree is ignored.
+ // TextIterator doesn't emit "text" since its layoutObject is not created. The
+ // shadow tree is ignored.
EXPECT_EQ("[Hello, ][ iterator.]", iterate<DOMTree>());
EXPECT_EQ("[Hello, ][shadow][ iterator.]", iterate<FlatTree>());
}
@@ -228,7 +229,8 @@ TEST_F(TextIteratorTest, EnteringShadowTreeWithOption) {
setBodyContent(bodyContent);
createShadowRootForElementWithIDAndSetInnerHTML(document(), "host",
shadowContent);
- // TextIterator emits "shadow" since TextIteratorEntersOpenShadowRoots is specified.
+ // TextIterator emits "shadow" since TextIteratorEntersOpenShadowRoots is
+ // specified.
EXPECT_EQ("[Hello, ][shadow][ iterator.]",
iterate<DOMTree>(TextIteratorEntersOpenShadowRoots));
EXPECT_EQ("[Hello, ][shadow][ iterator.]",
@@ -245,7 +247,8 @@ TEST_F(TextIteratorTest, EnteringShadowTreeWithMultipleShadowTreesWithOption) {
shadowContent1);
createShadowRootForElementWithIDAndSetInnerHTML(document(), "host",
shadowContent2);
- // The first isn't emitted because a layoutObject for the first is not created.
+ // The first isn't emitted because a layoutObject for the first is not
+ // created.
EXPECT_EQ("[Hello, ][second shadow][ iterator.]",
iterate<DOMTree>(TextIteratorEntersOpenShadowRoots));
EXPECT_EQ("[Hello, ][second shadow][ iterator.]",
@@ -275,9 +278,11 @@ TEST_F(TextIteratorTest,
"<div>Hello, <span id='host'>text</span> iterator.</div>";
static const char* shadowContent =
"<span><content>content</content> shadow</span>";
- // In this case a layoutObject for "text" is created, and emitted AFTER any nodes in the shadow tree.
- // This order does not match the order of the rendered texts, but at this moment it's the expected behavior.
- // FIXME: Fix this. We probably need pure-renderer-based implementation of TextIterator to achieve this.
+ // In this case a layoutObject for "text" is created, and emitted AFTER any
+ // nodes in the shadow tree. This order does not match the order of the
+ // rendered texts, but at this moment it's the expected behavior.
+ // FIXME: Fix this. We probably need pure-renderer-based implementation of
+ // TextIterator to achieve this.
setBodyContent(bodyContent);
createShadowRootForElementWithIDAndSetInnerHTML(document(), "host",
shadowContent);
@@ -372,7 +377,8 @@ TEST_F(TextIteratorTest, FullyClippedContentsDistributed) {
setBodyContent(bodyContent);
createShadowRootForElementWithIDAndSetInnerHTML(document(), "host",
shadowContent);
- // FIXME: The text below is actually invisible but TextIterator currently thinks it's visible.
+ // FIXME: The text below is actually invisible but TextIterator currently
+ // thinks it's visible.
EXPECT_EQ("[\n][Am I visible?]",
iterate<DOMTree>(TextIteratorEntersOpenShadowRoots));
EXPECT_EQ("", iterate<FlatTree>(TextIteratorEntersOpenShadowRoots));
@@ -391,7 +397,8 @@ TEST_F(TextIteratorTest, IgnoresContainersClipDistributed) {
setBodyContent(bodyContent);
createShadowRootForElementWithIDAndSetInnerHTML(document(), "host",
shadowContent);
- // FIXME: The text below is actually visible but TextIterator currently thinks it's invisible.
+ // FIXME: The text below is actually visible but TextIterator currently thinks
+ // it's invisible.
// [\n][Nobody can find me!]
EXPECT_EQ("", iterate<DOMTree>(TextIteratorEntersOpenShadowRoots));
EXPECT_EQ("[Nobody can find me!]",

Powered by Google App Engine
This is Rietveld 408576698