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

Unified Diff: third_party/WebKit/Source/core/testing/Internals.cpp

Issue 2393013002: reflow comments in core/{clipboard,streams,testing,timing} (Closed)
Patch Set: comments (heh!) 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/testing/Internals.cpp
diff --git a/third_party/WebKit/Source/core/testing/Internals.cpp b/third_party/WebKit/Source/core/testing/Internals.cpp
index 840546dac12637633adb9895e7f155badceeb735..0b0e303fcaea3f32d1e32b04085442a40333669b 100644
--- a/third_party/WebKit/Source/core/testing/Internals.cpp
+++ b/third_party/WebKit/Source/core/testing/Internals.cpp
@@ -742,7 +742,8 @@ ShadowRoot* Internals::createUserAgentShadowRoot(Element* host) {
}
ShadowRoot* Internals::shadowRoot(Element* host) {
- // FIXME: Internals::shadowRoot() in tests should be converted to youngestShadowRoot() or oldestShadowRoot().
+ // FIXME: Internals::shadowRoot() in tests should be converted to
+ // youngestShadowRoot() or oldestShadowRoot().
// https://bugs.webkit.org/show_bug.cgi?id=78465
return youngestShadowRoot(host);
}
@@ -1199,7 +1200,8 @@ Range* Internals::rangeFromLocationAndLength(Element* scope,
unsigned Internals::locationFromRange(Element* scope, const Range* range) {
ASSERT(scope && range);
- // PlainTextRange depends on Layout information, make sure layout it up to date.
+ // PlainTextRange depends on Layout information, make sure layout it up to
+ // date.
scope->document().updateStyleAndLayoutIgnorePendingStylesheets();
return PlainTextRange::create(*scope, *range).start();
@@ -1207,7 +1209,8 @@ unsigned Internals::locationFromRange(Element* scope, const Range* range) {
unsigned Internals::lengthFromRange(Element* scope, const Range* range) {
ASSERT(scope && range);
- // PlainTextRange depends on Layout information, make sure layout it up to date.
+ // PlainTextRange depends on Layout information, make sure layout it up to
+ // date.
scope->document().updateStyleAndLayoutIgnorePendingStylesheets();
return PlainTextRange::create(*scope, *range).length();
@@ -1573,8 +1576,8 @@ static PaintLayer* findLayerForGraphicsLayer(PaintLayer* searchRoot,
return searchRoot;
}
- // Search right to left to increase the chances that we'll choose the top-most layers in a
- // grouped mapping for squashing.
+ // Search right to left to increase the chances that we'll choose the top-most
+ // layers in a grouped mapping for squashing.
for (PaintLayer* child = searchRoot->lastChild(); child;
child = child->previousSibling()) {
PaintLayer* foundLayer =
@@ -1765,7 +1768,8 @@ StaticNodeList* Internals::nodesFromRect(Document* document,
HitTestRequest request(hitType);
- // When ignoreClipping is false, this method returns null for coordinates outside of the viewport.
+ // When ignoreClipping is false, this method returns null for coordinates
+ // outside of the viewport.
if (!request.ignoreClipping() &&
!frameView->visibleContentRect().intersects(HitTestLocation::rectForPoint(
point, topPadding, rightPadding, bottomPadding, leftPadding)))
« no previous file with comments | « third_party/WebKit/Source/core/testing/Internals.h ('k') | third_party/WebKit/Source/core/testing/v8/WebCoreTestSupport.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698