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

Unified Diff: cc/debug/frame_timing_tracker.h

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_request.cc ('k') | cc/debug/frame_timing_tracker.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/debug/frame_timing_tracker.h
diff --git a/cc/debug/frame_timing_tracker.h b/cc/debug/frame_timing_tracker.h
deleted file mode 100644
index f6f9d2b5e4877972948c691bd6aec167c1f0265e..0000000000000000000000000000000000000000
--- a/cc/debug/frame_timing_tracker.h
+++ /dev/null
@@ -1,98 +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.
-
-#ifndef CC_DEBUG_FRAME_TIMING_TRACKER_H_
-#define CC_DEBUG_FRAME_TIMING_TRACKER_H_
-
-#include <stdint.h>
-
-#include <memory>
-#include <unordered_map>
-#include <utility>
-#include <vector>
-
-#include "base/macros.h"
-#include "base/time/time.h"
-#include "cc/base/cc_export.h"
-#include "cc/base/delayed_unique_notifier.h"
-
-namespace cc {
-
-class LayerTreeHostImpl;
-
-// This class maintains a history of timestamps and rect IDs to communicate
-// frame events back to Blink
-// TODO(mpb): Start using this. crbug.com/442554
-class CC_EXPORT FrameTimingTracker {
- public:
- struct CC_EXPORT CompositeTimingEvent {
- CompositeTimingEvent(int, base::TimeTicks);
- ~CompositeTimingEvent();
-
- int frame_id;
- base::TimeTicks timestamp;
- };
-
- using CompositeTimingSet =
- std::unordered_map<int64_t, std::vector<CompositeTimingEvent>>;
-
- struct CC_EXPORT MainFrameTimingEvent {
- MainFrameTimingEvent(int frame_id,
- base::TimeTicks timestamp,
- base::TimeTicks end_time);
- ~MainFrameTimingEvent();
-
- int frame_id;
- base::TimeTicks timestamp;
- base::TimeTicks end_time;
- };
-
- using MainFrameTimingSet =
- std::unordered_map<int64_t, std::vector<MainFrameTimingEvent>>;
-
- static std::unique_ptr<FrameTimingTracker> Create(
- LayerTreeHostImpl* layer_tree_host_impl);
-
- ~FrameTimingTracker();
-
- // This routine takes all of the individual CompositeEvents stored in the
- // tracker and collects them by "rect_id", as in the example below.
- // [ {f_id1,r_id1,t1}, {f_id2,r_id1,t2}, {f_id3,r_id2,t3} ]
- // ====>
- // [ {r_id1,<{f_id1,t1},{f_id2,t2}>}, {r_id2,<{f_id3,t3}>} ]
- std::unique_ptr<CompositeTimingSet> GroupCompositeCountsByRectId();
-
- // This routine takes all of the individual MainFrameEvents stored in the
- // tracker and collects them by "rect_id", as in the example below.
- std::unique_ptr<MainFrameTimingSet> GroupMainFrameCountsByRectId();
-
- // This routine takes a timestamp and an array of frame_id,rect_id pairs
- // and generates CompositeTimingEvents (frame_id, timestamp) and adds them to
- // internal hash_map keyed on rect_id
- using FrameAndRectIds = std::pair<int, int64_t>;
- void SaveTimeStamps(base::TimeTicks timestamp,
- const std::vector<FrameAndRectIds>& frame_ids);
-
- void SaveMainFrameTimeStamps(const std::vector<int64_t>& request_ids,
- base::TimeTicks main_frame_time,
- base::TimeTicks end_time,
- int source_frame_number);
-
- private:
- explicit FrameTimingTracker(LayerTreeHostImpl* layer_tree_host_impl);
-
- void PostEvents();
-
- std::unique_ptr<CompositeTimingSet> composite_events_;
- std::unique_ptr<MainFrameTimingSet> main_frame_events_;
-
- LayerTreeHostImpl* layer_tree_host_impl_;
- DelayedUniqueNotifier post_events_notifier_;
-
- DISALLOW_COPY_AND_ASSIGN(FrameTimingTracker);
-};
-
-} // namespace cc
-
-#endif // CC_DEBUG_FRAME_TIMING_TRACKER_H_
« no previous file with comments | « cc/debug/frame_timing_request.cc ('k') | cc/debug/frame_timing_tracker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698