Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2745)

Unified Diff: cc/debug/frame_timing_tracker.cc

Issue 1897123002: Remove current implementation of frame timing events. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/debug/frame_timing_tracker.h ('k') | cc/debug/frame_timing_tracker_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/debug/frame_timing_tracker.cc
diff --git a/cc/debug/frame_timing_tracker.cc b/cc/debug/frame_timing_tracker.cc
deleted file mode 100644
index d0b9a9b0e8b87338f58720ad159cf95fdc838f54..0000000000000000000000000000000000000000
--- a/cc/debug/frame_timing_tracker.cc
+++ /dev/null
@@ -1,119 +0,0 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "cc/debug/frame_timing_tracker.h"
-
-#include <stdint.h>
-
-#include <algorithm>
-#include <limits>
-
-#include "base/memory/ptr_util.h"
-#include "base/metrics/histogram.h"
-#include "cc/trees/layer_tree_host_impl.h"
-#include "cc/trees/proxy.h"
-
-namespace cc {
-namespace {
-int kSendTimingIntervalMS = 200;
-}
-
-FrameTimingTracker::CompositeTimingEvent::CompositeTimingEvent(
- int _frame_id,
- base::TimeTicks _timestamp)
- : frame_id(_frame_id), timestamp(_timestamp) {
-}
-
-FrameTimingTracker::CompositeTimingEvent::~CompositeTimingEvent() {
-}
-
-FrameTimingTracker::MainFrameTimingEvent::MainFrameTimingEvent(
- int frame_id,
- base::TimeTicks timestamp,
- base::TimeTicks end_time)
- : frame_id(frame_id), timestamp(timestamp), end_time(end_time) {
-}
-
-FrameTimingTracker::MainFrameTimingEvent::~MainFrameTimingEvent() {
-}
-
-// static
-std::unique_ptr<FrameTimingTracker> FrameTimingTracker::Create(
- LayerTreeHostImpl* layer_tree_host_impl) {
- return base::WrapUnique(new FrameTimingTracker(layer_tree_host_impl));
-}
-
-FrameTimingTracker::FrameTimingTracker(LayerTreeHostImpl* layer_tree_host_impl)
- : layer_tree_host_impl_(layer_tree_host_impl),
- post_events_notifier_(
- layer_tree_host_impl_->GetTaskRunner(),
- base::Bind(&FrameTimingTracker::PostEvents, base::Unretained(this)),
- base::TimeDelta::FromMilliseconds(kSendTimingIntervalMS)) {}
-
-FrameTimingTracker::~FrameTimingTracker() {
-}
-
-void FrameTimingTracker::SaveTimeStamps(
- base::TimeTicks timestamp,
- const std::vector<FrameAndRectIds>& frame_ids) {
- if (!composite_events_)
- composite_events_.reset(new CompositeTimingSet);
- for (const auto& pair : frame_ids) {
- (*composite_events_)[pair.second].push_back(
- CompositeTimingEvent(pair.first, timestamp));
- }
- if (!post_events_notifier_.HasPendingNotification())
- post_events_notifier_.Schedule();
-}
-
-void FrameTimingTracker::SaveMainFrameTimeStamps(
- const std::vector<int64_t>& request_ids,
- base::TimeTicks main_frame_time,
- base::TimeTicks end_time,
- int source_frame_number) {
- if (!main_frame_events_)
- main_frame_events_.reset(new MainFrameTimingSet);
- for (const auto& request : request_ids) {
- std::vector<MainFrameTimingEvent>& events = (*main_frame_events_)[request];
- events.push_back(
- MainFrameTimingEvent(source_frame_number, main_frame_time, end_time));
- }
- if (!post_events_notifier_.HasPendingNotification())
- post_events_notifier_.Schedule();
-}
-
-std::unique_ptr<FrameTimingTracker::CompositeTimingSet>
-FrameTimingTracker::GroupCompositeCountsByRectId() {
- if (!composite_events_)
- return base::WrapUnique(new CompositeTimingSet);
- for (auto& infos : *composite_events_) {
- std::sort(
- infos.second.begin(), infos.second.end(),
- [](const CompositeTimingEvent& lhs, const CompositeTimingEvent& rhs) {
- return lhs.timestamp < rhs.timestamp;
- });
- }
- return std::move(composite_events_);
-}
-
-std::unique_ptr<FrameTimingTracker::MainFrameTimingSet>
-FrameTimingTracker::GroupMainFrameCountsByRectId() {
- if (!main_frame_events_)
- return base::WrapUnique(new MainFrameTimingSet);
- for (auto& infos : *main_frame_events_) {
- std::sort(
- infos.second.begin(), infos.second.end(),
- [](const MainFrameTimingEvent& lhs, const MainFrameTimingEvent& rhs) {
- return lhs.timestamp < rhs.timestamp;
- });
- }
- return std::move(main_frame_events_);
-}
-
-void FrameTimingTracker::PostEvents() {
- layer_tree_host_impl_->PostFrameTimingEvents(GroupCompositeCountsByRectId(),
- GroupMainFrameCountsByRectId());
-}
-
-} // namespace cc
« no previous file with comments | « cc/debug/frame_timing_tracker.h ('k') | cc/debug/frame_timing_tracker_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698