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

Side by Side Diff: content/renderer/media/buffered_data_source_host_impl_unittest.cc

Issue 224093011: Move DataSourceHost to BufferedDataSourceHost. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add unittest. Created 6 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 2013 The Chromium Authors. All rights reserved.
scherkus (not reviewing) 2014/04/08 04:36:18 ditto
sandersd (OOO until July 31) 2014/04/08 22:42:14 Done.
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 "content/renderer/media/buffered_data_source_host_impl.h"
6 #include "testing/gtest/include/gtest/gtest.h"
7
8 namespace content {
9
10 class BufferedDataSourceHostImplTest : public testing::Test {
11 public:
12 BufferedDataSourceHostImplTest() {}
13
14 void Add() {
15 host_.AddBufferedTimeRanges(&ranges_, base::TimeDelta::FromSeconds(10));
16 }
17
18 protected:
19 BufferedDataSourceHostImpl host_;
20 media::Ranges<base::TimeDelta> ranges_;
21
22 DISALLOW_COPY_AND_ASSIGN(BufferedDataSourceHostImplTest);
23 };
24
scherkus (not reviewing) 2014/04/08 04:36:18 should include a test post-construction to test th
sandersd (OOO until July 31) 2014/04/08 22:42:14 Done.
25 TEST_F(BufferedDataSourceHostImplTest, AddBufferedTimeRanges) {
26 host_.AddBufferedByteRange(10, 20);
27 host_.SetTotalBytes(100);
28 Add();
29
30 EXPECT_EQ(ranges_.size(), 1u);
scherkus (not reviewing) 2014/04/08 04:36:18 the param order for EXPECT_EQ is (expected, actual
sandersd (OOO until July 31) 2014/04/08 22:42:14 Done.
31 EXPECT_EQ(ranges_.start(0), base::TimeDelta::FromSeconds(1));
32 EXPECT_EQ(ranges_.end(0), base::TimeDelta::FromSeconds(2));
33 }
34
35 TEST_F(BufferedDataSourceHostImplTest, AddBufferedTimeRanges_Merges) {
36 ranges_.Add(base::TimeDelta::FromSeconds(0), base::TimeDelta::FromSeconds(1));
37 host_.AddBufferedByteRange(10, 20);
38 host_.SetTotalBytes(100);
39 Add();
40
41 EXPECT_EQ(ranges_.size(), 1u);
42 EXPECT_EQ(ranges_.start(0), base::TimeDelta::FromSeconds(0));
43 EXPECT_EQ(ranges_.end(0), base::TimeDelta::FromSeconds(2));
44 }
45
46 TEST_F(BufferedDataSourceHostImplTest, AddBufferedTimeRanges_Snaps) {
47 host_.AddBufferedByteRange(5, 995);
48 host_.SetTotalBytes(1000);
49 Add();
50
51 EXPECT_EQ(ranges_.size(), 1u);
52 EXPECT_EQ(ranges_.start(0), base::TimeDelta::FromSeconds(0));
53 EXPECT_EQ(ranges_.end(0), base::TimeDelta::FromSeconds(10));
54 }
55
56 TEST_F(BufferedDataSourceHostImplTest, SetTotalBytes) {
57 host_.AddBufferedByteRange(10, 20);
58 Add();
59
60 EXPECT_EQ(ranges_.size(), 0u);
61
62 host_.SetTotalBytes(100);
63 Add();
64
65 EXPECT_EQ(ranges_.size(), 1u);
66 }
67
68 TEST_F(BufferedDataSourceHostImplTest, DidLoadingProgress) {
69 EXPECT_FALSE(host_.DidLoadingProgress());
70 host_.AddBufferedByteRange(10, 20);
71 EXPECT_TRUE(host_.DidLoadingProgress());
72 EXPECT_FALSE(host_.DidLoadingProgress());
73 }
74
75 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698