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

Unified Diff: ppapi/shared_impl/media_stream_buffer_manager_unittest.cc

Issue 1864293002: Convert //ppapi to use std::unique_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: more nullptr Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ppapi/shared_impl/media_stream_buffer_manager.cc ('k') | ppapi/shared_impl/ppb_audio_shared.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/shared_impl/media_stream_buffer_manager_unittest.cc
diff --git a/ppapi/shared_impl/media_stream_buffer_manager_unittest.cc b/ppapi/shared_impl/media_stream_buffer_manager_unittest.cc
index 4bb250a2c66e39a2a35e6acd7773b246361c258f..97cf90a17b42db364b10bfd7ca2cddda5661d8c8 100644
--- a/ppapi/shared_impl/media_stream_buffer_manager_unittest.cc
+++ b/ppapi/shared_impl/media_stream_buffer_manager_unittest.cc
@@ -8,7 +8,6 @@
#include <utility>
-#include "base/memory/scoped_ptr.h"
#include "base/memory/shared_memory.h"
#include "ppapi/c/pp_errors.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -18,9 +17,9 @@ using base::SharedMemoryCreateOptions;
namespace {
-scoped_ptr<SharedMemory> CreateSharedMemory(int32_t buffer_size,
- int32_t number_of_buffers) {
- scoped_ptr<SharedMemory> shared_memory(new SharedMemory());
+std::unique_ptr<SharedMemory> CreateSharedMemory(int32_t buffer_size,
+ int32_t number_of_buffers) {
+ std::unique_ptr<SharedMemory> shared_memory(new SharedMemory());
SharedMemoryCreateOptions options;
options.size = buffer_size * number_of_buffers;
options.executable = false;
@@ -48,7 +47,7 @@ TEST(MediaStreamBufferManager, General) {
const int32_t kBufferSize = 128;
MockDelegate delegate;
MediaStreamBufferManager manager(&delegate);
- scoped_ptr<SharedMemory> shared_memory =
+ std::unique_ptr<SharedMemory> shared_memory =
CreateSharedMemory(kBufferSize, kNumberOfBuffers);
// SetBuffers with enqueue_all_buffers = true;
EXPECT_TRUE(manager.SetBuffers(kNumberOfBuffers, kBufferSize,
@@ -105,7 +104,7 @@ TEST(MediaStreamBufferManager, General) {
const int32_t kBufferSize = 128;
MockDelegate delegate;
MediaStreamBufferManager manager(&delegate);
- scoped_ptr<SharedMemory> shared_memory =
+ std::unique_ptr<SharedMemory> shared_memory =
CreateSharedMemory(kBufferSize, kNumberOfBuffers);
// SetBuffers with enqueue_all_buffers = false;
EXPECT_TRUE(manager.SetBuffers(kNumberOfBuffers, kBufferSize,
@@ -134,7 +133,7 @@ TEST(MediaStreamBufferManager, ResetBuffers) {
MockDelegate delegate;
MediaStreamBufferManager manager(&delegate);
{
- scoped_ptr<SharedMemory> shared_memory(new SharedMemory());
+ std::unique_ptr<SharedMemory> shared_memory(new SharedMemory());
SharedMemoryCreateOptions options;
options.size = kBufferSize1 * kNumberOfBuffers1;
options.executable = false;
@@ -160,7 +159,7 @@ TEST(MediaStreamBufferManager, ResetBuffers) {
}
{
- scoped_ptr<SharedMemory> shared_memory =
+ std::unique_ptr<SharedMemory> shared_memory =
CreateSharedMemory(kBufferSize2, kNumberOfBuffers2);
// SetBuffers with enqueue_all_buffers = true;
EXPECT_TRUE(manager.SetBuffers(kNumberOfBuffers2, kBufferSize2,
« no previous file with comments | « ppapi/shared_impl/media_stream_buffer_manager.cc ('k') | ppapi/shared_impl/ppb_audio_shared.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698