Commit 872c08a9 authored by Alex Clarke's avatar Alex Clarke Committed by Commit Bot

Add trace events for Emulation.setVirtualTimePolicy

This makes it much easier to diagnose traces where virtual time is enabled.

Bug: 777763
Change-Id: I844b2cc189fea2d324692bd2474fe597a2794380
Reviewed-on: https://chromium-review.googlesource.com/926126Reviewed-by: default avatarDmitry Gozman <dgozman@chromium.org>
Commit-Queue: Alex Clarke <alexclarke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537787}
parent 7cc0f1d0
...@@ -13,6 +13,7 @@ ...@@ -13,6 +13,7 @@
#include "core/page/Page.h" #include "core/page/Page.h"
#include "platform/geometry/DoubleRect.h" #include "platform/geometry/DoubleRect.h"
#include "platform/graphics/Color.h" #include "platform/graphics/Color.h"
#include "platform/instrumentation/tracing/TraceEvent.h"
#include "platform/scheduler/util/thread_cpu_throttler.h" #include "platform/scheduler/util/thread_cpu_throttler.h"
#include "platform/wtf/Time.h" #include "platform/wtf/Time.h"
#include "public/platform/Platform.h" #include "public/platform/Platform.h"
...@@ -228,6 +229,8 @@ WTF::TimeTicks InspectorEmulationAgent::ApplyVirtualTimePolicy( ...@@ -228,6 +229,8 @@ WTF::TimeTicks InspectorEmulationAgent::ApplyVirtualTimePolicy(
WTF::TimeTicks virtual_time_base_ticks( WTF::TimeTicks virtual_time_base_ticks(
web_local_frame_->View()->Scheduler()->EnableVirtualTime()); web_local_frame_->View()->Scheduler()->EnableVirtualTime());
if (new_policy.virtual_time_budget_ms) { if (new_policy.virtual_time_budget_ms) {
TRACE_EVENT_ASYNC_BEGIN1("renderer.scheduler", "VirtualTimeBudget", this,
"budget", *new_policy.virtual_time_budget_ms);
WTF::TimeDelta budget_amount = WTF::TimeDelta budget_amount =
WTF::TimeDelta::FromMillisecondsD(*new_policy.virtual_time_budget_ms); WTF::TimeDelta::FromMillisecondsD(*new_policy.virtual_time_budget_ms);
web_local_frame_->View()->Scheduler()->GrantVirtualTimeBudget( web_local_frame_->View()->Scheduler()->GrantVirtualTimeBudget(
...@@ -258,6 +261,7 @@ Response InspectorEmulationAgent::setNavigatorOverrides( ...@@ -258,6 +261,7 @@ Response InspectorEmulationAgent::setNavigatorOverrides(
} }
void InspectorEmulationAgent::VirtualTimeBudgetExpired() { void InspectorEmulationAgent::VirtualTimeBudgetExpired() {
TRACE_EVENT_ASYNC_END0("renderer.scheduler", "VirtualTimeBudget", this);
web_local_frame_->View()->Scheduler()->SetVirtualTimePolicy( web_local_frame_->View()->Scheduler()->SetVirtualTimePolicy(
WebViewScheduler::VirtualTimePolicy::kPause); WebViewScheduler::VirtualTimePolicy::kPause);
GetFrontend()->virtualTimeBudgetExpired(); GetFrontend()->virtualTimeBudgetExpired();
......
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