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

Side by Side Diff: webrtc/common_video/bitrate_adjuster_unittest.cc

Issue 2358993004: Enable the -Wundef warning for clang (Closed)
Patch Set: rebase Created 4 years, 2 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
« no previous file with comments | « webrtc/common_types.h ('k') | webrtc/common_video/h264/pps_parser_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright 2016 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 #include "testing/gtest/include/gtest/gtest.h" 11 #include "webrtc/test/gtest.h"
12 12
13 #include "webrtc/common_video/include/bitrate_adjuster.h" 13 #include "webrtc/common_video/include/bitrate_adjuster.h"
14 #include "webrtc/system_wrappers/include/clock.h" 14 #include "webrtc/system_wrappers/include/clock.h"
15 15
16 namespace webrtc { 16 namespace webrtc {
17 17
18 class BitrateAdjusterTest : public ::testing::Test { 18 class BitrateAdjusterTest : public ::testing::Test {
19 public: 19 public:
20 BitrateAdjusterTest() 20 BitrateAdjusterTest()
21 : clock_(0), 21 : clock_(0),
(...skipping 138 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 EXPECT_EQ(initial_target_bitrate_bps, adjuster_.GetAdjustedBitrateBps()); 160 EXPECT_EQ(initial_target_bitrate_bps, adjuster_.GetAdjustedBitrateBps());
161 161
162 // 1.05 * 1.05 is 1.1 which is greater than tolerance for the initial target 162 // 1.05 * 1.05 is 1.1 which is greater than tolerance for the initial target
163 // bitrate. Since we didn't advance the clock the adjuster never updated. 163 // bitrate. Since we didn't advance the clock the adjuster never updated.
164 target_bitrate_bps = (1 + delta_pct) * target_bitrate_bps; 164 target_bitrate_bps = (1 + delta_pct) * target_bitrate_bps;
165 adjuster_.SetTargetBitrateBps(target_bitrate_bps); 165 adjuster_.SetTargetBitrateBps(target_bitrate_bps);
166 EXPECT_EQ(target_bitrate_bps, adjuster_.GetAdjustedBitrateBps()); 166 EXPECT_EQ(target_bitrate_bps, adjuster_.GetAdjustedBitrateBps());
167 } 167 }
168 168
169 } // namespace webrtc 169 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/common_types.h ('k') | webrtc/common_video/h264/pps_parser_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698