DevTools: [CSS] do not treat random css comments as disabled properties

third_party/WebKit/Source/devtools/front_end/sdk/CSSStyleModel.js

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 1421 matching lines @@
 
         /**
          * @param {string} token
          * @param {?string} tokenType
          * @param {number} column
          * @param {number} newColumn
          */
         function processToken(token, tokenType, column, newColumn)
         {
             if (!insideProperty) {
-                var isDisabledProperty = tokenType && tokenType.includes("css-comment") && token.includes(":");
+                var disabledProperty = tokenType && tokenType.includes("css-comment") && isDisabledProperty(token);
                 var isPropertyStart = tokenType && (tokenType.includes("css-meta") || tokenType.includes("css-property"));
-                if (isDisabledProperty) {
+                if (disabledProperty) {
                     result = result.trimRight() + indentation + token;
                 } else if (isPropertyStart) {
                     insideProperty = true;
                     propertyText = token;
                 } else if (token !== ";") {
                     result += token;
                 }
                 return;
             }
 
             if (token === "}" || token === ";") {
                 result = result.trimRight() + indentation + propertyText.trim() + ";";
                 insideProperty = false;
                 if (token === "}")
                     result += "}";
             } else {
                 propertyText += token;
             }
         }
+
+        /**
+         * @param {string} text
+         * @return {boolean}
+         */
+        function isDisabledProperty(text)
+        {
+            var colon = text.indexOf(":");
+            if (colon === -1)
+                return false;
+            var propertyName = text.substring(2, colon).trim();
+            return CSS.supports(propertyName, "initial") || propertyName.startsWith("-moz-") || propertyName.startsWith("-o-") || propertyName.startsWith("-webkit-") || propertyName.startsWith("-ms-");

[pfeldman, 2016/01/04 21:27:01]

You should either query the target (generated) css property set or whitelist all
of the supported ones (get them from the cm highlight).

[lushnikov, 2016/01/11 23:52:11]

How are your suggestions better than this? 

+        }
     },
 
     /**
      * @param {string} text
      * @return {string}
      */
     _detectIndentation: function(text)
     {
         var lines = text.split("\n");
         if (lines.length < 2)
@@ skipping 906 matching lines @@
  * @constructor
  * @param {?WebInspector.CSSStyleDeclaration} inlineStyle
  * @param {?WebInspector.CSSStyleDeclaration} attributesStyle
  */
 WebInspector.CSSStyleModel.InlineStyleResult = function(inlineStyle, attributesStyle)
 {
     this.inlineStyle = inlineStyle;
     this.attributesStyle = attributesStyle;
 }