remove bit rotted comment about css properties
authortony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 17 May 2012 18:34:37 +0000 (18:34 +0000)
committertony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 17 May 2012 18:34:37 +0000 (18:34 +0000)
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):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@117468 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/css/CSSParser.cpp

index 4f5d8b2..b643733 100644 (file)
@@ -1,3 +1,20 @@
+2012-05-17  Tony Chang  <tony@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  <enne@google.com>
 
         Insert source file and line number for v8 function calls into tracing
index 817d3f4..1b250eb 100644 (file)
@@ -1430,15 +1430,6 @@ bool CSSParser::parseValue(CSSPropertyID propId, bool important)
     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);