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

Unified Diff: services/media/audio/audio_track_impl.h

Issue 1986303002: Motown: Use new TimelineTransform and related definitions (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Fixes per feedback. Created 4 years, 7 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/audio/audio_track_impl.h
diff --git a/services/media/audio/audio_track_impl.h b/services/media/audio/audio_track_impl.h
index f282849a9d3c846d7666c4d2173e7a16e7138ed8..71ec3e1a8ada89ce940368b9d78fb3dcbfa337d7 100644
--- a/services/media/audio/audio_track_impl.h
+++ b/services/media/audio/audio_track_impl.h
@@ -15,7 +15,7 @@
#include "mojo/services/media/common/cpp/linear_transform.h"
#include "services/media/audio/audio_pipe.h"
#include "services/media/audio/fwd_decls.h"
-#include "services/media/common/rate_control_base.h"
+#include "services/media/common/timeline_control_site.h"
namespace mojo {
namespace media {
@@ -41,9 +41,7 @@ class AudioTrackImpl : public AudioTrack {
// Accessors used by AudioOutputs during mixing to access parameters which are
// important for the mixing process.
- void SnapshotRateTrans(LinearTransform* out, uint32_t* generation = nullptr) {
- rate_control_.SnapshotCurrentTransform(out, generation);
- }
+ void SnapshotRateTrans(LinearTransform* out, uint32_t* generation = nullptr);
const LinearTransform::Ratio& FractionalFrameToMediaTimeRatio() const {
return frame_to_media_ratio_;
@@ -63,7 +61,8 @@ class AudioTrackImpl : public AudioTrack {
void Describe(const DescribeCallback& cbk) override;
void Configure(AudioTrackConfigurationPtr configuration,
InterfaceRequest<MediaConsumer> req) override;
- void GetRateControl(InterfaceRequest<RateControl> req) override;
+ void GetTimelineControlSite(InterfaceRequest<MediaTimelineControlSite> req)
+ override;
void SetGain(float db_gain) override;
// Methods called by our AudioPipe.
@@ -80,7 +79,8 @@ class AudioTrackImpl : public AudioTrack {
AudioServerImpl* owner_;
Binding<AudioTrack> binding_;
AudioPipe pipe_;
- RateControlBase rate_control_;
+ TimelineControlSite timeline_control_site_;
+ TimelineRate frames_per_ns_;
LinearTransform::Ratio frame_to_media_ratio_;
uint32_t bytes_per_frame_ = 1;
AudioMediaTypeDetailsPtr format_;
« no previous file with comments | « mojo/services/media/core/interfaces/timeline_controller.mojom ('k') | services/media/audio/audio_track_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698