125716 - Implement getComputedStyle() for 'overflow' property

Status: RESOLVED FIXED

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

Description

[Christopher Aillon (sabbatical, not receiving bugmail)]

Comment 1

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: Patch

Implement overflow, and also add eCSSProperty_visibility to
|queryableProperties| as it was missing.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 69657 [details] [diff] [review]
Patch

Please leave the nsComputedDOMStyle::GetOverflow(nsAWritableString& aOverflow)
in there.

I'd put this next to the clip implementation.

Also, that looks like it'll output "" instead of
"auto" for auto overflow.

Comment 3

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: Better patch addressing bz's comments

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 69714 [details] [diff] [review]
Better patch addressing bz's comments

r=bzbarsky

Comment 5

[Marc Attinasi]

Comment on attachment 69714 [details] [diff] [review]
Better patch addressing bz's comments

sr=attinasi

Comment 6

[Christopher Aillon (sabbatical, not receiving bugmail)]

Fixed.