Last Comment Bug 649119 - remove parsing and storage of CSS 2.0 aural style sheet properties
: remove parsing and storage of CSS 2.0 aural style sheet properties
Status: RESOLVED FIXED
: dev-doc-complete
Product: Core
Classification: Components
Component: CSS Parsing and Computation (show other bugs)
: Trunk
: All All
: P4 normal (vote)
: mozilla6
Assigned To: David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
:
Mentors:
Depends on:
Blocks: 264517
  Show dependency treegraph
 
Reported: 2011-04-11 14:08 PDT by David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
Modified: 2011-04-30 16:19 PDT (History)
6 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (57.52 KB, patch)
2011-04-11 14:49 PDT, David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
bzbarsky: review+
Details | Diff | Review

Description David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2011-04-11 14:08:09 PDT
We currently parse and store the CSS 2.0 aural style sheet properties.  We've never computed the values, though (bug 47159), which means they're not useful for anything.  The support we have is also way off from the current approach to aural CSS at http://dev.w3.org/csswg/css3-speech/ .

This code is getting in the way of cleaning up nsComputedDOMStyle (bug 264517), because I'd like to be able to assume that we have computed style implementations for all properties we support.
Comment 1 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2011-04-11 14:49:24 PDT
Created attachment 525181 [details] [diff] [review]
patch
Comment 2 Boris Zbarsky [:bz] (Out June 25-July 6) 2011-04-21 08:18:02 PDT
Comment on attachment 525181 [details] [diff] [review]
patch

r=me
Comment 3 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2011-04-21 20:25:34 PDT
https://hg.mozilla.org/mozilla-central/rev/b5832780b4c2
Comment 4 Jean-Yves Perrier [:teoli] 2011-04-24 01:12:57 PDT
I've added the dev-doc-needed flag as there was some documentation about this in the MDC.

See (non exhaustive list):
https://developer.mozilla.org/en/CSS/azimuth
https://developer.mozilla.org/en/CSS/Aural
https://developer.mozilla.org/fr/CSS/azimuth

I think Aural should be replaced by a text saying it is deprecated and the pages for azimuth should be simply atomized.

We'll see what Sheppy thinks about this.
Comment 5 Eric Shepherd [:sheppy] 2011-04-24 18:33:32 PDT
Here's what I've done:

https://developer.mozilla.org/en/CSS/azimuth

Updated with a compatibility table that explains the state of support in Firefox; also added a note that the page needs updating, and tagged it as such as well.

Also added a note to Firefox 6 for developers.

Note You need to log in before you can comment on or make changes to this bug.