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

Side by Side Diff: media/base/data_buffer_unittest.cc

Issue 16297002: Update media/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « media/base/audio_renderer_mixer_unittest.cc ('k') | media/base/decoder_buffer_queue.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 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "base/string_util.h" 5 #include "base/string_util.h"
6 #include "media/base/data_buffer.h" 6 #include "media/base/data_buffer.h"
7 #include "testing/gtest/include/gtest/gtest.h" 7 #include "testing/gtest/include/gtest/gtest.h"
8 8
9 namespace media { 9 namespace media {
10 10
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 } 92 }
93 93
94 TEST(DataBufferTest, ReadingWriting) { 94 TEST(DataBufferTest, ReadingWriting) {
95 const char kData[] = "hello"; 95 const char kData[] = "hello";
96 const int kDataSize = arraysize(kData); 96 const int kDataSize = arraysize(kData);
97 const char kNewData[] = "chromium"; 97 const char kNewData[] = "chromium";
98 const int kNewDataSize = arraysize(kNewData); 98 const int kNewDataSize = arraysize(kNewData);
99 99
100 // Create a DataBuffer. 100 // Create a DataBuffer.
101 scoped_refptr<DataBuffer> buffer(new DataBuffer(kDataSize)); 101 scoped_refptr<DataBuffer> buffer(new DataBuffer(kDataSize));
102 ASSERT_TRUE(buffer); 102 ASSERT_TRUE(buffer.get());
103 103
104 uint8* data = buffer->GetWritableData(); 104 uint8* data = buffer->GetWritableData();
105 ASSERT_TRUE(data); 105 ASSERT_TRUE(data);
106 memcpy(data, kData, kDataSize); 106 memcpy(data, kData, kDataSize);
107 buffer->SetDataSize(kDataSize); 107 buffer->SetDataSize(kDataSize);
108 const uint8* read_only_data = buffer->GetData(); 108 const uint8* read_only_data = buffer->GetData();
109 ASSERT_EQ(data, read_only_data); 109 ASSERT_EQ(data, read_only_data);
110 ASSERT_EQ(0, memcmp(read_only_data, kData, kDataSize)); 110 ASSERT_EQ(0, memcmp(read_only_data, kData, kDataSize));
111 EXPECT_FALSE(buffer->IsEndOfStream()); 111 EXPECT_FALSE(buffer->IsEndOfStream());
112 112
113 scoped_refptr<DataBuffer> buffer2(new DataBuffer(kNewDataSize + 10)); 113 scoped_refptr<DataBuffer> buffer2(new DataBuffer(kNewDataSize + 10));
114 data = buffer2->GetWritableData(); 114 data = buffer2->GetWritableData();
115 ASSERT_TRUE(data); 115 ASSERT_TRUE(data);
116 memcpy(data, kNewData, kNewDataSize); 116 memcpy(data, kNewData, kNewDataSize);
117 buffer2->SetDataSize(kNewDataSize); 117 buffer2->SetDataSize(kNewDataSize);
118 read_only_data = buffer2->GetData(); 118 read_only_data = buffer2->GetData();
119 EXPECT_EQ(kNewDataSize, buffer2->GetDataSize()); 119 EXPECT_EQ(kNewDataSize, buffer2->GetDataSize());
120 ASSERT_EQ(data, read_only_data); 120 ASSERT_EQ(data, read_only_data);
121 EXPECT_EQ(0, memcmp(read_only_data, kNewData, kNewDataSize)); 121 EXPECT_EQ(0, memcmp(read_only_data, kNewData, kNewDataSize));
122 } 122 }
123 123
124 } // namespace media 124 } // namespace media
OLDNEW
« no previous file with comments | « media/base/audio_renderer_mixer_unittest.cc ('k') | media/base/decoder_buffer_queue.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698