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

Unified Diff: third_party/WebKit/Source/core/dom/DocumentStatisticsCollectorTest.cpp

Issue 2390543002: Reflow comments in core/dom/. (Closed)
Patch Set: Reformat comments in core/dom/. 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/dom/DocumentStatisticsCollectorTest.cpp
diff --git a/third_party/WebKit/Source/core/dom/DocumentStatisticsCollectorTest.cpp b/third_party/WebKit/Source/core/dom/DocumentStatisticsCollectorTest.cpp
index 8ca4596ba53580ba2089686013c79d83b78ad854..6b47d4410f790fe71542499d378b4f0edbba5339 100644
--- a/third_party/WebKit/Source/core/dom/DocumentStatisticsCollectorTest.cpp
+++ b/third_party/WebKit/Source/core/dom/DocumentStatisticsCollectorTest.cpp
@@ -20,7 +20,8 @@ namespace blink {
// Saturate the length of a paragraph to save time.
const unsigned kTextContentLengthSaturation = 1000;
-// Filter out short P elements. The threshold is set to around 2 English sentences.
+// Filter out short P elements. The threshold is set to around 2 English
+// sentences.
const unsigned kParagraphLengthThreshold = 140;
class DocumentStatisticsCollectorTest : public ::testing::Test {
@@ -100,17 +101,26 @@ TEST_F(DocumentStatisticsCollectorTest, CountElements) {
TEST_F(DocumentStatisticsCollectorTest, CountScore) {
setHtmlInnerHTML(
"<p class='menu' id='article'>1</p>" // textContentLength = 1
- "<ul><li><p>12</p></li></ul>" // textContentLength = 2, skipped because under li
- "<p class='menu'>123</p>" // textContentLength = 3, skipped because unlikelyCandidates
+ "<ul><li><p>12</p></li></ul>" // textContentLength = 2, skipped because
+ // under li
+ "<p class='menu'>123</p>" // textContentLength = 3, skipped because
+ // unlikelyCandidates
"<p>"
"12345678901234567890123456789012345678901234567890"
"12345678901234567890123456789012345678901234567890"
"12345678901234567890123456789012345678901234"
"</p>" // textContentLength = 144
- "<p style='display:none'>12345</p>" // textContentLength = 5, skipped because invisible
- "<div style='display:none'><p>123456</p></div>" // textContentLength = 6, skipped because invisible
- "<div style='visibility:hidden'><p>1234567</p></div>" // textContentLength = 7, skipped because invisible
- "<p style='opacity:0'>12345678</p>" // textContentLength = 8, skipped because invisible
+ "<p style='display:none'>12345</p>" // textContentLength = 5, skipped
+ // because invisible
+ "<div style='display:none'><p>123456</p></div>" // textContentLength = 6,
+ // skipped because
+ // invisible
+ "<div style='visibility:hidden'><p>1234567</p></div>" // textContentLength
+ // = 7, skipped
+ // because
+ // invisible
+ "<p style='opacity:0'>12345678</p>" // textContentLength = 8, skipped
+ // because invisible
"<p><a href='#'>1234 </a>6 <b> 9</b></p>" // textContentLength = 9
"<ul><li></li><p>123456789012</p></ul>" // textContentLength = 12
);

Powered by Google App Engine
This is Rietveld 408576698