fix($injector): remove INSTANTIATING entry when done

getService flags services as INSTANTIATING while it calls their
provider factory, in order to detect circular dependencies. If
the service is instantiated correctly, the INSTANTIATING flag is
overwritten with the actual service. However, if the service is
not instantiated correctly, the INSTANTIATING flag should still
be removed, or all further requests for this service will be
mis-detected as a circular dependency.

Closes #4361
Closes #5577
test(injector): add missing test for #5577

Add a missing test for fix that was merged via #5577
test(compileSpec): fix broken build on FF

FF 26.0 now throws:

"TypeError: NodeList doesn't have an indexed property setter."

when we try to assign to `childNodes[1]`, since this test still works properly
on Chrome and the issue being tested is not a cross-browser issues, I'm
just making the patchability check more robust instead of trying to figure
out how to make this test fully pass on FF.
docs(README): add analytics
docs: moar analytics for all md files
fix(animate): remove trailing `s`
fix($httpBackend): Ignore multiple calls to onreadystatechange with readyState=4

On mobile webkit `onreadystatechange` might by called multiple times
with `readyState===4`  caused by xhrs that are resolved while the app is
in the background.

 Fixes #5426.
test(rootScope): reorganize $watch deregistration specs into a describe
fix(Scope): don't let watch deregistration mess up the dirty-checking digest loop

Closes #5525
docs($compile): fix typo

Closes #5599
fix(ngRepeat): allow for more flexible coding style in ngRepeat expression

With this change it's possible to split the ng-repeat expression into multiple
lines at any point in the expression where white-space is expected.

Closes #5537
Closes #5598
fix($browser): remove base href domain when url begins with '//'

This change prevents an incorrect appBase url from being calculated when the
<base> href's domain begins with '//'.

Closes #5606
style($browser): remove ws, fix typo
fix($location): $location.path() behaviour when $locationChangeStart is triggered by the browser

Fixed inconsistency in $location.path() behaviour on the $locationChangeStart event when using
back/forward buttons in the browser or manually changing the url in the address bar.
$location.path() now returns the target url in these cases.

Closes #4989
Closes #5089
Closes #5118
Closes #5580
docs(error/ngRepeat/dupes): fix typo

Closes #5610
fix($httpBackend): use ActiveX XHR when making PATCH requests on IE8

IE8's native XHR doesn't support PATCH requests, but the ActiveX one does.

I'm also removing the noxhr error doc because nobody will ever get that error.

Closes #2518
Closes #5043
fix(httpBackend): fix 'type mismatch' error on IE8 after each request
fix(ngShow/ngHide, ngIf): functions with zero args should be truthy

Previously, expressions that were a function with one or more arguments evaluated to
true, but functions with zero arguments evaluated to false.

This behavior seems both unintentional and undesirable. This patch makes a function
truthy regardless of its number of arguments.

Closes #5414
fix($resource): prevent URL template from collapsing into an empty string

if url template would result in an empty string, we should make a request
to '/' instead.

Closes #5455
Closes #5493
docs(\$sniffer): fix minor typo

Closes #5544