Commit d0cf7297 authored by Alexei Filippov's avatar Alexei Filippov Committed by Commit Bot

[runtime-call-stats] Temporarily change DCHECK to CHECK in RCS::Enter and LEAVE

This is to catch remaining instances or main thread's RCS accessed from other
threads. It could have a small negative impact on performance with RCS enabled.
We are going to revert this patch within a week.

BUG=chromium:760649

Change-Id: I437bf7206829c813c0090552c031199840f4baf4
Reviewed-on: https://chromium-review.googlesource.com/728398Reviewed-by: 's avatarCamillo Bruni <cbruni@chromium.org>
Commit-Queue: Alexei Filippov <alph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#48761}
parent f65251be
......@@ -476,7 +476,7 @@ const int RuntimeCallStats::counters_count =
// static
void RuntimeCallStats::Enter(RuntimeCallStats* stats, RuntimeCallTimer* timer,
CounterId counter_id) {
DCHECK(stats->IsCalledOnTheSameThread());
CHECK(stats->IsCalledOnTheSameThread());
RuntimeCallCounter* counter = &(stats->*counter_id);
DCHECK_NOT_NULL(counter->name());
timer->Start(counter, stats->current_timer_.Value());
......@@ -486,7 +486,7 @@ void RuntimeCallStats::Enter(RuntimeCallStats* stats, RuntimeCallTimer* timer,
// static
void RuntimeCallStats::Leave(RuntimeCallStats* stats, RuntimeCallTimer* timer) {
DCHECK(stats->IsCalledOnTheSameThread());
CHECK(stats->IsCalledOnTheSameThread());
if (stats->current_timer_.Value() != timer) {
// The branch is added to catch a crash crbug.com/760649
RuntimeCallTimer* stack_top = stats->current_timer_.Value();
......
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