reflow comments in core/imagebitmap,core/html/track

third_party/WebKit/Source/core/html/track/vtt/VTTCue.cpp

 /*
  * Copyright (c) 2013, Opera Software ASA. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 112 matching lines @@
     exceptionState.throwDOMException(
         IndexSizeError,
         ExceptionMessages::indexOutsideRange<double>(
             "value", value, 0, ExceptionMessages::InclusiveBound, 100,
             ExceptionMessages::InclusiveBound));
     return true;
   }
   return false;
 }
 
-// Sets inline CSS properties on passed in element if value is not an empty string
+// Sets inline CSS properties on passed in element if value is not an empty
+// string.
 static void setInlineStylePropertyIfNotEmpty(Element& element,
                                              CSSPropertyID propertyID,
                                              const String& value) {
   if (!value.isEmpty())
     element.setInlineStyleProperty(propertyID, value);
 }
 
 VTTCueBox::VTTCueBox(Document& document)
     : HTMLDivElement(document),
       m_snapToLinesPosition(std::numeric_limits<float>::quiet_NaN()) {
   setShadowPseudoId(AtomicString("-webkit-media-text-track-display"));
 }
 
 void VTTCueBox::applyCSSProperties(
     const VTTDisplayParameters& displayParameters) {
   // http://dev.w3.org/html5/webvtt/#applying-css-properties-to-webvtt-node-objects
 
-  // Initialize the (root) list of WebVTT Node Objects with the following CSS settings:
+  // Initialize the (root) list of WebVTT Node Objects with the following CSS
+  // settings:
 
   // the 'position' property must be set to 'absolute'
   setInlineStyleProperty(CSSPropertyPosition, CSSValueAbsolute);
 
   //  the 'unicode-bidi' property must be set to 'plaintext'
   setInlineStyleProperty(CSSPropertyUnicodeBidi, CSSValueWebkitPlaintext);
 
   // the 'direction' property must be set to direction
   setInlineStyleProperty(CSSPropertyDirection, displayParameters.direction);
 
   // the 'writing-mode' property must be set to writing-mode
   setInlineStyleProperty(CSSPropertyWebkitWritingMode,
                          displayParameters.writingMode);
 
   const FloatPoint& position = displayParameters.position;
 
   // the 'top' property must be set to top,
   setInlineStyleProperty(CSSPropertyTop, position.y(),
                          CSSPrimitiveValue::UnitType::Percentage);
 
   // the 'left' property must be set to left
   setInlineStyleProperty(CSSPropertyLeft, position.x(),
                          CSSPrimitiveValue::UnitType::Percentage);
 
-  // the 'width' property must be set to width, and the 'height' property  must be set to height
+  // the 'width' property must be set to width, and the 'height' property  must
+  // be set to height
   if (displayParameters.writingMode == CSSValueHorizontalTb) {
     setInlineStyleProperty(CSSPropertyWidth, displayParameters.size,
                            CSSPrimitiveValue::UnitType::Percentage);
     setInlineStyleProperty(CSSPropertyHeight, CSSValueAuto);
   } else {
     setInlineStyleProperty(CSSPropertyWidth, CSSValueAuto);
     setInlineStyleProperty(CSSPropertyHeight, displayParameters.size,
                            CSSPrimitiveValue::UnitType::Percentage);
   }
 
@@ skipping 241 matching lines @@
   cueWillChange();
   m_cueAlignment = alignment;
   cueDidChange();
 }
 
 void VTTCue::setText(const String& text) {
   if (m_text == text)
     return;
 
   cueWillChange();
-  // Clear the document fragment but don't bother to create it again just yet as we can do that
-  // when it is requested.
+  // Clear the document fragment but don't bother to create it again just yet as
+  // we can do that when it is requested.
   m_vttNodeTree = nullptr;
   m_text = text;
   cueDidChange();
 }
 
 void VTTCue::createVTTNodeTree() {
   if (!m_vttNodeTree)
     m_vttNodeTree =
         VTTParser::createDocumentFragmentFromCueText(document(), m_text);
 }
@@ skipping 73 matching lines @@
 }
 
 class VTTTextRunIterator : public TextRunIterator {
  public:
   VTTTextRunIterator() {}
   VTTTextRunIterator(const TextRun* textRun, unsigned offset)
       : TextRunIterator(textRun, offset) {}
 
   bool atParagraphSeparator() const {
     // Within a cue, paragraph boundaries are only denoted by Type B characters,
-    // such as U+000A LINE FEED (LF), U+0085 NEXT LINE (NEL), and U+2029 PARAGRAPH SEPARATOR.
+    // such as U+000A LINE FEED (LF), U+0085 NEXT LINE (NEL),
+    // and U+2029 PARAGRAPH SEPARATOR.
     return WTF::Unicode::category(current()) &
            WTF::Unicode::Separator_Paragraph;
   }
 };
 
 // Almost the same as determineDirectionality in core/html/HTMLElement.cpp, but
 // that one uses a "plain" TextRunIterator (which only checks for '\n').
 static TextDirection determineDirectionality(const String& value,
                                              bool& hasStrongDirectionality) {
   TextRun run(value);
@@ skipping 35 matching lines @@
 
 float VTTCue::calculateComputedTextPosition() const {
   // http://dev.w3.org/html5/webvtt/#dfn-cue-computed-position
 
   // 1. If the position is numeric, then return the value of the position and
   // abort these steps. (Otherwise, the position is the special value auto.)
   if (!textPositionIsAuto())
     return m_textPosition;
 
   switch (m_cueAlignment) {
-    // 2. If the cue text alignment is start or left, return 0 and abort these steps.
+    // 2. If the cue text alignment is start or left, return 0 and abort these
+    // steps.
     case Start:
     case Left:
       return 0;
-    // 3. If the cue text alignment is end or right, return 100 and abort these steps.
+    // 3. If the cue text alignment is end or right, return 100 and abort these
+    // steps.
     case End:
     case Right:
       return 100;
     // 4. If the cue text alignment is middle, return 50 and abort these steps.
     case Middle:
       return 50;
     default:
       NOTREACHED();
       return 0;
   }
@@ skipping 182 matching lines @@
   DCHECK(track() && track()->isRendered() && isActive());
 
   if (!m_displayTree) {
     m_displayTree = VTTCueBox::create(document());
     m_displayTree->appendChild(m_cueBackgroundBox);
   }
 
   DCHECK_EQ(m_displayTree->firstChild(), m_cueBackgroundBox);
 
   if (!m_displayTreeShouldChange) {
-    // Apply updated user style overrides for text tracks when display tree doesn't change.
-    // This ensures that the track settings are refreshed when the video is
-    // replayed or when the user slides back to an already rendered track.
+    // Apply updated user style overrides for text tracks when display tree
+    // doesn't change.  This ensures that the track settings are refreshed when
+    // the video is replayed or when the user slides back to an already rendered
+    // track.
     applyUserOverrideCSSProperties();
     return m_displayTree;
   }
 
   createVTTNodeTree();
 
   m_cueBackgroundBox->removeChildren();
   m_vttNodeTree->cloneChildNodes(m_cueBackgroundBox.get());
 
   // TODO(foolip): The region identifier may be non-empty without there being
@@ skipping 53 matching lines @@
   VTTCueBox* displayBox = getDisplayTree();
   VTTRegion* region = 0;
   if (track()->regions())
     region = track()->regions()->getRegionById(regionId());
 
   if (!region) {
     // If cue has an empty region identifier or there is no WebVTT region
     // whose region identifier is identical to cue's region identifier, run
     // the following substeps:
     if (displayBox->hasChildren() && !container.contains(displayBox)) {
-      // Note: the display tree of a cue is removed when the active flag of the cue is unset.
+      // Note: the display tree of a cue is removed when the active flag of the
+      // cue is unset.
       container.appendChild(displayBox);
     }
   } else {
     // Let region be the WebVTT region whose region identifier matches the
     // region identifier of cue.
     HTMLDivElement* regionNode = region->getDisplayTree(document());
 
     // Append the region to the viewport, if it was not already.
     if (!container.contains(regionNode))
       container.appendChild(regionNode);
@@ skipping 40 matching lines @@
   //    number. Let that number be the percentage.
   // 8. If percentage is outside the range 0..100, then fail.
   // 9. Return percentage.
   return input.scanPercentage(number) && !isInvalidPercentage(number);
 }
 
 void VTTCue::parseSettings(const String& inputString) {
   VTTScanner input(inputString);
 
   while (!input.isAtEnd()) {
-    // The WebVTT cue settings part of a WebVTT cue consists of zero or more of the following components, in any order,
-    // separated from each other by one or more U+0020 SPACE characters or U+0009 CHARACTER TABULATION (tab) characters.
+    // The WebVTT cue settings part of a WebVTT cue consists of zero or more of
+    // the following components, in any order, separated from each other by one
+    // or more U+0020 SPACE characters or U+0009 CHARACTER TABULATION (tab)
+    // characters.
     input.skipWhile<VTTParser::isValidSettingDelimiter>();
 
     if (input.isAtEnd())
       break;
 
-    // When the user agent is to parse the WebVTT settings given by a string input for a text track cue cue,
+    // When the user agent is to parse the WebVTT settings given by a string
+    // input for a text track cue cue,
     // the user agent must run the following steps:
     // 1. Let settings be the result of splitting input on spaces.
-    // 2. For each token setting in the list settings, run the following substeps:
-    //    1. If setting does not contain a U+003A COLON character (:), or if the first U+003A COLON character (:)
-    //       in setting is either the first or last character of setting, then jump to the step labeled next setting.
-    //    2. Let name be the leading substring of setting up to and excluding the first U+003A COLON character (:) in that string.
+    // 2. For each token setting in the list settings, run the following
+    //    substeps:
+    //    1. If setting does not contain a U+003A COLON character (:), or if the
+    //       first U+003A COLON character (:) in setting is either the first or
+    //       last character of setting, then jump to the step labeled next
+    //       setting.
+    //    2. Let name be the leading substring of setting up to and excluding
+    //       the first U+003A COLON character (:) in that string.
     CueSetting name = settingName(input);
 
-    // 3. Let value be the trailing substring of setting starting from the character immediately after the first U+003A COLON character (:) in that string.
+    // 3. Let value be the trailing substring of setting starting from the
+    //    character immediately after the first U+003A COLON character (:) in
+    //    that string.
     VTTScanner::Run valueRun =
         input.collectUntil<VTTParser::isValidSettingDelimiter>();
 
-    // 4. Run the appropriate substeps that apply for the value of name, as follows:
+    // 4. Run the appropriate substeps that apply for the value of name, as
+    //    follows:
     switch (name) {
       case Vertical: {
         // If name is a case-sensitive match for "vertical"
         // 1. If value is a case-sensitive match for the string "rl", then
         //    let cue's WebVTT cue writing direction be vertical
         //    growing left.
         if (input.scanRun(valueRun, verticalGrowingLeftKeyword()))
           m_writingDirection = VerticalGrowingLeft;
 
         // 2. Otherwise, if value is a case-sensitive match for the string
         //    "lr", then let cue's WebVTT cue writing direction be
         //    vertical growing right.
         else if (input.scanRun(valueRun, verticalGrowingRightKeyword()))
           m_writingDirection = VerticalGrowingRight;
         break;
       }
       case Line: {
         // If name is a case-sensitive match for "line"
         // Steps 1 - 2 skipped.
         float number;
         // 3. If linepos does not contain at least one ASCII digit, then
         //    jump to the step labeled next setting.
-        // 4. If the last character in linepos is a U+0025 PERCENT SIGN character (%)
+        // 4. If the last character in linepos is a U+0025 PERCENT SIGN
+        //    character (%)
         //
         //    If parse a percentage string from linepos doesn't fail, let
         //    number be the returned percentage, otherwise jump to the step
         //    labeled next setting.
         bool isPercentage = scanPercentage(input, number);
         if (!isPercentage) {
           // Otherwise
           //
           // 1. If linepos contains any characters other than U+002D
           //    HYPHEN-MINUS characters (-) and ASCII digits, then jump to
@@ skipping 133 matching lines @@
 }
 
 DEFINE_TRACE(VTTCue) {
   visitor->trace(m_vttNodeTree);
   visitor->trace(m_cueBackgroundBox);
   visitor->trace(m_displayTree);
   TextTrackCue::trace(visitor);
 }
 
 }  // namespace blink