Closed Bug 925193 Opened 11 years ago Closed 11 years ago

domUtils.getCSSValuesForProperty() does not return "unset" in autocomplete values

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: miker, Assigned: heycam)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

getCSSValuesForProperty() fails to return "unset" in autocomplete values.

const DOMUtils = Cc["@mozilla.org/inspector/dom-utils;1"].getService(Ci.inIDOMUtils);
DOMUtils.getCSSValuesForProperty("color").indexOf("unset"); // returns -1
Er, yes, we needed to add "unset" to http://hg.mozilla.org/mozilla-central/file/a141e39bf6da/layout/inspector/src/inDOMUtils.cpp#l595

I hate this duplication stuff.  :(
Ah, right.  Lucky we don't add CSS-wide keywords very often.
Assignee: nobody → cam
Attached patch patchSplinter Review
Attachment #815189 - Flags: review?(bzbarsky)
Comment on attachment 815189 [details] [diff] [review]
patch

r=me especially if you fix the tests?
Attachment #815189 - Flags: review?(bzbarsky) → review+
Depends on: 925218
(In reply to Boris Zbarsky [:bz] (Vacation Oct 12 - Oct 27) from comment #4)
> r=me especially if you fix the tests?

Yes that would help wouldn't it.  I'll update layout/inspector/tests/test_bug877690.html before landing.
https://hg.mozilla.org/mozilla-central/rev/4dad2edca27e
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: