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

Unified Diff: media/base/data_buffer_unittest.cc

Issue 19739: Changed media::Buffer interfaces to use base::TimeDelta. (Closed)
Patch Set: Created 11 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/base/data_buffer.cc ('k') | media/base/synchronizer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/data_buffer_unittest.cc
diff --git a/media/base/data_buffer_unittest.cc b/media/base/data_buffer_unittest.cc
index 809d7ffe569e12bd419a9ff1ebf93f3301279d02..9836cedd439abc2f3a096af933e2f4594bc614a2 100644
--- a/media/base/data_buffer_unittest.cc
+++ b/media/base/data_buffer_unittest.cc
@@ -14,6 +14,10 @@ TEST(DataBufferTest, Basic) {
const size_t kDataSize = arraysize(kData);
const char kNewData[] = "chromium";
const size_t kNewDataSize = arraysize(kNewData);
+ const base::TimeDelta kTimestampA = base::TimeDelta::FromMicroseconds(1337);
+ const base::TimeDelta kDurationA = base::TimeDelta::FromMicroseconds(1667);
+ const base::TimeDelta kTimestampB = base::TimeDelta::FromMicroseconds(1234);
+ const base::TimeDelta kDurationB = base::TimeDelta::FromMicroseconds(5678);
// Create our buffer and copy some data into it.
char* data = new char[kBufferSize];
@@ -23,16 +27,17 @@ TEST(DataBufferTest, Basic) {
// Create a DataBuffer.
scoped_refptr<DataBuffer> buffer;
- buffer = new DataBuffer(data, kBufferSize, kDataSize, 1337, 1667);
+ buffer = new DataBuffer(data, kBufferSize, kDataSize,
+ kTimestampA, kDurationA);
ASSERT_TRUE(buffer.get());
// Test StreamSample implementation.
- EXPECT_EQ(1337, buffer->GetTimestamp());
- EXPECT_EQ(1667, buffer->GetDuration());
- buffer->SetTimestamp(1234);
- buffer->SetDuration(5678);
- EXPECT_EQ(1234, buffer->GetTimestamp());
- EXPECT_EQ(5678, buffer->GetDuration());
+ EXPECT_TRUE(kTimestampA == buffer->GetTimestamp());
+ EXPECT_TRUE(kDurationA == buffer->GetDuration());
+ buffer->SetTimestamp(kTimestampB);
+ buffer->SetDuration(kDurationB);
+ EXPECT_TRUE(kTimestampB == buffer->GetTimestamp());
+ EXPECT_TRUE(kDurationB == buffer->GetDuration());
// Test Buffer implementation.
ASSERT_EQ(data, buffer->GetData());
« no previous file with comments | « media/base/data_buffer.cc ('k') | media/base/synchronizer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698