57524 - [FIX]styleSheet .disabled not reset when <meta> element changes preferred stylesheet [AltSS]

Status: RESOLVED FIXED

(Core :: DOM: CSS Object Model, defect, P1)

Description

[Hixie (not reading bugmail)]

STEPS TO REPRODUCE
   Cause the following to happen during a document load:
     1. A preferred stylesheet is declared.
     2. The default style is set to another title.
     3. The new preferred stylesheet is declared.
   The attached testcase does this with something like these three lines:
      <style type="text/css" title="Alternate"></style>
      <meta http-equiv="Default-Style" content="Preferred">
      <style type="text/css" title="Preferred"></style>
   ...where each line corresponds to the respective step above.

EXPECTED RESULTS
   The first stylesheet should be disabled, and the DOM should say it is 
   disabled; the second should be enabled. Thus going to View|Use Stylesheet
   should show a single bullet next to the last item.

ACTUAL RESULTS
   The first stylesheet is correctly disabled, but the DOM claims that both 
   stylesheets are enabled. That is, the "disabled" attribute of the styleSheet
   interface is 'false' for both stylesheets. This can be seen in the alternate
   stylesheets menu as both items are selected.

The testcase attempts to show this. The "marked" and "actually" comments should
be the same. At the moment, the results read:
   1. Alternate Stylesheet (marked ENABLED, actually DISABLED)
   2. Preferred Stylesheet (marked ENABLED, actually ENABLED)
It _should_ read:
   1. Alternate Stylesheet (marked DISABLED, actually DISABLED)
   2. Preferred Stylesheet (marked ENABLED, actually ENABLED)
...that is the alternate stylesheet line should say it is marked DISABLED. This
works fine if you actually _switch_ (using the menu) to one of the stylesheets.

Tested with Windows 2000 commercial branch bits 2000102017 and scc's Mac bits
on Thursday night. This bug is visible on scc's restricted access Journal page.

Comment 1

[Hixie (not reading bugmail)]

Attachment: Self explanatory testcase as described in initial description.

Comment 2

[Johnny Stenback (:jst)]

Talking this from claytons list, and --> Future

Comment 3

[Hixie (not reading bugmail)]

(marking alternate style sheet bugs for easy tracking...)

Comment 4

[Hixie (not reading bugmail)]

Taking QA Contact on all open or unverified DOM Style bugs...

Comment 5

[Boris Zbarsky [:bzbarsky]]

Attachment: Proposed fix

Tell the sheet we're disabling it instead of just disabling.

Comment 6

[Peter Van der Beken [:peterv]]

Comment on attachment 71232 [details] [diff] [review]
Proposed fix

r=peterv

Comment 7

[Boris Zbarsky [:bzbarsky]]

taking bug

Comment 8

[Johnny Stenback (:jst)]

Comment on attachment 71232 [details] [diff] [review]
Proposed fix

sr=jst

Comment 9

[Boris Zbarsky [:bzbarsky]]

*** This bug has been marked as a duplicate of 124042 ***

Comment 10

[Boris Zbarsky [:bzbarsky]]

um... that was a typo.  :)

Comment 11

[Asa Dotzler [:asa]]

Comment on attachment 71232 [details] [diff] [review]
Proposed fix

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 12

[Boris Zbarsky [:bzbarsky]]

checked in on trunk.