Title: [276386] trunk/Source/WebCore
Revision
276386
Author
[email protected]
Date
2021-04-21 12:45:22 -0700 (Wed, 21 Apr 2021)

Log Message

CSSComputedStyleDeclaration.cpp should use C++ style comments
https://bugs.webkit.org/show_bug.cgi?id=224875

Patch by Tyler Wilcock <[email protected]> on 2021-04-21
Reviewed by Darin Adler.

* css/CSSComputedStyleDeclaration.cpp:
(WebCore::ComputedStyleExtractor::valueForPropertyInStyle):
Use // instead of /* */ for comments, as this is suggested WebKit
coding style.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (276385 => 276386)


--- trunk/Source/WebCore/ChangeLog	2021-04-21 19:44:43 UTC (rev 276385)
+++ trunk/Source/WebCore/ChangeLog	2021-04-21 19:45:22 UTC (rev 276386)
@@ -1,3 +1,15 @@
+2021-04-21  Tyler Wilcock  <[email protected]>
+
+        CSSComputedStyleDeclaration.cpp should use C++ style comments
+        https://bugs.webkit.org/show_bug.cgi?id=224875
+
+        Reviewed by Darin Adler.
+
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::ComputedStyleExtractor::valueForPropertyInStyle):
+        Use // instead of /* */ for comments, as this is suggested WebKit
+        coding style.
+
 2021-04-21  Aditya Keerthi  <[email protected]>
 
         [iOS][FCR] Adjust border radius for large buttons and <select>

Modified: trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp (276385 => 276386)


--- trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp	2021-04-21 19:44:43 UTC (rev 276385)
+++ trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp	2021-04-21 19:45:22 UTC (rev 276386)
@@ -3906,7 +3906,7 @@
         }
 #endif
 
-        /* Individual properties not part of the spec */
+        // Individual properties not part of the spec
         case CSSPropertyBackgroundRepeatX:
         case CSSPropertyBackgroundRepeatY:
             break;
@@ -3947,13 +3947,13 @@
         case CSSPropertyQuotes:
             return valueForQuotes(style.quotes());
 
-        /* Unimplemented CSS 3 properties (including CSS3 shorthand properties) */
+        // Unimplemented CSS 3 properties (including CSS3 shorthand properties)
         case CSSPropertyAll:
         case CSSPropertyAnimation:
         case CSSPropertyWebkitTextEmphasis:
             break;
 
-        /* Directional properties are resolved by resolveDirectionAwareProperty() before the switch. */
+        // Directional properties are resolved by resolveDirectionAwareProperty() before the switch.
         case CSSPropertyBorderBlockEnd:
         case CSSPropertyBorderBlockEndColor:
         case CSSPropertyBorderBlockEndStyle:
@@ -4015,18 +4015,18 @@
         case CSSPropertyAdditiveSymbols:
             break;
 
-        /* Unimplemented @font-face properties */
+        // Unimplemented @font-face properties
         case CSSPropertySrc:
         case CSSPropertyUnicodeRange:
         case CSSPropertyFontDisplay:
             break;
 
-        /* Other unimplemented properties */
+        // Other unimplemented properties
         case CSSPropertyPage: // for @page
         case CSSPropertySize: // for @page
             break;
 
-        /* Unimplemented -webkit- properties */
+        // Unimplemented -webkit- properties
         case CSSPropertyWebkitBorderRadius:
         case CSSPropertyWebkitMarginCollapse:
         case CSSPropertyWebkitMarqueeDirection:
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to