Commit 2b8f0454 authored by Nicolas Pena's avatar Nicolas Pena Committed by Commit Bot

Remove RecordQueueingTimeClient

Bug: 832680, 830882
Change-Id: I7fc5145f1b1ce9fee9bff6470cc55cda578e2c64
Reviewed-on: https://chromium-review.googlesource.com/1127259Reviewed-by: default avatarAlexander Timin <altimin@chromium.org>
Reviewed-by: default avatarTimothy Dresser <tdresser@chromium.org>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574161}
parent c8792104
......@@ -62,10 +62,9 @@ base::TimeDelta ExpectedQueueingTimeFromTask(base::TimeTicks task_start,
} // namespace
QueueingTimeEstimator::QueueingTimeEstimator(
QueueingTimeEstimator::Client* client,
base::TimeDelta window_duration,
int steps_per_window)
QueueingTimeEstimator::QueueingTimeEstimator(Client* client,
base::TimeDelta window_duration,
int steps_per_window)
: client_(client),
window_step_width_(window_duration / steps_per_window),
renderer_backgrounded_(kLaunchingProcessIsBackgrounded),
......@@ -240,11 +239,10 @@ void QueueingTimeEstimator::Calculator::AddQueueingTime(
queueing_time;
}
void QueueingTimeEstimator::Calculator::EndStep(
QueueingTimeEstimator::Client* client) {
void QueueingTimeEstimator::Calculator::EndStep(Client* client) {
step_queueing_times_.Add(step_expected_queueing_time_);
// RendererScheduler reports the queueing time once per disjoint window.
// MainThreadSchedulerImpl reports the queueing time once per disjoint window.
// |stepEQT|stepEQT|stepEQT|stepEQT|stepEQT|stepEQT|
// Report: |-------window EQT------|
// Discard: |-------window EQT------|
......@@ -305,28 +303,5 @@ bool QueueingTimeEstimator::RunningAverage::IndexIsZero() const {
return index_ == 0;
}
// Keeps track of the queueing time.
class RecordQueueingTimeClient : public QueueingTimeEstimator::Client {
public:
// QueueingTimeEstimator::Client implementation:
void OnQueueingTimeForWindowEstimated(base::TimeDelta queueing_time,
bool is_disjoint_window) override {
queueing_time_ = queueing_time;
}
void OnReportFineGrainedExpectedQueueingTime(
const char* split_description,
base::TimeDelta queueing_time) override {}
base::TimeDelta queueing_time() { return queueing_time_; }
RecordQueueingTimeClient() = default;
~RecordQueueingTimeClient() override = default;
private:
base::TimeDelta queueing_time_;
DISALLOW_COPY_AND_ASSIGN(RecordQueueingTimeClient);
};
} // namespace scheduler
} // namespace blink
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