Make container classes in SkTemplates.h more consistent

tests/TemplatesTest.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkTemplates.h"
 #include "Test.h"
 
@@ skipping 49 matching lines @@
 
     // reset and realloc > stack size
     array.reset(2);
     array.realloc(3);
     array[0] = 1;
     REPORTER_ASSERT(reporter, array[0] == 1);
     array.realloc(1);
     REPORTER_ASSERT(reporter, array[0] == 1);
 }
 
+constexpr int static kStackPreallocCount = 10;
+
+// Ensures the buffers in SkTemplates.h all have a consistent api.
+template<typename TBuffer> static void test_buffer_apis(skiatest::Reporter* reporter) {
+    REPORTER_ASSERT(reporter, !TBuffer(0lu).get());
+    REPORTER_ASSERT(reporter, TBuffer(1).get());
+    REPORTER_ASSERT(reporter, TBuffer(kStackPreallocCount).get());
+    REPORTER_ASSERT(reporter, TBuffer(kStackPreallocCount + 1).get());
+
+    TBuffer buffer;
+    // The default constructor may or may not init to empty, depending on the type of buffer.
+
+    buffer.reset(1);
+    REPORTER_ASSERT(reporter, buffer.get());
+
+    buffer.reset(kStackPreallocCount);
+    REPORTER_ASSERT(reporter, buffer.get());
+
+    buffer.reset(kStackPreallocCount + 1);
+    REPORTER_ASSERT(reporter, buffer.get());
+
+    buffer.reset(0);
+    REPORTER_ASSERT(reporter, !buffer.get());
+}
+
 DEF_TEST(Templates, reporter) {
     test_automalloc_realloc(reporter);
 }
+
+DEF_TEST(TemplateBuffers, reporter) {
+    test_buffer_apis<SkAutoTArray<int> >(reporter);
+    test_buffer_apis<SkAutoSTArray<kStackPreallocCount, int> >(reporter);
+    test_buffer_apis<SkAutoTMalloc<int> >(reporter);
+    test_buffer_apis<SkAutoSTMalloc<kStackPreallocCount, int> >(reporter);
+}