reflow comments in core/svg/

third_party/WebKit/Source/core/svg/properties/SVGProperty.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 33 matching lines @@
 class SVGPropertyBase : public GarbageCollectedFinalized<SVGPropertyBase> {
   WTF_MAKE_NONCOPYABLE(SVGPropertyBase);
 
  public:
   // Properties do not have a primitive type by default
   typedef void PrimitiveType;
 
   virtual ~SVGPropertyBase() {}
 
   // FIXME: remove this in WebAnimations transition.
-  // This is used from SVGAnimatedNewPropertyAnimator for its animate-by-string implementation.
+  // This is used from SVGAnimatedNewPropertyAnimator for its animate-by-string
+  // implementation.
   virtual SVGPropertyBase* cloneForAnimation(const String&) const = 0;
 
   virtual String valueAsString() const = 0;
 
-  // FIXME: remove below and just have this inherit AnimatableValue in WebAnimations transition.
+  // FIXME: remove below and just have this inherit AnimatableValue in
+  // WebAnimations transition.
   virtual void add(SVGPropertyBase*, SVGElement*) = 0;
   virtual void calculateAnimatedValue(SVGAnimationElement*,
                                       float percentage,
                                       unsigned repeatCount,
                                       SVGPropertyBase* from,
                                       SVGPropertyBase* to,
                                       SVGPropertyBase* toAtEndOfDurationValue,
                                       SVGElement*) = 0;
   virtual float calculateDistance(SVGPropertyBase* to, SVGElement*) = 0;
 
   virtual AnimatedPropertyType type() const = 0;
 
   SVGPropertyBase* ownerList() const { return m_ownerList; }
 
   void setOwnerList(SVGPropertyBase* ownerList) {
     // Previous owner list must be cleared before setting new owner list.
     ASSERT((!ownerList && m_ownerList) || (ownerList && !m_ownerList));
 
     m_ownerList = ownerList;
   }
 
   DEFINE_INLINE_VIRTUAL_TRACE() {}
 
  protected:
   SVGPropertyBase() : m_ownerList(nullptr) {}
 
  private:
-  // Oilpan: the back reference to the owner should be a Member, but this can create
-  // cycles when SVG properties meet the off-heap InterpolationValue hierarchy.
-  // Not tracing it is safe, albeit an undesirable state of affairs.
+  // Oilpan: the back reference to the owner should be a Member, but this can
+  // create cycles when SVG properties meet the off-heap InterpolationValue
+  // hierarchy.  Not tracing it is safe, albeit an undesirable state of affairs.

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

Super nit: one space after periods for consistency.

I think everyone our age learned to type two periods as a holdover from
typewriters or something... I still type two by accident.

There are about 10 instances of this but I've just grouped them into this
comment. If you agree, please grep the patch for ".  "

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

I use vim's gq to join lines manually, and it defaults to two spaces
after periods.

Looks like we deliberately decided that we don't care about this a while
ago: https://codereview.chromium.org/609633004

   // See http://crbug.com/528275 for the detail.
   UntracedMember<SVGPropertyBase> m_ownerList;
 };
 
 #define DEFINE_SVG_PROPERTY_TYPE_CASTS(thisType)            \
   DEFINE_TYPE_CASTS(thisType, SVGPropertyBase, value,       \
                     value->type() == thisType::classType(), \
                     value.type() == thisType::classType());
 
 }  // namespace blink
 
 #endif  // SVGProperty_h