Bug 8223 – WebKit fails to update style on element via DOM when value contains !important

• Component: DOM
• Status: RESOLVED FIXED

[James Cox]

Element style set via CSS, and later adapted via a JS DOM call to the style attribute (margin). The update does not take place and the element is not redrawn. 

Works as expected on ie 6 / ffox

[James Cox]

Created attachment 7548 [details]
reduced cast

[Rob Buis]

Created attachment 8559 [details]
First attempt

This patch fixes the problem with !important parsing. I can make a testcase
if the code gets okayed.
Cheers,

Rob.

[Rob Buis]

Created attachment 8577 [details]
Now with testcase

I added a simple, but hopefully correct testcase.
Cheers,

Rob.

[Dave Hyatt]

Comment on attachment 8577 [details]
Now with testcase

The Obj-C bindings need fixing too.  They all have the same bug (repeated over and over again).

- (void)setBackgroundRepeat:(NSString *)backgroundRepeat
{
    [self setProperty:@"background-repeat" :backgroundRepeat :@""];
}

The empty string is passed for priority.  You need to make an Obj-C method for your new setProperty call and then change all the callers to use that instead.

[Dave Hyatt]

Comment on attachment 8577 [details]
Now with testcase

I changed my mind.  These CSS bindings in Obj-C are horribly broken anyway, including properties we don't even support in the engine and leaving out ones that we do.  I think this issue should just be punted until auto-generation can fix it for us.

[Darin Adler]

(In reply to comment #5)
> These CSS bindings in Obj-C are horribly broken anyway,
> including properties we don't even support in the engine and leaving out ones
> that we do.

The list of methods to set CSS properties comes from the DOM specification verbatim. It was not intended to match the list of properties we support.

On the other hand, maybe that should change.

[Timothy Hatcher]

Landed in r14700.

[Alexey Proskuryakov]

This will likely be undone in bug 73941.