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

Unified Diff: services/media/framework/parts/null_sink.h

Issue 1678433002: Motown: Remove LPCM optimizations, fix prepare, add flush, add ActiveMultistreamSink model/stage (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: A couple of missed fixes. Created 4 years, 10 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: services/media/framework/parts/null_sink.h
diff --git a/services/media/framework/parts/null_sink.h b/services/media/framework/parts/null_sink.h
index 41846b59153d6ee591b794de0599da14b8666509..1e7591741f5a55aff92ff2963bba1e45fb7c9410 100644
--- a/services/media/framework/parts/null_sink.h
+++ b/services/media/framework/parts/null_sink.h
@@ -10,23 +10,19 @@
namespace mojo {
namespace media {
-class NullSink;
-
-typedef SharedPtr<NullSink, ActiveSink> NullSinkPtr;
-
// Sink that throws packets away.
class NullSink : public ActiveSink {
public:
- static NullSinkPtr Create() { return NullSinkPtr(new NullSink()); }
+ static std::shared_ptr<NullSink> Create() {
+ return std::shared_ptr<NullSink>(new NullSink());
+ }
~NullSink() override;
// ActiveSink implementation.
- bool must_allocate() const override;
-
- Allocator* allocator() override;
+ PayloadAllocator* allocator() override;
- void SetDemandCallback(DemandCallback demand_callback) override;
+ void SetDemandCallback(const DemandCallback& demand_callback) override;
void Prime() override;

Powered by Google App Engine
This is Rietveld 408576698