Bug 12827 - REGRESSION(r19183): -[DOMCSSStyleDeclaration setValue:forKey:] doesn't have an immediate effect
Summary: REGRESSION(r19183): -[DOMCSSStyleDeclaration setValue:forKey:] doesn't have a...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Geoffrey Garen
URL:
Keywords: HasReduction, Regression
Depends on:
Blocks:
 
Reported: 2007-02-19 23:26 PST by Timothy Hatcher
Modified: 2007-02-20 13:43 PST (History)
1 user (show)

See Also:

Attachments
Testcase (32.39 KB, application/zip)
2007-02-19 23:26 PST, Timothy Hatcher 		no flags Details
patch (1.36 KB, patch)
2007-02-20 12:18 PST, Geoffrey Garen 		mitz: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 2007-02-19 23:26:03 PST 
- Build and run the attached project, and click Move Up.
- In Tiger, when you click on the "move up" button, you will see that "Test"  immediately moves up.
- In TOT, you will not see the change until you move your mouse button over the WebView.

This is not just a problem with the setValue:forKey: or setValue:forKeyPath: cases, the direct setProperty:value:priority: method also fails to layout or render the change.

Using the nightlies, this regressed between r19136 and r19186.
Comment 1 Timothy Hatcher 2007-02-19 23:26:40 PST 
Created attachment 13267 [details]
Testcase
Comment 2 Timothy Hatcher 2007-02-20 01:06:34 PST 
This was caused by r19183.
Comment 3 Geoffrey Garen 2007-02-20 11:03:24 PST 
I'll have a look.
Comment 4 Geoffrey Garen 2007-02-20 12:18:24 PST 
Created attachment 13278 [details]
patch
Comment 5 mitz 2007-02-20 12:47:00 PST 
Comment on attachment 13278 [details]
patch

!(r!=me)
Comment 6 Geoffrey Garen 2007-02-20 13:43:54 PST 
Committed revision 19742.