From 5b7177c2658e0c054245edb7b34576daba62046f Mon Sep 17 00:00:00 2001
From: Pete Bacon Darwin <pete@bacondarwin.com>
Date: Mon, 15 Jul 2013 12:34:13 +0100
Subject: [PATCH] fix($compile): remove redundant attr.specified

attr.specified is deprecated in Gecko 7.0 and creates unwanted
warnings on FireFox.  It seems that modern browsers, but possibly not
IE, only provide specified attributes when calling element.getAttributes()
and so removing this is unlikely to break applications.
See https://developer.mozilla.org/en-US/docs/Web/API/Attr and
http://msdn.microsoft.com/en-us/library/ie/ms534637(v=vs.85).aspx

Closes #2160
---
 src/ng/compile.js | 42 ++++++++++++++++++++----------------------
 1 file changed, 20 insertions(+), 22 deletions(-)

diff --git a/src/ng/compile.js b/src/ng/compile.js
index b33e830a42c0..12ec7322ffdd 100644
--- a/src/ng/compile.js
+++ b/src/ng/compile.js
@@ -524,29 +524,27 @@ function $CompileProvider($provide) {
             var index;
 
             attr = nAttrs[j];
-            if (attr.specified) {
-              name = attr.name;
-              // support ngAttr attribute binding
-              ngAttrName = directiveNormalize(name);
-              if (NG_ATTR_BINDING.test(ngAttrName)) {
-                name = ngAttrName.substr(6).toLowerCase();
-              }
-              if ((index = ngAttrName.lastIndexOf('Start')) != -1 && index == ngAttrName.length - 5) {
-                attrStartName = name;
-                attrEndName = name.substr(0, name.length - 5) + 'end';
-                name = name.substr(0, name.length - 6);
-              }
-              nName = directiveNormalize(name.toLowerCase());
-              attrsMap[nName] = name;
-              attrs[nName] = value = trim((msie && name == 'href')
-                ? decodeURIComponent(node.getAttribute(name, 2))
-                : attr.value);
-              if (getBooleanAttrName(node, nName)) {
-                attrs[nName] = true; // presence means true
-              }
-              addAttrInterpolateDirective(node, directives, value, nName);
-              addDirective(directives, nName, 'A', maxPriority, ignoreDirective, attrStartName, attrEndName);
+            name = attr.name;
+            // support ngAttr attribute binding
+            ngAttrName = directiveNormalize(name);
+            if (NG_ATTR_BINDING.test(ngAttrName)) {
+              name = ngAttrName.substr(6).toLowerCase();
+            }
+            if ((index = ngAttrName.lastIndexOf('Start')) != -1 && index == ngAttrName.length - 5) {
+              attrStartName = name;
+              attrEndName = name.substr(0, name.length - 5) + 'end';
+              name = name.substr(0, name.length - 6);
+            }
+            nName = directiveNormalize(name.toLowerCase());
+            attrsMap[nName] = name;
+            attrs[nName] = value = trim((msie && name == 'href')
+              ? decodeURIComponent(node.getAttribute(name, 2))
+              : attr.value);
+            if (getBooleanAttrName(node, nName)) {
+              attrs[nName] = true; // presence means true
             }
+            addAttrInterpolateDirective(node, directives, value, nName);
+            addDirective(directives, nName, 'A', maxPriority, ignoreDirective, attrStartName, attrEndName);
           }
 
           // use class as directive