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

Unified Diff: third_party/WebKit/Source/core/html/parser/HTMLParserIdioms.h

Issue 2386893002: Reformat comments in core/html/parser (Closed)
Patch Set: self review 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/html/parser/HTMLParserIdioms.h
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLParserIdioms.h b/third_party/WebKit/Source/core/html/parser/HTMLParserIdioms.h
index 37e17246e74f3188cfb2d41946e702ba68daede3..f10637e86ab515851a71274c329a3f82c9b38584 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLParserIdioms.h
+++ b/third_party/WebKit/Source/core/html/parser/HTMLParserIdioms.h
@@ -10,16 +10,16 @@
* notice, this list of conditions and the following disclaimer in the
* documentation and/or other materials provided with the distribution.
*
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
* WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
- * DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE FOR ANY
- * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
- * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
- * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON
- * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
- * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ * DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE FOR
+ * ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
#ifndef HTMLParserIdioms_h
@@ -40,13 +40,16 @@ namespace blink {
// Strip leading and trailing whitespace as defined by the HTML specification.
String stripLeadingAndTrailingHTMLSpaces(const String&);
-// An implementation of the HTML specification's algorithm to convert a number to a string for number and range types.
+// An implementation of the HTML specification's algorithm to convert a number
+// to a string for number and range types.
String serializeForNumberType(const Decimal&);
String serializeForNumberType(double);
-// Convert the specified string to a decimal/double. If the conversion fails, the return value is fallback value or NaN if not specified.
-// Leading or trailing illegal characters cause failure, as does passing an empty string.
-// The double* parameter may be 0 to check if the string can be parsed without getting the result.
+// Convert the specified string to a decimal/double. If the conversion fails,
+// the return value is fallback value or NaN if not specified. Leading or
+// trailing illegal characters cause failure, as does passing an empty string.
+// The double* parameter may be 0 to check if the string can be parsed without
+// getting the result.
Decimal parseToDecimalForNumberType(
const String&,
const Decimal& fallbackValue = Decimal::nan());
@@ -70,15 +73,17 @@ WTF::TextEncoding encodingFromMetaAttributes(const HTMLAttributeList&);
// Space characters as defined by the HTML specification.
template <typename CharType>
inline bool isHTMLSpace(CharType character) {
- // Histogram from Apple's page load test combined with some ad hoc browsing some other test suites.
+ // Histogram from Apple's page load test combined with some ad hoc browsing
+ // some other test suites.
//
// 82%: 216330 non-space characters, all > U+0020
// 11%: 30017 plain space characters, U+0020
// 5%: 12099 newline characters, U+000A
// 2%: 5346 tab characters, U+0009
//
- // No other characters seen. No U+000C or U+000D, and no other control characters.
- // Accordingly, we check for non-spaces first, then space, then newline, then tab, then the other characters.
+ // No other characters seen. No U+000C or U+000D, and no other control
+ // characters. Accordingly, we check for non-spaces first, then space, then
+ // newline, then tab, then the other characters.
return character <= ' ' &&
(character == ' ' || character == '\n' || character == '\t' ||
@@ -128,4 +133,5 @@ inline static String attemptStaticStringCreation(const String str) {
}
} // namespace blink
-#endif
+
+#endif // HTMLParserIdioms_h

Powered by Google App Engine
This is Rietveld 408576698