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

Side by Side Diff: cc/debug/frame_timing_tracker_unittest.cc

Issue 914403003: cc: Add main frame timing info to frame timing tracker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string> 5 #include <string>
6 6
7 #include "base/time/time.h" 7 #include "base/time/time.h"
8 #include "base/trace_event/trace_event_argument.h" 8 #include "base/trace_event/trace_event_argument.h"
9 #include "base/values.h" 9 #include "base/values.h"
10 #include "cc/debug/frame_timing_tracker.h" 10 #include "cc/debug/frame_timing_tracker.h"
(...skipping 19 matching lines...) Expand all
30 } 30 }
31 value->EndArray(); 31 value->EndArray();
32 value->EndDictionary(); 32 value->EndDictionary();
33 } 33 }
34 value->EndArray(); 34 value->EndArray();
35 return value->ToString(); 35 return value->ToString();
36 } 36 }
37 37
38 TEST(FrameTimingTrackerTest, DefaultTrackerIsEmpty) { 38 TEST(FrameTimingTrackerTest, DefaultTrackerIsEmpty) {
39 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create()); 39 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create());
40 EXPECT_EQ("{\"values\":[]}", ToString(tracker->GroupCountsByRectId())); 40 EXPECT_EQ("{\"values\":[]}",
41 ToString(tracker->GroupCompositeCountsByRectId()));
41 } 42 }
42 43
43 TEST(FrameTimingTrackerTest, NoFrameIdsIsEmpty) { 44 TEST(FrameTimingTrackerTest, NoFrameIdsIsEmpty) {
44 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create()); 45 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create());
45 std::vector<std::pair<int, int64_t>> ids; 46 std::vector<std::pair<int, int64_t>> ids;
46 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(100), ids); 47 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(100), ids);
47 EXPECT_EQ("{\"values\":[]}", ToString(tracker->GroupCountsByRectId())); 48 EXPECT_EQ("{\"values\":[]}",
49 ToString(tracker->GroupCompositeCountsByRectId()));
48 } 50 }
49 51
50 TEST(FrameTimingTrackerTest, OneFrameId) { 52 TEST(FrameTimingTrackerTest, OneFrameId) {
51 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create()); 53 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create());
52 std::vector<std::pair<int, int64_t>> ids; 54 std::vector<std::pair<int, int64_t>> ids;
53 ids.push_back(std::make_pair(1, 2)); 55 ids.push_back(std::make_pair(1, 2));
54 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(100), ids); 56 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(100), ids);
55 EXPECT_EQ( 57 EXPECT_EQ(
56 "{\"values\":[{\"events\":[" 58 "{\"values\":[{\"events\":["
57 "{\"frame_id\":1,\"timestamp\":100}],\"rect_id\":2}]}", 59 "{\"frame_id\":1,\"timestamp\":100}],\"rect_id\":2}]}",
58 ToString(tracker->GroupCountsByRectId())); 60 ToString(tracker->GroupCompositeCountsByRectId()));
59 } 61 }
60 62
61 TEST(FrameTimingTrackerTest, UnsortedTimestampsIds) { 63 TEST(FrameTimingTrackerTest, UnsortedTimestampsIds) {
62 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create()); 64 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create());
63 std::vector<std::pair<int, int64_t>> ids; 65 std::vector<std::pair<int, int64_t>> ids;
64 ids.push_back(std::make_pair(1, 2)); 66 ids.push_back(std::make_pair(1, 2));
65 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(200), ids); 67 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(200), ids);
66 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(400), ids); 68 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(400), ids);
67 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(100), ids); 69 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(100), ids);
68 EXPECT_EQ( 70 EXPECT_EQ(
69 "{\"values\":[{\"events\":[" 71 "{\"values\":[{\"events\":["
70 "{\"frame_id\":1,\"timestamp\":100}," 72 "{\"frame_id\":1,\"timestamp\":100},"
71 "{\"frame_id\":1,\"timestamp\":200}," 73 "{\"frame_id\":1,\"timestamp\":200},"
72 "{\"frame_id\":1,\"timestamp\":400}],\"rect_id\":2}]}", 74 "{\"frame_id\":1,\"timestamp\":400}],\"rect_id\":2}]}",
73 ToString(tracker->GroupCountsByRectId())); 75 ToString(tracker->GroupCompositeCountsByRectId()));
74 } 76 }
75 77
76 TEST(FrameTimingTrackerTest, MultipleFrameIds) { 78 TEST(FrameTimingTrackerTest, MultipleFrameIds) {
77 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create()); 79 scoped_ptr<FrameTimingTracker> tracker(FrameTimingTracker::Create());
78 80
79 std::vector<std::pair<int, int64_t>> ids200; 81 std::vector<std::pair<int, int64_t>> ids200;
80 ids200.push_back(std::make_pair(1, 2)); 82 ids200.push_back(std::make_pair(1, 2));
81 ids200.push_back(std::make_pair(1, 3)); 83 ids200.push_back(std::make_pair(1, 3));
82 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(200), ids200); 84 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(200), ids200);
83 85
84 std::vector<std::pair<int, int64_t>> ids400; 86 std::vector<std::pair<int, int64_t>> ids400;
85 ids400.push_back(std::make_pair(2, 2)); 87 ids400.push_back(std::make_pair(2, 2));
86 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(400), ids400); 88 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(400), ids400);
87 89
88 std::vector<std::pair<int, int64_t>> ids100; 90 std::vector<std::pair<int, int64_t>> ids100;
89 ids100.push_back(std::make_pair(3, 2)); 91 ids100.push_back(std::make_pair(3, 2));
90 ids100.push_back(std::make_pair(2, 3)); 92 ids100.push_back(std::make_pair(2, 3));
91 ids100.push_back(std::make_pair(3, 4)); 93 ids100.push_back(std::make_pair(3, 4));
92 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(100), ids100); 94 tracker->SaveTimeStamps(base::TimeTicks::FromInternalValue(100), ids100);
93 95
94 std::string result = ToString(tracker->GroupCountsByRectId()); 96 std::string result = ToString(tracker->GroupCompositeCountsByRectId());
95 97
96 EXPECT_EQ(strlen( 98 EXPECT_EQ(strlen(
97 "{\"values\":[{\"events\":[" 99 "{\"values\":[{\"events\":["
98 "{\"frame_id\":3,\"timestamp\":100}," 100 "{\"frame_id\":3,\"timestamp\":100},"
99 "{\"frame_id\":1,\"timestamp\":200}," 101 "{\"frame_id\":1,\"timestamp\":200},"
100 "{\"frame_id\":2,\"timestamp\":400}],\"rect_id\":2}," 102 "{\"frame_id\":2,\"timestamp\":400}],\"rect_id\":2},"
101 "{\"events\":[" 103 "{\"events\":["
102 "{\"frame_id\":2,\"timestamp\":100}," 104 "{\"frame_id\":2,\"timestamp\":100},"
103 "{\"frame_id\":1,\"timestamp\":200}],\"rect_id\":3}," 105 "{\"frame_id\":1,\"timestamp\":200}],\"rect_id\":3},"
104 "{\"events\":[" 106 "{\"events\":["
(...skipping 11 matching lines...) Expand all
116 "{\"frame_id\":2,\"timestamp\":100}," 118 "{\"frame_id\":2,\"timestamp\":100},"
117 "{\"frame_id\":1,\"timestamp\":200}],\"rect_id\":3}")); 119 "{\"frame_id\":1,\"timestamp\":200}],\"rect_id\":3}"));
118 EXPECT_NE(std::string::npos, 120 EXPECT_NE(std::string::npos,
119 result.find( 121 result.find(
120 "{\"events\":[" 122 "{\"events\":["
121 "{\"frame_id\":3,\"timestamp\":100}],\"rect_id\":4}")); 123 "{\"frame_id\":3,\"timestamp\":100}],\"rect_id\":4}"));
122 } 124 }
123 125
124 } // namespace 126 } // namespace
125 } // namespace cc 127 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698