Closed Bug 378535 Opened 17 years ago Closed 17 years ago

[FIX]Implement "content: none"

Categories

(Core :: CSS Parsing and Computation, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9alpha5

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Fix (obsolete) — Splinter Review
It's easy; might as well
Attachment #262584 - Flags: superreview?(dbaron)
Attachment #262584 - Flags: review?(dbaron)
Attached patch With tests tooSplinter Review
Attachment #262585 - Flags: superreview?(dbaron)
Attachment #262585 - Flags: review?(dbaron)
Is the first hunk of the patch for a different bug?
er, yes.  And it's already checked in, in fact...
I'm pretty sure this'll turn a few mochitest todos into passing.
Blocks: 377731
Attachment #262584 - Attachment is obsolete: true
Attachment #262584 - Flags: superreview?(dbaron)
Attachment #262584 - Flags: review?(dbaron)
I'll watch out for it.  ;)
Comment on attachment 262585 [details] [diff] [review]
With tests too

r+sr=dbaron

Please update the mochitest todos in test_value_storage.html and test_value_computation.html (by removing the line that notes content:none is unimplemented), and run the mochitests before checkin.  (See bug 279246 comment 41 for instructions if you need them.)
Attachment #262585 - Flags: superreview?(dbaron)
Attachment #262585 - Flags: superreview+
Attachment #262585 - Flags: review?(dbaron)
Attachment #262585 - Flags: review+
Checked in, with the mochitest changes.
Status: NEW → RESOLVED
Closed: 17 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: mozilla1.9alpha4 → mozilla1.9alpha5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: