From 99d40323577c87db9cebc49b14a3bb48f44793f7 Mon Sep 17 00:00:00 2001
From: Caitlin Potter <caitpotter88@gmail.com>
Date: Sun, 10 Aug 2014 02:58:09 -0400
Subject: [PATCH] fix($compile): make '='-bindings NaN-aware

Update parent and child scopes correctly when a '='-binding changes from a NaN value.
---
 src/ng/compile.js      |  2 +-
 test/ng/compileSpec.js | 22 ++++++++++++++++++++++
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/src/ng/compile.js b/src/ng/compile.js
index c95a1032f06d..1d6c62650327 100644
--- a/src/ng/compile.js
+++ b/src/ng/compile.js
@@ -1507,7 +1507,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
                 if (parentGet.literal) {
                   compare = equals;
                 } else {
-                  compare = function(a,b) { return a === b; };
+                  compare = function(a,b) { return a === b || (a !== a && b !== b); };
                 }
                 parentSet = parentGet.assign || function() {
                   // reset the change, or we will throw this exception on every $digest
diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js
index fe8257aaafa9..d955835f6117 100755
--- a/test/ng/compileSpec.js
+++ b/test/ng/compileSpec.js
@@ -2869,6 +2869,28 @@ describe('$compile', function() {
     });
 
 
+    it('should update parent scope when "="-bound NaN changes', inject(function($compile, $rootScope) {
+      $rootScope.num = NaN;
+      compile('<div my-component reference="num"></div>');
+      var isolateScope = element.isolateScope();
+      expect(isolateScope.reference).toBeNaN();
+
+      isolateScope.$apply(function(scope) { scope.reference = 64; });
+      expect($rootScope.num).toBe(64);
+    }));
+
+
+    it('should update isolate scope when "="-bound NaN changes', inject(function($compile, $rootScope) {
+      $rootScope.num = NaN;
+      compile('<div my-component reference="num"></div>');
+      var isolateScope = element.isolateScope();
+      expect(isolateScope.reference).toBeNaN();
+
+      $rootScope.$apply(function(scope) { scope.num = 64; });
+      expect(isolateScope.reference).toBe(64);
+    }));
+
+
     describe('bind-once', function () {
 
       function countWatches(scope) {