Fix the benchmark app to handle real-world events

apps/benchmark/event_unittest.cc

Index: apps/benchmark/event_unittest.cc
diff --git a/apps/benchmark/event_unittest.cc b/apps/benchmark/event_unittest.cc
index 53441d316d460ac79ea2bcb768f69901a995387a..588c909a8054e560cd97e6a53afcfec219ed1c12 100644
--- a/apps/benchmark/event_unittest.cc
+++ b/apps/benchmark/event_unittest.cc
@@ -178,6 +178,27 @@ TEST(GetEventsTest, AsyncEvents) {
   EXPECT_EQ(base::TimeDelta::FromInternalValue(2), events[1].duration);
 }
 
+TEST(GetEventsTest, AsyncEventsIsString) {

[ppi, 2015/10/01 13:09:44]

AsyncEventsIdIsString

[etiennej, 2015/10/01 13:25:17]

Done.

+  std::vector<std::string> event_specs(4);
+  event_specs[0] =
+      "{\"tid\":1001,\"id\":"\a\",\"ts\":1,\"ph\":\"S\",\"cat\":\"cc\","
+      "\"name\":\"t1 event\"}";
+  event_specs[2] =
+      "{\"tid\":1003,\"id\":\"a\",\"ts\":3,\"ph\":\"F\",\"cat\":\"cc\","
+      "\"name\":\"t1 event\"}";
+
+  std::string trace_json = "[" + JoinString(event_specs, ',') + "]";
+  std::vector<Event> events;
+  ASSERT_TRUE(GetEvents(trace_json, &events));
+  ASSERT_EQ(1u, events.size());
+
+  EXPECT_EQ(EventType::COMPLETE, events[0].type);
+  EXPECT_EQ("t1 event", events[0].name);
+  EXPECT_EQ("cc", events[0].categories);
+  EXPECT_EQ(base::TimeTicks::FromInternalValue(1), events[0].timestamp);
+  EXPECT_EQ(base::TimeDelta::FromInternalValue(2), events[0].duration);
+}
+
 }  // namespace
 
 }  // namespace benchmark