Closed Bug 372781 Opened 18 years ago Closed 16 years ago

remove non-cairo-only code for 'transparent' and no alpha colors from style system

Categories

(Core :: CSS Parsing and Computation, defect)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: dbaron, Assigned: dbaron)

References

(Blocks 1 open bug)

Details

Now that non-cairo builds are no longer supported, there's a bunch of style system code we can remove: * mHandleAlphaColors in the CSS parser * all the code that has separate support for 'transparent' keywords for background-color and border-color. Note that the performance tests that are testing our special transparent support should instead test whether the alpha component is zero.
Flags: blocking1.9?
It might be easy to remove -moz-use-text-color at the same time, even though that was obsoleted by currentColor, not by cairo.
Flags: blocking1.9? → blocking1.9-
Whiteboard: [wanted-1.9]
Blocks: 380923
QA Contact: ian → style-system
Flags: wanted1.9+
Whiteboard: [wanted-1.9]
We should also remove -moz-rgba and -moz-hsla. The mHandleAlphaColors stuff is being removed in bug 413632.
Depends on: 413632
Flags: wanted1.9-
Flags: wanted1.9+
Flags: wanted-next+
Removing -moz-use-text-color actually isn't so easy; see bug 271586 (which has a partial patch), especially bug 271586 comment 43.
Depends on: 453916
dbaron: what's left on this one?
Nothing, I suppose.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
And I also landed http://hg.mozilla.org/mozilla-central/rev/84a1b8853494 as a reminder about the remaining issue.
You need to log in before you can comment on or make changes to this bug.