Only schedule a widget update when necessary

Only schedule a widget update when necessary

The vast majority of the Timers scheduled in Blink come
from this one call in performPostLayoutTasks, at least
according to my survey of the key_silk_cases.

However updateWidget() already early-returns if
there are no widgets to update.  This moves the
timer scheduling into its own function (so that the
FROM_HERE result is more descriptive) and also
checks the same early-return as updateWidget, just
before bothering to schedule the timer.

I do not expect this to have a performance impact, since
scheduling a timer should be cheap and in the common case
(no plugins) these timers would always just immediately
exit on firing but at least removing this timer schedule
will reduce chatter in our trace files if nothing else.

NOTRY=true
BUG=350592
R=abarth@chromium.org

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

[abarth-chromium, 2014-03-10 23:32:04]

lgtm

[abarth-chromium, 2014-03-10 23:32:20]

I bet this saves a bunch of power.

[eseidel, 2014-03-10 23:32:20]

The CQ bit was checked by eseidel@chromium.org

[eseidel, 2014-03-10 23:39:58]

It's juicy!

[eseidel, 2014-03-10 23:45:33]

The CQ bit was checked by eseidel@chromium.org

[commit-bot: I haz the power, 2014-03-10 23:45:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eseidel@chromium.org/193413002/1

[tonyg, 2014-03-11 00:02:38]

Nice!

Reducing wakeups should be very juicy indeed. We can keep an eye on one of the
energy graphs to find out just how juicy:
https://chromeperf.appspot.com/report?rev=215611&masters=ChromiumPerf&bots=ch...

[commit-bot: I haz the power, 2014-03-11 01:02:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-11 01:02:50]

Try jobs failed on following builders: linux_blink

[eseidel, 2014-03-11 02:37:20]

The CQ bit was checked by eseidel@chromium.org

[commit-bot: I haz the power, 2014-03-11 02:37:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eseidel@chromium.org/193413002/1

[eseidel, 2014-03-11 04:44:37]

Sorry CQ, I give up on you.

[eseidel, 2014-03-11 04:44:49]

The CQ bit was unchecked by eseidel@chromium.org

[eseidel, 2014-03-11 04:44:50]

The CQ bit was checked by eseidel@chromium.org

[eseidel, 2014-03-11 05:07:54]

Committed patchset #1 manually as r168883 (presubmit successful).