97771 - getComputedStyle answers for background-color #ffffff instead of transparent

Status: RESOLVED FIXED

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

Description

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

1. use test case attached
2. click on all anchors
3. all values ok but the 4th one ; answer should be "transparent"

Comment 1

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Attachment: test case showing bug

Comment 2

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Attachment: patch v1

Comment 3

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

bug ==> myself ; patch proposed for review...

Comment 4

[Pierre Saslawsky]

The 'transparent' keyword should be obtained from 
nsCSSProps::kBackgroundColorKTable[]

Comment 5

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Attachment: patch v2

Comment 6

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

/me should stop jetlagging... Pierre: new version attached

Comment 7

[Pierre Saslawsky]

r=pierre

Comment 8

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

I just discovered that the query of getComputedStyle on the body of a html
document answers, with my patch, "transparent". See
http://lxr.mozilla.org/seamonkey/source/content/html/content/src/nsHTMLBodyElement.cpp#609
to understand why. Yet another patch coming soon...

(btw, working on CSS in Composer, and therefore using DOM 2 Style on large scale
 is __really__ helpful for debugging)

Comment 9

[Johnny Stenback (:jst)]

Comment on attachment 47806 [details] [diff] [review]
patch v2

sr=jst

Comment 10

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Comment on attachment 47806 [details] [diff] [review]
patch v2

patch in attachment #47806 [details] [diff] [review] obsolete because of comment 2001-08-31 08:15

Comment 11

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Attachment: patch v3

Comment 12

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

pPierre : can you review patch v3, please ?
jst : can you confirm sr= on new patch please ?

Comment 13

[Pierre Saslawsky]

Comment on attachment 48284 [details] [diff] [review]
patch v3

r=pierre

Comment 14

[David Hyatt]

Comment on attachment 48284 [details] [diff] [review]
patch v3

sr=hyatt.  yay! i just ran into this today! i'm glad you're fixing it!

Comment 15

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Checked in trunk