Digest cycles initiated by the grid

[YonatanKra]

There are way too many digest cycles for the grid load/rendering. I've managed to reduce the load by setting some of the $timeouts to $timeout(method, 0, false). I've managed to reduce the number by 2/3 this way, with no seeming side effects (and my app now loads much faster - I have around 9-10 grids on one page, that can open side menus with more, so this is critical...).

I've created a plnkr that shows that every load, you get roughly 35 digest cycles - with no extra feature involved:
http://plnkr.co/edit/uOYKeJvANJu1b9zk8Ac1?p=catalogue (it also shows that the grid increases the number of digests each time it reloads which might indicate a memory leak, but then again, it might be just the demo app).

For instnace:

ui-grid/src/js/core/directives/ui-grid.js

Line 244 in 1dc9931

$timeout(init);

init is being called iniside a $timeout. It then calls grid.refreshCanvas which in turn does another $timeout (

ui-grid/src/js/core/factories/Grid.js

Line 2121 in a1e6a04

$timeout(function() {

), hence calleing twich to a digest cycle...

As I said, I've managed to reduce this number of cycles with the method above. Is there a special reason why there are so many $timeouts inside the core code?

[swalters]

If all the unit tests work and all of the tutorials work, then I see nothing but positives for this change. You want to submit a PR?

[csvan]

Made a PR for this: #5829

Ping @swalters

[csvan]

@YonatanKra #5829 just got merged, if you are happy with the changes maybe this can be closed.