Remove the Media Query "forward compatibly syntax" support in alignment with HTML5 spec

Source/core/css/MediaList.cpp

Index: Source/core/css/MediaList.cpp
diff --git a/Source/core/css/MediaList.cpp b/Source/core/css/MediaList.cpp
index 07ef23df72ecc3621a174de22c4639da9d25eb7f..449b08c267c2430562b156e4fdcd0367d8db8893 100644
--- a/Source/core/css/MediaList.cpp
+++ b/Source/core/css/MediaList.cpp
@@ -38,28 +38,20 @@ namespace WebCore {
  *
  * Media Queries, Media Types and Media Descriptors.
  *
- * Slight problem with syntax error handling:
+ * Media queries, as described in the Media Queries Level 3 specification, build on
+ * the mechanism outlined in HTML4. The syntax of media queries fit into the media
+ * type syntax reserved in HTML4. The media attribute of HTML4 also exists in XHTML
+ * and generic XML. The same syntax can also be used inside in the ‘@media’ and

[apavlov, 2013/06/04 14:17:48]

extraneous "in" ("...inside in the @media...").

You also use '’' for quoting, which seems to be a non-ASCII character

+ * ‘@import’ rules of CSS.
+ *
+ * However, the parsing rules for media queries are incompatible with those of HTML4
+ * so that they are consistent with those of media queries used in CSS.

[apavlov, 2013/06/04 14:17:48]

"so that" sounds a bit confusing here. Were you trying to say that even though
the parsing rules are incompatible with those specified in HTML4, they are
consistent with those of media queries used in CSS? Then just replace "so that
they" by "and" (...are incompatible ... and are consistent...)

+ *
+ * HTML5 (at the moment of writing still work in progress) references the Media Queries
+ * specification directly and thus updates the rules for HTML.
+ *
  * CSS 2.1 Spec (http://www.w3.org/TR/CSS21/media.html)
- * specifies that failing media type parsing is a syntax error
  * CSS 3 Media Queries Spec (http://www.w3.org/TR/css3-mediaqueries/)
- * specifies that failing media query is a syntax error
- * HTML 4.01 spec (http://www.w3.org/TR/REC-html40/present/styles.html#adef-media)
- * specifies that Media Descriptors should be parsed with forward-compatible syntax
- * DOM Level 2 Style Sheet spec (http://www.w3.org/TR/DOM-Level-2-Style/)
- * talks about MediaList.mediaText and refers
- *   -  to Media Descriptors of HTML 4.0 in context of StyleSheet
- *   -  to Media Types of CSS 2.0 in context of CSSMediaRule and CSSImportRule
- *
- * These facts create situation where same (illegal) media specification may result in
- * different parses depending on whether it is media attr of style element or part of
- * css @media rule.
- * <style media="screen and resolution > 40dpi"> ..</style> will be enabled on screen devices where as
- * @media screen and resolution > 40dpi {..} will not.
- * This gets more counter-intuitive in JavaScript:
- * document.styleSheets[0].media.mediaText = "screen and resolution > 40dpi" will be ok and
- * enabled, while
- * document.styleSheets[0].cssRules[0].media.mediaText = "screen and resolution > 40dpi" will
- * throw SYNTAX_ERR exception.
  */
 
 MediaQuerySet::MediaQuerySet()
@@ -89,22 +81,6 @@ MediaQuerySet::~MediaQuerySet()
 {
 }
 
-static String parseMediaDescriptor(const String& string)
-{
-    // http://www.w3.org/TR/REC-html40/types.html#type-media-descriptors
-    // "Each entry is truncated just before the first character that isn't a
-    // US ASCII letter [a-zA-Z] (ISO 10646 hex 41-5a, 61-7a), digit [0-9] (hex 30-39),
-    // or hyphen (hex 2d)."
-    unsigned length = string.length();
-    unsigned i = 0;
-    for (; i < length; ++i) {
-        unsigned short c = string[i];
-        if (!((c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z') || (c >= '1' && c <= '9') || (c == '-')))
-            break;
-    }
-    return string.left(i);
-}
-
 PassOwnPtr<MediaQuery> MediaQuerySet::parseMediaQuery(const String& queryString, MediaQueryParserMode mode)
 {
     CSSParser parser(CSSStrictMode);
@@ -114,12 +90,6 @@ PassOwnPtr<MediaQuery> MediaQuerySet::parseMediaQuery(const String& queryString,
         return parsedQuery.release();
 
     switch (mode) {
-    case MediaQueryForwardCompatibleSyntaxMode: {
-        String medium = parseMediaDescriptor(queryString);
-        if (!medium.isNull())
-            return adoptPtr(new MediaQuery(MediaQuery::None, medium, nullptr));
-        // Fall through.
-    }
     case MediaQueryNormalMode:
         return adoptPtr(new MediaQuery(MediaQuery::None, "not all", nullptr));
     case MediaQueryStrictMode: