Re: [webkit-dev] Initial painting time changed?

2012-02-01 Thread Nikolas Zimmermann
Am 31.01.2012 um 00:08 schrieb Nikolas Zimmermann: > > Am 30.01.2012 um 23:30 schrieb Simon Fraser: >> >> Are we actually sure that the updateStyleForAllDocuments() changes caused >> this? >> >> Perhaps DRT/WTR need to do something now that they didn't do before. It's definitely not caused b

Re: [webkit-dev] Initial painting time changed?

2012-01-30 Thread Nikolas Zimmermann
Am 30.01.2012 um 23:30 schrieb Simon Fraser: > > Are we actually sure that the updateStyleForAllDocuments() changes caused > this? > > Perhaps DRT/WTR need to do something now that they didn't do before. I didn't try to back it out yet - I hope to find time for it tomorrow. Niko

Re: [webkit-dev] Initial painting time changed?

2012-01-30 Thread Simon Fraser
On Jan 30, 2012, at 9:26 PM, Nikolas Zimmermann wrote: > Am 30.01.2012 um 21:44 schrieb James Robinson: > >> I didn't see any regressions, pixel tests or otherwise, when running with >> this patch, nor did the bots. What platform / test configuration are you >> seeing issues on? > Well, that's

Re: [webkit-dev] Initial painting time changed?

2012-01-30 Thread Nikolas Zimmermann
Am 30.01.2012 um 21:44 schrieb James Robinson: > I didn't see any regressions, pixel tests or otherwise, when running with > this patch, nor did the bots. What platform / test configuration are you > seeing issues on? Well, that's exactly the problem, there are no regressions - if we break re

Re: [webkit-dev] Initial painting time changed?

2012-01-30 Thread James Robinson
On Mon, Jan 30, 2012 at 12:19 PM, Nikolas Zimmermann < zimmerm...@physik.rwth-aachen.de> wrote: > > Am 30.01.2012 um 19:38 schrieb James Robinson: > > >> The document was laid out and painted once before the timeout fired. Most >> SVG repainting tests work this way: setup document initially, then

Re: [webkit-dev] Initial painting time changed?

2012-01-30 Thread Nikolas Zimmermann
Am 30.01.2012 um 19:38 schrieb James Robinson: > > The document was laid out and painted once before the timeout fired. Most SVG > repainting tests work this way: setup document initially, then use a > setTimeout(doSomething, 0) to eg. change an attribute, then dump the document > to capture

Re: [webkit-dev] Initial painting time changed?

2012-01-30 Thread James Robinson
On Mon, Jan 30, 2012 at 3:56 AM, Nikolas Zimmermann < zimmerm...@physik.rwth-aachen.de> wrote: > Good evening WebKit folks, > > while working on some SVG Filter dynamic update issues I found out that > long-standing assumptions about the paint time don't hold anymore: > > > > > function doTest(

Re: [webkit-dev] Initial painting time changed?

2012-01-30 Thread Simon Fraser
Maybe from the Document::updateStyleForAllDocuments() removal ? Simon On Jan 30, 2012, at 11:56 AM, Nikolas Zimmermann wrote: > Good evening WebKit folks, > > while working on some SVG Filter dynamic update issues I found out that > long-standing