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

Side by Side Diff: components/tracing/core/proto_utils_unittest.cc

Issue 2043913006: tracing v2: Add proto_utils.h (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@tv_ups2
Patch Set: address review comments Created 4 years, 5 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 2016 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 #include "components/tracing/core/proto_utils.h"
6
7 #include "base/logging.h"
8 #include "testing/gtest/include/gtest/gtest.h"
9
10 namespace tracing {
11 namespace v2 {
12 namespace proto {
13 namespace {
14
15 bool VarIntHelperU32(const char* expected, size_t length, uint32_t value) {
16 uint8_t buf[32];
17 uint8_t* res = WriteVarIntU32(value, buf);
18 if (static_cast<size_t>(res - buf) != length)
19 return false;
20 if (memcmp(expected, buf, length))
21 return false;
22 return true;
23 }
24
25 bool VarIntHelperU64(const char* expected, size_t length, uint64_t value) {
26 uint8_t buf[32];
27 uint8_t* res = WriteVarIntU64(value, buf);
28 if (static_cast<size_t>(res - buf) != length)
29 return false;
30 if (memcmp(expected, buf, length))
31 return false;
32 return true;
33 }
34
35 #define EXPECT_VARINT32_EQ(expected, expected_length, value) \
36 EXPECT_PRED3(VarIntHelperU32, expected, expected_length, value)
37
38 #define EXPECT_VARINT64_EQ(expected, expected_length, value) \
39 EXPECT_PRED3(VarIntHelperU64, expected, expected_length, value)
40
41 TEST(ProtoUtilsTest, Serialization) {
42 EXPECT_EQ(0x08u, MakeTagVarInt(1));
43 EXPECT_EQ(0x09u, MakeTagFixed64(1));
44 EXPECT_EQ(0x0Au, MakeTagLengthLimited(1));
45 EXPECT_EQ(0x0Du, MakeTagFixed32(1));
46
47 EXPECT_EQ(0x03F8u, MakeTagVarInt(0x7F));
48 EXPECT_EQ(0x03F9u, MakeTagFixed64(0x7F));
49 EXPECT_EQ(0x03FAu, MakeTagLengthLimited(0x7F));
50 EXPECT_EQ(0x03FDu, MakeTagFixed32(0x7F));
51
52 EXPECT_EQ(0x0400u, MakeTagVarInt(0x80));
53 EXPECT_EQ(0x0401u, MakeTagFixed64(0x80));
54 EXPECT_EQ(0x0402u, MakeTagLengthLimited(0x80));
55 EXPECT_EQ(0x0405u, MakeTagFixed32(0x80));
56
57 EXPECT_EQ(0x01FFF8u, MakeTagVarInt(0x3fff));
58 EXPECT_EQ(0x01FFF9u, MakeTagFixed64(0x3fff));
59 EXPECT_EQ(0x01FFFAu, MakeTagLengthLimited(0x3fff));
60 EXPECT_EQ(0x01FFFDu, MakeTagFixed32(0x3fff));
61
62 EXPECT_EQ(0x020000u, MakeTagVarInt(0x4000));
63 EXPECT_EQ(0x020001u, MakeTagFixed64(0x4000));
64 EXPECT_EQ(0x020002u, MakeTagLengthLimited(0x4000));
65 EXPECT_EQ(0x020005u, MakeTagFixed32(0x4000));
66
67 EXPECT_VARINT32_EQ("\x00", 1, 0);
alph 2016/07/06 17:29:52 nit: you can drop the length arg and infer it from
Primiano Tucci (use gerrit) 2016/07/07 12:59:25 you mean using strlen? The problem is this case he
68 EXPECT_VARINT32_EQ("\x00", 1, 0);
69 EXPECT_VARINT32_EQ("\x01", 1, 0x1);
70 EXPECT_VARINT32_EQ("\x7f", 1, 0x7F);
71 EXPECT_VARINT32_EQ("\xFF\x01", 2, 0xFF);
72 EXPECT_VARINT32_EQ("\xFF\x7F", 2, 0x3FFF);
73 EXPECT_VARINT32_EQ("\x80\x80\x01", 3, 0x4000);
74 EXPECT_VARINT32_EQ("\xFF\xFF\x7F", 3, 0x1FFFFF);
75 EXPECT_VARINT32_EQ("\x80\x80\x80\x01", 4, 0x200000);
76 EXPECT_VARINT32_EQ("\xFF\xFF\xFF\x7F", 4, 0xFFFFFFF);
77 EXPECT_VARINT32_EQ("\x80\x80\x80\x80\x01", 5, 0x10000000);
78 EXPECT_VARINT32_EQ("\xFF\xFF\xFF\xFF\x0F", 5, 0xFFFFFFFF);
79
80 EXPECT_VARINT64_EQ("\x00", 1, 0);
81 EXPECT_VARINT64_EQ("\x01", 1, 0x1);
82 EXPECT_VARINT64_EQ("\x7f", 1, 0x7F);
83 EXPECT_VARINT64_EQ("\xFF\x01", 2, 0xFF);
84 EXPECT_VARINT64_EQ("\xFF\x7F", 2, 0x3FFF);
85 EXPECT_VARINT64_EQ("\x80\x80\x01", 3, 0x4000);
86 EXPECT_VARINT64_EQ("\xFF\xFF\x7F", 3, 0x1FFFFF);
87 EXPECT_VARINT64_EQ("\x80\x80\x80\x01", 4, 0x200000);
88 EXPECT_VARINT64_EQ("\xFF\xFF\xFF\x7F", 4, 0xFFFFFFF);
89 EXPECT_VARINT64_EQ("\x80\x80\x80\x80\x01", 5, 0x10000000);
90 EXPECT_VARINT64_EQ("\xFF\xFF\xFF\xFF\x0F", 5, 0xFFFFFFFF);
91 EXPECT_VARINT64_EQ("\x80\x80\x80\x80\x10", 5, 0x100000000);
92 EXPECT_VARINT64_EQ("\xFF\xFF\xFF\xFF\x7F", 5, 0x7FFFFFFFF);
93 EXPECT_VARINT64_EQ("\x80\x80\x80\x80\x80\x01", 6, 0x800000000);
94 EXPECT_VARINT64_EQ("\xFF\xFF\xFF\xFF\xFF\x7F", 6, 0x3FFFFFFFFFF);
95 EXPECT_VARINT64_EQ("\x80\x80\x80\x80\x80\x80\x01", 7, 0x40000000000);
96 EXPECT_VARINT64_EQ("\xFF\xFF\xFF\xFF\xFF\xFF\x7F", 7, 0x1FFFFFFFFFFFF);
97 EXPECT_VARINT64_EQ("\x80\x80\x80\x80\x80\x80\x80\x01", 8, 0x2000000000000);
98 EXPECT_VARINT64_EQ("\xFF\xFF\xFF\xFF\xFF\xFF\xFF\x7F", 8, 0xFFFFFFFFFFFFFF);
99 EXPECT_VARINT64_EQ("\x80\x80\x80\x80\x80\x80\x80\x80\x01", 9,
100 0x100000000000000);
101 EXPECT_VARINT64_EQ("\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\x7F", 9,
102 0x7FFFFFFFFFFFFFFF);
103 EXPECT_VARINT64_EQ("\x80\x80\x80\x80\x80\x80\x80\x80\x80\x01", 10,
104 0x8000000000000000);
105 EXPECT_VARINT64_EQ("\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\xFF\x01", 10,
106 0xFFFFFFFFFFFFFFFF);
107
108 uint8_t buf[4];
109
110 WriteRedundantVarIntUnsigned(0, sizeof(buf), buf);
111 EXPECT_EQ(0, memcmp("\x80\x80\x80\x00", buf, sizeof(buf)));
112
113 WriteRedundantVarIntUnsigned(1, sizeof(buf), buf);
114 EXPECT_EQ(0, memcmp("\x81\x80\x80\x00", buf, sizeof(buf)));
115
116 WriteRedundantVarIntUnsigned(0x80, sizeof(buf), buf);
117 EXPECT_EQ(0, memcmp("\x80\x81\x80\x00", buf, sizeof(buf)));
118
119 WriteRedundantVarIntUnsigned(0x332211, sizeof(buf), buf);
120 EXPECT_EQ(0, memcmp("\x91\xC4\xCC\x01", buf, sizeof(buf)));
121 }
122
123 } // namespace
124 } // namespace proto
125 } // namespace v2
126 } // namespace tracing
OLDNEW
« components/tracing/core/proto_utils.h ('K') | « components/tracing/core/proto_utils.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698