Use ScopedVector instead of std::vector in case of repeated messages.

base/json/json_value_converter_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/json/json_value_converter.h"
 
 #include <string>
 #include <vector>
 
 #include "base/values.h"
 #include "base/json/json_reader.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/scoped_vector.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace base {
 namespace {
 
 // Very simple messages.
 struct SimpleMessage {
   int foo;
   std::string bar;
   bool baz;
   std::vector<int> ints;
   SimpleMessage() : foo(0), baz(false) {}
 
   static void RegisterJSONConverter(
       base::JSONValueConverter<SimpleMessage>* converter) {
     converter->RegisterIntField("foo", &SimpleMessage::foo);
     converter->RegisterStringField("bar", &SimpleMessage::bar);
     converter->RegisterBoolField("baz", &SimpleMessage::baz);
     converter->RegisterRepeatedInt("ints", &SimpleMessage::ints);
   }
 };
 
 // For nested messages.
 struct NestedMessage {
   double foo;
   SimpleMessage child;
-  std::vector<SimpleMessage> children;
+  ScopedVector<SimpleMessage> children;
 
   NestedMessage() : foo(0) {}
 
   static void RegisterJSONConverter(
       base::JSONValueConverter<NestedMessage>* converter) {
     converter->RegisterDoubleField("foo", &NestedMessage::foo);
     converter->RegisterNestedField("child", &NestedMessage::child);
     converter->RegisterRepeatedMessage("children", &NestedMessage::children);
   }
 };
@@ skipping 47 matching lines @@
   NestedMessage message;
   base::JSONValueConverter<NestedMessage> converter;
   EXPECT_TRUE(converter.Convert(*value.get(), &message));
 
   EXPECT_EQ(1.0, message.foo);
   EXPECT_EQ(1, message.child.foo);
   EXPECT_EQ("bar", message.child.bar);
   EXPECT_TRUE(message.child.baz);
 
   EXPECT_EQ(2, static_cast<int>(message.children.size()));
-  const SimpleMessage& first_child = message.children[0];
-  EXPECT_EQ(2, first_child.foo);
-  EXPECT_EQ("foobar", first_child.bar);
-  EXPECT_TRUE(first_child.baz);
+  const SimpleMessage* first_child = message.children[0];
+  ASSERT_TRUE(first_child);
+  EXPECT_EQ(2, first_child->foo);
+  EXPECT_EQ("foobar", first_child->bar);
+  EXPECT_TRUE(first_child->baz);
 
-  const SimpleMessage& second_child = message.children[1];
-  EXPECT_EQ(3, second_child.foo);
-  EXPECT_EQ("barbaz", second_child.bar);
-  EXPECT_FALSE(second_child.baz);
+  const SimpleMessage* second_child = message.children[1];
+  ASSERT_TRUE(second_child);
+  EXPECT_EQ(3, second_child->foo);
+  EXPECT_EQ("barbaz", second_child->bar);
+  EXPECT_FALSE(second_child->baz);
 }
 
 TEST(JSONValueConverterTest, ParseFailures) {
   const char normal_data[] =
       "{\n"
       "  \"foo\": 1,\n"
       "  \"bar\": 2,\n" // "bar" is an integer here.
       "  \"baz\": true,\n"
       "  \"ints\": [1, 2]"
       "}\n";
@@ skipping 21 matching lines @@
   EXPECT_TRUE(converter.Convert(*value.get(), &message));
 
   EXPECT_EQ(1, message.foo);
   EXPECT_TRUE(message.baz);
   EXPECT_EQ(2, static_cast<int>(message.ints.size()));
   EXPECT_EQ(1, message.ints[0]);
   EXPECT_EQ(2, message.ints[1]);
 }
 
 }  // namespace base