perf(merge): remove unnecessary wrapping of jqLite element

gkalpak · gkalpak · commit ce6a96b0d76d · 2015-11-03T17:49:54.000+02:00
Fixes 75292a6#commitcomment-14137538 Closes #13236
diff --git a/src/Angular.js b/src/Angular.js
@@ -363,7 +363,7 @@ function baseExtend(dst, objs, deep) {
         } else if (src.nodeName) {
           dst[key] = src.cloneNode(true);
         } else if (isElement(src)) {
-          dst[key] = jqLite(src).clone();
+          dst[key] = src.clone();
         } else {
           if (!isObject(dst[key])) dst[key] = isArray(src) ? [] : {};
           baseExtend(dst[key], [src], true);
diff --git a/test/AngularSpec.js b/test/AngularSpec.js
@@ -626,8 +626,10 @@ describe('angular', function() {
 
 
     it('should copy(clone) elements', function() {
-      var src = { element: document.createElement('div'),
-        jqObject: jqLite("<p><span>s1</span><span>s2</span></p>").find("span") };
+      var src = {
+        element: document.createElement('div'),
+        jqObject: jqLite('<p><span>s1</span><span>s2</span></p>').find('span')
+      };
       var dst = {};
 
       merge(dst, src);
@@ -638,7 +640,7 @@ describe('angular', function() {
       expect(isElement(dst.element)).toBeTruthy();
       expect(dst.element.nodeName).toBeDefined(); // i.e it is a DOM element
       expect(isElement(dst.jqObject)).toBeTruthy();
-      expect(dst.jqObject.nodeName).toBeUndefined(dst.jqObject); // i.e it is a jqLite/jquery object
+      expect(dst.jqObject.nodeName).toBeUndefined(); // i.e it is a jqLite/jQuery object
     });
   });
 
