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

Unified Diff: third_party/WebKit/Source/core/frame/FrameSerializer.cpp

Issue 2393313002: reflow comments in core/frame (Closed)
Patch Set: tweak 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/frame/FrameSerializer.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameSerializer.cpp b/third_party/WebKit/Source/core/frame/FrameSerializer.cpp
index b694504695cc6e64cf6efa9c8f6d394bb4ca66f2..7812a8d0e713d8a496a0fb5136bc14c972fb3157 100644
--- a/third_party/WebKit/Source/core/frame/FrameSerializer.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameSerializer.cpp
@@ -157,7 +157,8 @@ void SerializerMarkupAccumulator::appendElement(StringBuilder& result,
if (!shouldIgnoreElement(element))
MarkupAccumulator::appendElement(result, element, namespaces);
- // TODO(tiger): Refactor MarkupAccumulator so it is easier to append an element like this, without special cases for XHTML
+ // TODO(tiger): Refactor MarkupAccumulator so it is easier to append an
+ // element like this, without special cases for XHTML
if (isHTMLHeadElement(element)) {
result.append("<meta http-equiv=\"Content-Type\" content=\"");
appendAttributeValue(result, m_document->suggestedMIMEType());
@@ -169,7 +170,8 @@ void SerializerMarkupAccumulator::appendElement(StringBuilder& result,
result.append("\">");
}
- // FIXME: For object (plugins) tags and video tag we could replace them by an image of their current contents.
+ // FIXME: For object (plugins) tags and video tag we could replace them by an
+ // image of their current contents.
}
void SerializerMarkupAccumulator::appendAttribute(StringBuilder& out,
@@ -233,7 +235,8 @@ void SerializerMarkupAccumulator::appendRewrittenAttribute(
m_elementsWithRewrittenLinks.add(&element);
// Append the rewritten attribute.
- // TODO(tiger): Refactor MarkupAccumulator so it is easier to append an attribute like this.
+ // TODO(tiger): Refactor MarkupAccumulator so it is easier to append an
+ // attribute like this.
out.append(' ');
out.append(attributeName);
out.append("=\"");
@@ -290,7 +293,8 @@ void FrameSerializer::serializeFrame(const LocalFrame& frame) {
continue;
Element& element = toElement(*node);
- // We have to process in-line style as it might contain some resources (typically background images).
+ // We have to process in-line style as it might contain some resources
+ // (typically background images).
if (element.isStyledElement()) {
retrieveResourcesForProperties(element.inlineStyle(), document);
retrieveResourcesForProperties(element.presentationAttributeStyle(),
@@ -487,9 +491,9 @@ void FrameSerializer::retrieveResourcesForProperties(
if (!styleDeclaration)
return;
- // The background-image and list-style-image (for ul or ol) are the CSS properties
- // that make use of images. We iterate to make sure we include any other
- // image properties there might be.
+ // The background-image and list-style-image (for ul or ol) are the CSS
+ // properties that make use of images. We iterate to make sure we include any
+ // other image properties there might be.
unsigned propertyCount = styleDeclaration->propertyCount();
for (unsigned i = 0; i < propertyCount; ++i) {
const CSSValue& cssValue = styleDeclaration->propertyAt(i).value();
« no previous file with comments | « third_party/WebKit/Source/core/frame/FrameSerializer.h ('k') | third_party/WebKit/Source/core/frame/FrameView.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698