rewrite TArray test to not delete an SkSTArray via a SkTArray pointer.

tests/TArrayTest.cpp

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkTArray.h"
 #include "Test.h"
 
@@ skipping 40 matching lines @@
     // { 0, 1, 2, 3 }, remove a middle, note shuffle
     a.removeShuffle(1);
     REPORTER_ASSERT(reporter, a.count() == SK_ARRAY_COUNT(b) - 2);
     REPORTER_ASSERT(reporter, a[0] == 0);
     REPORTER_ASSERT(reporter, a[1] == 3);
     REPORTER_ASSERT(reporter, a[2] == 2);
 
     // {0, 3, 2 }
 }
 
-namespace {
-SkTArray<int>* make() {
-    typedef SkTArray<int> IntArray;
-    return new IntArray;
-}
+static void test_swap(skiatest::Reporter* reporter) {
+    SkTArray<int> arr;
+    SkSTArray< 5, int> arr5;
+    SkSTArray<10, int> arr10;
+    SkSTArray<20, int> arr20;
 
-template <int N> SkTArray<int>* make_s() {
-    typedef SkSTArray<N, int> IntArray;
-    return new IntArray;
-}
-}
+    SkTArray<int>* arrays[] = { &arr, &arr5, &arr10, &arr20 };
+    int sizes[] = {0, 1, 5, 10, 15, 20, 25};
 
-static void test_swap(skiatest::Reporter* reporter) {
-    typedef SkTArray<int>* (*ArrayMaker)();
-    ArrayMaker arrayMakers[] = {make, make_s<5>, make_s<10>, make_s<20>};
-    static int kSizes[] = {0, 1, 5, 10, 15, 20, 25};
-    for (size_t arrayA = 0; arrayA < SK_ARRAY_COUNT(arrayMakers); ++arrayA) {
-        for (size_t arrayB = arrayA; arrayB < SK_ARRAY_COUNT(arrayMakers); ++arrayB) {
-            for (size_t dataSizeA = 0; dataSizeA < SK_ARRAY_COUNT(kSizes); ++dataSizeA) {
-                for (size_t dataSizeB = 0; dataSizeB < SK_ARRAY_COUNT(kSizes); ++dataSizeB) {
-                    int curr = 0;
-                    SkTArray<int>* a = arrayMakers[arrayA]();
-                    SkTArray<int>* b = arrayMakers[arrayB]();
-                    for (int i = 0; i < kSizes[dataSizeA]; ++i) {
-                        a->push_back(curr++);
-                    }
-                    for (int i = 0; i < kSizes[dataSizeB]; ++i) {
-                        b->push_back(curr++);
-                    }
-                    a->swap(b);
-                    REPORTER_ASSERT(reporter, kSizes[dataSizeA] == b->count());
-                    REPORTER_ASSERT(reporter, kSizes[dataSizeB] == a->count());
-                    curr = 0;
-                    for (int i = 0; i < kSizes[dataSizeA]; ++i) {
-                        REPORTER_ASSERT(reporter, curr++ == (*b)[i]);
-                    }
-                    for (int i = 0; i < kSizes[dataSizeB]; ++i) {
-                        REPORTER_ASSERT(reporter, curr++ == (*a)[i]);
-                    }
-                    delete b;
+    for (auto a : arrays) {
+    for (auto b : arrays) {
+        if (a == b) {
+            continue;
+        }
 
-                    a->swap(a);
-                    curr = kSizes[dataSizeA];
-                    for (int i = 0; i < kSizes[dataSizeB]; ++i) {
-                        REPORTER_ASSERT(reporter, curr++ == (*a)[i]);
-                    }
-                    delete a;
-                }
-            }
-        }
-    }
+        for (auto sizeA : sizes) {
+        for (auto sizeB : sizes) {
+            a->reset();
+            b->reset();
+
+            int curr = 0;
+            for (int i = 0; i < sizeA; i++) { a->push_back(curr++); }
+            for (int i = 0; i < sizeB; i++) { b->push_back(curr++); }
+
+            a->swap(b);
+            REPORTER_ASSERT(reporter, b->count() == sizeA);
+            REPORTER_ASSERT(reporter, a->count() == sizeB);
+
+            curr = 0;
+            for (int x : *b) { REPORTER_ASSERT(reporter, x == curr++); }
+            for (int x : *a) { REPORTER_ASSERT(reporter, x == curr++); }
+
+            a->swap(a);
+            curr = sizeA;
+            for (int x : *a) { REPORTER_ASSERT(reporter, x == curr++); }
+        }}
+    }}
 }
 
 DEF_TEST(TArray, reporter) {
     TestTSet_basic<true>(reporter);
     TestTSet_basic<false>(reporter);
     test_swap(reporter);
 }