Backed out changeset ff36938d2716 (bug 1182723) for bustage.
authorRyan VanderMeulen <ryanvm@gmail.com>
Wed, 22 Jul 2015 22:29:45 -0400
changeset 204945 a6823b984097
parent 204944 ff36938d2716
child 204946 2e6f1d4deff9
push id865
push userryanvm@gmail.com
push dateThu, 23 Jul 2015 02:30:36 +0000
treeherdermozilla-b2g32_v2_0@2e6f1d4deff9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1182723
milestone32.0
backs outff36938d2716
Backed out changeset ff36938d2716 (bug 1182723) for bustage.
xpcom/glue/nsTArray.h
--- a/xpcom/glue/nsTArray.h
+++ b/xpcom/glue/nsTArray.h
@@ -815,19 +815,17 @@ public:
   operator const FallibleTArray<E>&() const {
     return *reinterpret_cast<const FallibleTArray<E>*>(this);
   }
 
   // The array's assignment operator performs a 'deep' copy of the given
   // array.  It is optimized to reuse existing storage if possible.
   // @param other  The array object to copy.
   self_type& operator=(const self_type& other) {
-    if (this != &aOther) {
-      ReplaceElementsAt(0, Length(), other.Elements(), other.Length());
-    }
+    ReplaceElementsAt(0, Length(), other.Elements(), other.Length());
     return *this;
   }
 
   // Return true if this array has the same length and the same
   // elements as |other|.
   template<typename Allocator>
   bool operator==(const nsTArray_Impl<E, Allocator>& other) const {
     size_type len = Length();