Title: [117468] trunk/Source/WebCore
Revision
117468
Author
t...@chromium.org
Date
2012-05-17 11:34:37 -0700 (Thu, 17 May 2012)

Log Message

remove bit rotted comment about css properties
https://bugs.webkit.org/show_bug.cgi?id=86750

Reviewed by Alexis Menard.

It's not clear to me what the first block was even referring to.
Aural doesn't seem to exist anymore. It might be worth keeping,
"All the CSS properties are not supported by the renderer at the
moment.", but it seems obvious to me.

No new tests, just removing a comment.

* css/CSSParser.cpp:
(WebCore::CSSParser::parseValue):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (117467 => 117468)


--- trunk/Source/WebCore/ChangeLog	2012-05-17 18:33:33 UTC (rev 117467)
+++ trunk/Source/WebCore/ChangeLog	2012-05-17 18:34:37 UTC (rev 117468)
@@ -1,3 +1,20 @@
+2012-05-17  Tony Chang  <t...@chromium.org>
+
+        remove bit rotted comment about css properties
+        https://bugs.webkit.org/show_bug.cgi?id=86750
+
+        Reviewed by Alexis Menard.
+
+        It's not clear to me what the first block was even referring to.
+        Aural doesn't seem to exist anymore. It might be worth keeping,
+        "All the CSS properties are not supported by the renderer at the
+        moment.", but it seems obvious to me.
+
+        No new tests, just removing a comment.
+
+        * css/CSSParser.cpp:
+        (WebCore::CSSParser::parseValue):
+
 2012-05-17  Adrienne Walker  <e...@google.com>
 
         Insert source file and line number for v8 function calls into tracing

Modified: trunk/Source/WebCore/css/CSSParser.cpp (117467 => 117468)


--- trunk/Source/WebCore/css/CSSParser.cpp	2012-05-17 18:33:33 UTC (rev 117467)
+++ trunk/Source/WebCore/css/CSSParser.cpp	2012-05-17 18:34:37 UTC (rev 117468)
@@ -1430,15 +1430,6 @@
     RefPtr<CSSValue> parsedValue;
 
     switch (propId) {
-        /* The comment to the left defines all valid value of this properties as defined
-         * in CSS 2, Appendix F. Property index
-         */
-
-        /* All the CSS properties are not supported by the renderer at the moment.
-         * Note that all the CSS2 Aural properties are only checked, if CSS_AURAL is defined
-         * (see parseAuralValues). As we don't support them at all this seems reasonable.
-         */
-
     case CSSPropertySize:                 // <length>{1,2} | auto | [ <page-size> || [ portrait | landscape] ]
         return parseSize(propId, important);
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to