From ef875ad0cf4349144cb4674e050dd160564f6dd9 Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Sun, 30 Oct 2011 15:11:25 -0700 Subject: feat(scope): better logging of infinite digest error Feedback team has often problems debugging inifinite digest errors, this change should reveal info about what watchers are causing the infinite loop --- src/Scope.js | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/Scope.js b/src/Scope.js index be5030cc..c4b9513b 100644 --- a/src/Scope.js +++ b/src/Scope.js @@ -260,7 +260,8 @@ Scope.prototype = { watcher = { fn: listenFn, last: Number.NaN, // NaN !== NaN. We used this to force $watch to fire on first run. - get: get + get: get, + exp: watchExp }; if (!array) { @@ -325,7 +326,8 @@ Scope.prototype = { asyncQueue, length, dirty, ttl = 100, - next, current, target = this; + next, current, target = this, + watchLog = []; if (target.$$phase) { throw Error(target.$$phase + ' already in progress'); @@ -356,6 +358,14 @@ Scope.prototype = { dirty = true; watch.last = copy(value); watch.fn(current, value, last); + if (ttl < 5) { + if (!watchLog[4-ttl]) watchLog[4-ttl] = []; + if (isFunction(watch.exp)) { + watchLog[4-ttl].push('fn: ' + (watch.exp.name || watch.exp.toString())); + } else { + watchLog[4-ttl].push(watch.exp); + } + } } } catch (e) { current.$service('$exceptionHandler')(e); @@ -376,7 +386,8 @@ Scope.prototype = { } while ((current = next)); if(!(ttl--)) { - throw Error('100 $digest() iterations reached. Aborting!'); + throw Error('100 $digest() iterations reached. Aborting!\n' + + 'Watchers fired in the last 5 iterations: ' + toJson(watchLog)); } } while (dirty); }, -- cgit v1.2.3