Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(310)

Unified Diff: ppapi/tests/test_dev_trace_event.cc

Issue 17555005: Add events with custom timestamps and thread id to PPAPI dev tracing interface. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: clock function Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..03845e0eee5ffb97aaa1bfe69fd2c909d51e9dda
--- /dev/null
+++ b/ppapi/tests/test_dev_trace_event.cc
@@ -0,0 +1,69 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
dmichael (off chromium) 2013/06/24 17:43:42 ditto, 2013
grosse 2013/06/24 23:16:46 Done.
+// 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);
+ RUN_TEST(Clock, 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("bar");
+ 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("bar");
+ interface_->AddTraceEventWithThreadIdAndTimestamp('B', cat_enabled, "foo", 0,
bradn 2013/06/22 16:09:25 Nit, maybe wrap these from the paren down, so they
grosse 2013/06/24 23:16:46 Done.
+ 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();
+}
+
+std::string TestDevTraceEvent::TestClock() {
+ int64_t last = interface_->Now();
+
+ for(int i=0; i<5; ++i){
+ int64_t next = interface_->Now();
+ ASSERT_LE(last, next);
+ last = next;
+ }
+
+ PASS();
+}

Powered by Google App Engine
This is Rietveld 408576698