Add events with custom timestamps and thread id to PPAPI dev tracing interface.

ppapi/tests/test_dev_trace_event.cc

Index: ppapi/tests/test_dev_trace_event.cc
diff --git a/ppapi/tests/test_dev_trace_event.cc b/ppapi/tests/test_dev_trace_event.cc
new file mode 100644
index 0000000000000000000000000000000000000000..02968b4c6ff5ec3a19da91d23082ac42c0b3f4d4
--- /dev/null
+++ b/ppapi/tests/test_dev_trace_event.cc
@@ -0,0 +1,56 @@
+// 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 "ppapi/tests/test_dev_trace_event.h"
+
+#include "ppapi/cpp/module.h"
+#include "ppapi/tests/testing_instance.h"
+
+REGISTER_TEST_CASE(DevTraceEvent);
+
+TestDevTraceEvent::TestDevTraceEvent(TestingInstance* instance)
+    : TestCase(instance),
+      interface_(NULL) {
+}
+
+bool TestDevTraceEvent::Init() {
+  interface_ = static_cast<const PPB_Trace_Event_Dev*>(
+      pp::Module::Get()->GetBrowserInterface(PPB_TRACE_EVENT_DEV_INTERFACE));
+  return !!interface_;
+}
+
+void TestDevTraceEvent::RunTests(const std::string& filter) {
+  RUN_TEST(Smoke, filter);
+  RUN_TEST(SmokeWithTimestamps, filter);
+}
+
+std::string TestDevTraceEvent::TestSmoke() {
+  // This test does not verify the log message actually reaches dev tracing, but
+  // it does test that the interface exists and that it can be called without
+  // crashing.
+  const void* cat_enabled = interface_->GetCategoryEnabled("");

[bradn, 2013/06/21 21:49:18]

Maybe pass a non-empty?

[grosse, 2013/06/24 23:16:46]

Done.

+  interface_->AddTraceEvent('B', cat_enabled, "foo", 0, 0, NULL, NULL, NULL, 0);
+  interface_->AddTraceEvent('E', cat_enabled, "foo", 0, 0, NULL, NULL, NULL, 0);
+  PASS();
+}
+
+std::string TestDevTraceEvent::TestSmokeWithTimestamps() {
+  // This test does not verify the log message actually reaches dev tracing, but
+  // it does test that the interface exists and that it can be called without
+  // crashing.
+  const void* cat_enabled = interface_->GetCategoryEnabled("");
+  interface_->AddTraceEventWithThreadIdAndTimestamp('B', cat_enabled, "foo", 0,

[bradn, 2013/06/21 21:49:18]

Maybe use different name pair (foo, bar)?

[grosse, 2013/06/24 23:16:46]

Ignored

+                                                    0, 42, 0, NULL, NULL, NULL,
+                                                    0);
+  interface_->AddTraceEventWithThreadIdAndTimestamp('B', cat_enabled, "foo", 0,
+                                                    1, 43, 0, NULL, NULL, NULL,
+                                                    0);
+  interface_->AddTraceEventWithThreadIdAndTimestamp('E', cat_enabled, "foo", 0,
+                                                    0, 44, 0, NULL, NULL, NULL,
+                                                    0);
+  interface_->AddTraceEventWithThreadIdAndTimestamp('E', cat_enabled, "foo", 0,
+                                                    1, 45, 0, NULL, NULL, NULL,
+                                                    0);
+  PASS();
+}