[Webkit-unassigned] [Bug 110348] New: Web Inspector: CSSProperty.status defaults to "style" not "active"

[bugzilla-daemon at webkit.org]

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

           Summary: Web Inspector: CSSProperty.status defaults to "style"
                    not "active"
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Web Inspector
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: timothy at apple.com
                CC: keishi at webkit.org, pmuellr at yahoo.com,
                    joepeck at webkit.org, pfeldman at chromium.org,
                    yurys at chromium.org, apavlov at chromium.org,
                    loislo at chromium.org, vsevik at chromium.org,
                    web-inspector-bugs at googlegroups.com, graouts at apple.com


The code and the json file description disagree on what an absent "status" means.

Inspector.json:
                    { "name": "status", "type": "string", "enum": ["active", "inactive", "disabled", "style"], "optional": true, "description": "The property status: \"active\" (implied if absent) if the property is effective in the style, \"inactive\" if the property is overridden by a same-named property in this style later on, \"disabled\" if the property is disabled by the user, \"style\" if the property is reported by the browser rather than by the CSS source parser." },

InspectorStyleSheet.cpp:
        // Default "status" == "style".
        if (status != TypeBuilder::CSS::CSSProperty::Status::Style)
            property->setStatus(status);

Defaulting to "style" makes more sense to me, and the code agrees.

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