[Webkit-unassigned] [Bug 135961] Web Inspector: Add layout test for lowercase CSSProperty names

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 15 15:41:56 PDT 2014


https://bugs.webkit.org/show_bug.cgi?id=135961





--- Comment #4 from Joseph Pecoraro <joepeck at webkit.org>  2014-08-15 15:42:05 PST ---
(From update of attachment 236638)
View in context: https://bugs.webkit.org/attachment.cgi?id=236638&action=review

> Source/WebInspectorUI/UserInterface/Models/CSSCompletions.js:86
> +        // CodeMirror is not included by Test.html, so skip MIME type associations.

This should be a generic comment. "CodeMirror is not included in tests, so skip MIME type associations.". The mention of Test.html is too specific and could go stale.

> LayoutTests/inspector/css/matched-style-properties.html:8
> +    BACKGROUND-COLOR:blue;
> +    COLOR:white;
> +    border-STYLE:solid;

Might as well test an all lower case, and wild camel case.

    position: absolute;
    ToP: 0;
    lEfT: 0;

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list