Tiny fix: initialize TickSample::frames_count with 0.

I suggest that the lack of initialization causes issue 358 to happen. In Profiler::Disengage an empty TickSample is inserted in order to wake up the Profiler thread. Issue reporter claims that crash happens in LogTickEvent function. My guess is that frames_couint receives a wild value.

Review URL: http://codereview.chromium.org/113939


git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@2068 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent e48095b8
...@@ -493,7 +493,7 @@ class Socket { ...@@ -493,7 +493,7 @@ class Socket {
// TickSample captures the information collected for each sample. // TickSample captures the information collected for each sample.
class TickSample { class TickSample {
public: public:
TickSample() : pc(0), sp(0), fp(0), state(OTHER) {} TickSample() : pc(0), sp(0), fp(0), state(OTHER), frames_count(0) {}
uintptr_t pc; // Instruction pointer. uintptr_t pc; // Instruction pointer.
uintptr_t sp; // Stack pointer. uintptr_t sp; // Stack pointer.
uintptr_t fp; // Frame pointer. uintptr_t fp; // Frame pointer.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment