angular.js/test/ng, branch v1.2.5

perf(jqLite): implement and use the `empty` method in place of `html(‘’)`

2013-12-13T10:07:11+00:00

jQuery's elem.html('') is way slower than elem.empty(). As clearing
element contents happens quite often in certain scenarios, switching
to using .empty() provides a significant performance boost when using
Angular with jQuery.

Closes #4457

perf(a): do not link when href or name exists in template

2013-12-13T08:31:25+00:00

Change the a directive to link and hookup a click event only when
there is no href or name in the template element.
In a large Google app, this results in about 800 fewer registrations,
saving a small but measurable amount of time and memory.

Closes #5362

fix(ngInclude): Add template to DOM before linking other directives

2013-12-13T01:18:44+00:00

The template needs to be added to the DOM before
other directives at the same element as `ngInclude` are linked.

Fixes #5247.

fix($compile): Allow literals in isolate scope references

2013-12-13T00:30:31+00:00

When a component uses an isolate scope reference
and the the component is used with an object literal
a new object is created on every evaluation.
Therefore the compiler needs to compare
the values of the parent and the isolate scope
using object equality and not object reference
equality.

Fixes #5296.

fix($compile): update cloned elements if the template arrives after the cloning

2013-12-06T06:16:25+00:00

If an element has a directive whose content is loaded using `templateUrl`,
and the element is cloned using a linking function before the template arrives,
the clone needs to be updated as well.

This also updates `ngIf` and `ngRepeat` to keep the connection to the clone
of a tranclude function, so that they know about the changes a directive with
`templateUrl` does to the element in the future.

Fixes to #4930.

chore(Scope): short-circuit after dirty-checking last dirty watcher

2013-12-05T23:37:37+00:00

Stop dirty-checking during $digest after the last dirty watcher has been re-checked.

This prevents unneeded re-checking of the remaining watchers (They were already
checked in the previous iteration), bringing a substantial performance improvement
to the average case run time of $digest.

Closes #5272
Closes #5287

fix(ngInit): evaluate ngInit before ngInclude

2013-12-05T07:26:56+00:00

The priority of ngInit is adjusted to occur before ngInclude, and after
ngController. This enables ngInit to initiallize values in a controller's
scope, and also to initiallize values before ngInclude executes.

Closes #5167
Closes #5208

fix(input): ensure ngModelWatch() triggers second digest pass when appropriate

2013-12-05T06:49:11+00:00

Due to an earlier change, ngModelWatch() no longer returns a value to the
caller. This means the digest loop has no way to tell if the watch actually
modified anything and so can not schedule another pass.

This means any watches that watch form or model controller changes
(e.g. watches on form.$valid) that are scheduled prior to an ngModelWatch()
will not be able to see any changes made therin.

This commit fixes this behavior by returning the latest evaluated ng-model
value.

Closes #5258
Closes #5282

fix($rootScope): broadcast $destroy event on $rootScope

2013-12-04T23:29:19+00:00

Fixes #5169

fix($rootScope): clear phase if an exception is raised by a watcher

2013-12-04T18:11:29+00:00

Add calls to clearPhase() when an exception is raised by a watcher
while a digest cycle, in order to not be stuck on `$digest` scope phase