Remove the recalc style timer

Remove the recalc style timer

This CL removes the recalc style time and instead drives style recalcuation off
of requestAnimationFrame. Previously, the recalc timer might fire in the middle
of a frame, which could cause us to recalc style twice: once in the middle of
the frame and again just before putting up the frame. Driving recalc using
requestAnimationFrame avoids this problem by deferring scheduled recalcs until
it's time to put up the frame.

BUG=337617
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=165988

[abarth-chromium, 2014-01-23 02:33:38]

Currently failing three tests:

+css3/device-adapt/opera/constrain-009.html	expected actual diff pretty diff
wdiff		text	pass
+css3/device-adapt/opera/constrain-010.html	expected actual diff pretty diff
wdiff		text	pass
+inspector/layer-tree-model.html	expected actual diff pretty diff
wdiff		text	pass

[eseidel, 2014-01-23 04:51:41]

If you're successful here, you can tackle the big burrito with
https://codereview.chromium.org/19782002/

[abarth-chromium, 2014-01-23 05:19:10]

On 2014/01/23 04:51:41, eseidel wrote:
> If you're successful here, you can tackle the big burrito with
> https://codereview.chromium.org/19782002/

I haven't seen that one show up in the Silk pages yet.  This one does though.

[abarth-chromium, 2014-01-27 20:34:25]

This CL is ready for review now.

[esprehn, 2014-01-27 20:49:42]

lgtm, amazing.

https://codereview.chromium.org/145133006/diff/100004/LayoutTests/inspector/l...
File LayoutTests/inspector/layer-tree-model.html (right):

https://codereview.chromium.org/145133006/diff/100004/LayoutTests/inspector/l...
LayoutTests/inspector/layer-tree-model.html:60:
InspectorTest.evaluateAndRunWhenTreeChanges("void(0)", function() {
Woah, crazy.

[commit-bot: I haz the power, 2014-01-27 21:02:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/abarth@chromium.org/145133006/100004

[commit-bot: I haz the power, 2014-01-27 22:21:33]

Retried try job too often on linux_blink_rel for step(s) webkit_unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[commit-bot: I haz the power, 2014-01-28 00:00:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/abarth@chromium.org/145133006/100005

[commit-bot: I haz the power, 2014-01-28 03:09:32]

Retried try job too often on win_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[commit-bot: I haz the power, 2014-01-28 03:50:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/abarth@chromium.org/145133006/100005

[esprehn, 2014-01-28 03:51:35]

It looks like you may have made
virtual/threaded/animations/display-none-terminates-animation.html fail on
windows? Not sure how though...

[abarth-chromium, 2014-01-28 04:33:02]

On 2014/01/28 03:51:35, esprehn wrote:
> It looks like you may have made
> virtual/threaded/animations/display-none-terminates-animation.html fail on
> windows? Not sure how though...

I be these setTimeouts need to be rafs:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

[commit-bot: I haz the power, 2014-01-28 05:46:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/abarth@chromium.org/145133006/150001

[commit-bot: I haz the power, 2014-01-28 07:17:30]

Retried try job too often on linux_blink_rel for step(s) blink_heap_unittests,
blink_platform_unittests, webkit_lint, webkit_python_tests, webkit_unit_tests,
wtf_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[commit-bot: I haz the power, 2014-01-28 07:22:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/abarth@chromium.org/145133006/150001

[commit-bot: I haz the power, 2014-01-28 08:32:18]

Change committed as 165918

[gavinp, 2014-01-28 14:18:06]

On 2014/01/28 08:32:18, I haz the power (commit-bot) wrote:
> Change committed as 165918

I'm a bit concerned that this patch caused failures in a number of builders
http://build.chromium.org/p/chromium.webkit/builders/Linux%20Tests%20%28dbg%29 ,
in paricular in PasswordAutofillAgentTest.InlineAutocomplete  and
BrowserPluginTest.ResizeFlowControl , so I'm rolling it out. Please tell me why
I'm wrong if I am!

[gavinp, 2014-01-28 14:20:08]

On 2014/01/28 14:18:06, gavinp wrote:
> On 2014/01/28 08:32:18, I haz the power (commit-bot) wrote:
> > Change committed as 165918
> 
> I'm a bit concerned that this patch caused failures in a number of builders
>
http://build.chromium.org/p/chromium.webkit/builders/Linux%2520Tests%2520%252...
,
> in paricular in PasswordAutofillAgentTest.InlineAutocomplete  and
> BrowserPluginTest.ResizeFlowControl , so I'm rolling it out. Please tell me
why
> I'm wrong if I am!

Reverted in r165934.

[commit-bot: I haz the power, 2014-01-29 02:25:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/abarth@chromium.org/145133006/150001

[commit-bot: I haz the power, 2014-01-29 02:26:34]

Change committed as 165988

[commit-bot: I haz the power, 2014-01-29 02:28:19]

CQ bit was unchecked on CL. Ignoring.