Revision: 24470
Author: [email protected]
Date: Wed Oct 8 14:08:12 2014 UTC
Log: Resolve race between starting the profiler thread and the logging
timer
BUG=none
[email protected]
LOG=n
Review URL: https://codereview.chromium.org/638943002
https://code.google.com/p/v8/source/detail?r=24470
Modified:
/branches/bleeding_edge/src/log.cc
=======================================
--- /branches/bleeding_edge/src/log.cc Wed Oct 8 11:33:31 2014 UTC
+++ /branches/bleeding_edge/src/log.cc Wed Oct 8 14:08:12 2014 UTC
@@ -1865,14 +1865,14 @@
if (Log::InitLogAtStart()) {
is_logging_ = true;
}
+
+ if (FLAG_log_internal_timer_events || FLAG_prof) timer_.Start();
if (FLAG_prof) {
profiler_ = new Profiler(isolate);
is_logging_ = true;
profiler_->Engage();
}
-
- if (FLAG_log_internal_timer_events || FLAG_prof) timer_.Start();
return true;
}
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.