From d9c532217e8321a60091c917a8163e39d4ed606c Mon Sep 17 00:00:00 2001 From: Sebastien Marchand Date: Fri, 1 Apr 2016 10:11:10 -0400 Subject: Add the TID to the CallStack. R=ivanpe@chromium.org Review URL: https://codereview.chromium.org/1849933002 . --- src/processor/call_stack.cc | 1 + src/processor/minidump_processor.cc | 1 + src/processor/minidump_processor_unittest.cc | 1 + 3 files changed, 3 insertions(+) (limited to 'src/processor') diff --git a/src/processor/call_stack.cc b/src/processor/call_stack.cc index e3276716..925f0846 100644 --- a/src/processor/call_stack.cc +++ b/src/processor/call_stack.cc @@ -48,6 +48,7 @@ void CallStack::Clear() { ++iterator) { delete *iterator; } + tid_ = 0; } } // namespace google_breakpad diff --git a/src/processor/minidump_processor.cc b/src/processor/minidump_processor.cc index b7d7a0d9..448152a8 100644 --- a/src/processor/minidump_processor.cc +++ b/src/processor/minidump_processor.cc @@ -267,6 +267,7 @@ ProcessResult MinidumpProcessor::Process( // one bad thread. BPLOG(ERROR) << "No stackwalker for " << thread_string; } + stack->set_tid(thread_id); process_state->threads_.push_back(stack.release()); process_state->thread_memory_regions_.push_back(thread_memory); } diff --git a/src/processor/minidump_processor_unittest.cc b/src/processor/minidump_processor_unittest.cc index 69e1f42e..609a2a68 100644 --- a/src/processor/minidump_processor_unittest.cc +++ b/src/processor/minidump_processor_unittest.cc @@ -414,6 +414,7 @@ TEST_F(MinidumpProcessorTest, TestBasicProcessing) { ASSERT_EQ(state.crash_reason(), "EXCEPTION_ACCESS_VIOLATION_WRITE"); ASSERT_EQ(state.crash_address(), 0x45U); ASSERT_EQ(state.threads()->size(), size_t(1)); + EXPECT_EQ((*state.threads())[0]->tid(), 3060); ASSERT_EQ(state.requesting_thread(), 0); EXPECT_EQ(1171480435U, state.time_date_stamp()); EXPECT_EQ(1171480435U, state.process_create_time()); -- cgit v1.2.1