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

Side by Side Diff: ui/latency/latency_tracker.h

Issue 2814483002: Splitting up RenderWidgetHostLatencyTracker and some renames. (Closed)
Patch Set: Rebase Created 3 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef UI_LATENCY_LATENCY_TRACKER_H_
6 #define UI_LATENCY_LATENCY_TRACKER_H_
7
8 #include <stdint.h>
tdresser 2017/04/10 16:15:54 Is this used?
mfomitchev 2017/04/10 19:00:07 Removed
9
10 #include <vector>
tdresser 2017/04/10 16:15:54 Is this used?
mfomitchev 2017/04/10 19:00:07 Removed
11
12 #include "base/macros.h"
13 #include "ui/latency/latency_info.h"
14
15 namespace ui {
16
17 // Utility class for tracking the latency of events.
18 class LatencyTracker {
tdresser 2017/04/10 16:15:54 Should we split up the RenderWidgetHostLatencyTrac
mfomitchev 2017/04/10 19:00:07 All the unit tests I see that test OnGpuSwapBuffer
19 public:
20 LatencyTracker() = default;
21 ~LatencyTracker() = default;
22
23 // Terminates latency tracking for events that triggered rendering, also
24 // performing relevant UMA latency reporting.
25 // Called when GPU buffers swap completes.
26 void OnGpuSwapBuffersCompleted(const LatencyInfo& latency);
27
28 protected:
29 virtual void ReportRapporScrollLatency(
30 const std::string& name,
31 const LatencyInfo::LatencyComponent& start_component,
32 const LatencyInfo::LatencyComponent& end_component);
33
34 private:
35 void ComputeTouchAndWheelScrollLatencyHistograms(
36 const LatencyInfo::LatencyComponent& gpu_swap_begin_component,
37 const LatencyInfo::LatencyComponent& gpu_swap_end_component,
38 const LatencyInfo& latency);
39
40 DISALLOW_COPY_AND_ASSIGN(LatencyTracker);
41 };
42
43 } // namespace latency
44
45 #endif // UI_LATENCY_LATENCY_TRACKER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698