Commit ce9ab3a9 authored by Matt Falkenhagen's avatar Matt Falkenhagen Committed by Commit Bot

service worker: Skip recording timing when wait for debugger is on.

We were skipping recording for the legacy EmbeddedWorkerInstance
metrics but not the ServiceWorker.StartTiming.* metrics.

It doesn't look like there's been a big effect regardless.

Bug: 855952
Change-Id: I4f2804e45e75b6fa4f7bded732e7e472e2033bdf
Reviewed-on: https://chromium-review.googlesource.com/1122060Reviewed-by: default avatarMakoto Shimazu <shimazu@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571862}
parent 66d2c586
...@@ -683,6 +683,7 @@ void EmbeddedWorkerInstance::OnRegisteredToDevToolsManager( ...@@ -683,6 +683,7 @@ void EmbeddedWorkerInstance::OnRegisteredToDevToolsManager(
// We don't measure the start time when wait_for_debugger flag is set. So // We don't measure the start time when wait_for_debugger flag is set. So
// we set the NULL time here. // we set the NULL time here.
step_time_ = base::TimeTicks(); step_time_ = base::TimeTicks();
inflight_start_task_->set_skip_recording_startup_time();
} }
for (auto& observer : listener_list_) for (auto& observer : listener_list_)
observer.OnRegisteredToDevToolsManager(); observer.OnRegisteredToDevToolsManager();
......
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