reflow comments in core/svg/

third_party/WebKit/Source/core/svg/properties/SVGListPropertyTearOffHelper.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 31 matching lines @@
 
  public:
   typedef ItemProperty ItemPropertyType;
   typedef typename ItemPropertyType::TearOffType ItemTearOffType;
 
   static ItemPropertyType* getValueForInsertionFromTearOff(
       ItemTearOffType* newItem,
       SVGElement* contextElement,
       const QualifiedName& attributeName) {
     // |newItem| is immutable, OR
-    // |newItem| belongs to a SVGElement, but it does not belong to an animated list
-    // (for example: "textElement.x.baseVal.appendItem(rectElement.width.baseVal)")
-    // Spec: If newItem is already in a list, then a new object is created with the same values as newItem and this item is inserted into the list.
+    // |newItem| belongs to a SVGElement, but it does not belong to an animated
+    // list

[pdr., 2016/10/04 21:05:30]

Nit: put "e.g.," on the previous line and split up the parenthesis. Or, if you
can squeeze it on one line just remove the quotes.

[Nico, 2016/10/04 21:10:23]

Done.

+    // (e.g.  "textElement.x.baseVal.appendItem(rectElement.width.baseVal)")
+    // Spec: If newItem is already in a list, then a new object is created with
+    // the same values as newItem and this item is inserted into the list.
     // Otherwise, newItem itself is inserted into the list.
     if (newItem->isImmutable() || newItem->target()->ownerList() ||
         newItem->contextElement()) {
       // We have to copy the incoming |newItem|,
-      // Otherwise we'll end up having two tearoffs that operate on the same SVGProperty. Consider the example below:
-      // SVGRectElements SVGAnimatedLength 'width' property baseVal points to the same tear off object
-      // that's inserted into SVGTextElements SVGAnimatedLengthList 'x'. textElement.x.baseVal.getItem(0).value += 150 would
-      // mutate the rectElement width _and_ the textElement x list. That's obviously wrong, take care of that.
+      // Otherwise we'll end up having two tearoffs that operate on the same
+      // SVGProperty. Consider the example below: SVGRectElements
+      // SVGAnimatedLength 'width' property baseVal points to the same tear off
+      // object that's inserted into SVGTextElements SVGAnimatedLengthList 'x'.
+      // textElement.x.baseVal.getItem(0).value += 150 would mutate the
+      // rectElement width _and_ the textElement x list. That's obviously wrong,
+      // take care of that.
       return newItem->target()->clone();
     }
 
     newItem->attachToSVGElementAttribute(contextElement, attributeName);
     return newItem->target();
   }
 
   static ItemTearOffType* createTearOff(ItemPropertyType* value,
                                         SVGElement* contextElement,
                                         PropertyIsAnimValType propertyIsAnimVal,
@@ skipping 133 matching lines @@
                                      QualifiedName::null());
   }
 
  private:
   Derived* toDerived() { return static_cast<Derived*>(this); }
 };
 
 }  // namespace blink
 
 #endif  // SVGListPropertyTearOffHelper_h