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

Unified Diff: content/browser/renderer_host/media/audio_input_sync_writer.h

Issue 2752533003: Remove ScopedVector from content/browser/. (Closed)
Patch Set: Created 3 years, 9 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
Index: content/browser/renderer_host/media/audio_input_sync_writer.h
diff --git a/content/browser/renderer_host/media/audio_input_sync_writer.h b/content/browser/renderer_host/media/audio_input_sync_writer.h
index 1d883e18a45319a39840fccf1e06b18797b2e9e1..880e49c2fc9e9b985f1adb1b8ea02ced5ccc6f9a 100644
--- a/content/browser/renderer_host/media/audio_input_sync_writer.h
+++ b/content/browser/renderer_host/media/audio_input_sync_writer.h
@@ -9,10 +9,11 @@
#include <stdint.h>
#include <deque>
+#include <memory>
+#include <vector>
#include "base/gtest_prod_util.h"
#include "base/macros.h"
-#include "base/memory/scoped_vector.h"
#include "base/process/process.h"
#include "base/sync_socket.h"
#include "base/time/time.h"
@@ -149,14 +150,14 @@ class CONTENT_EXPORT AudioInputSyncWriter
// Vector of audio buses allocated during construction and deleted in the
// destructor.
- ScopedVector<media::AudioBus> audio_buses_;
+ std::vector<std::unique_ptr<media::AudioBus>> audio_buses_;
// Fifo for audio that is used in case there isn't room in the shared memory.
// This can for example happen under load when the consumer side is starved.
// It should ideally be rare, but we need to guarantee that the data arrives
// since audio processing such as echo cancelling requires that to perform
// properly.
- ScopedVector<media::AudioBus> overflow_buses_;
+ std::vector<std::unique_ptr<media::AudioBus>> overflow_buses_;
struct OverflowParams {
double volume;
uint32_t hardware_delay_bytes;

Powered by Google App Engine
This is Rietveld 408576698