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

Side by Side Diff: media/mojo/services/media_renderer_apptest.cc

Issue 692323002: Move Liveness from DemuxerStreamProvider to DemuxerStream. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix mojo Created 6 years, 1 month 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/bind.h" 5 #include "base/bind.h"
6 #include "base/message_loop/message_loop.h" 6 #include "base/message_loop/message_loop.h"
7 #include "base/run_loop.h" 7 #include "base/run_loop.h"
8 #include "media/base/audio_decoder_config.h" 8 #include "media/base/audio_decoder_config.h"
9 #include "media/base/channel_layout.h" 9 #include "media/base/channel_layout.h"
10 #include "media/base/demuxer_stream_provider.h" 10 #include "media/base/demuxer_stream_provider.h"
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 public: 44 public:
45 FakeDemuxerStream() {} 45 FakeDemuxerStream() {}
46 ~FakeDemuxerStream() override {} 46 ~FakeDemuxerStream() override {}
47 47
48 // media::Demuxer implementation. 48 // media::Demuxer implementation.
49 media::DemuxerStream* GetStream(media::DemuxerStream::Type type) override { 49 media::DemuxerStream* GetStream(media::DemuxerStream::Type type) override {
50 if (type == media::DemuxerStream::AUDIO) 50 if (type == media::DemuxerStream::AUDIO)
51 return this; 51 return this;
52 return nullptr; 52 return nullptr;
53 } 53 }
54 media::DemuxerStreamProvider::Liveness GetLiveness() const override {
55 return media::DemuxerStreamProvider::LIVENESS_UNKNOWN;
56 }
57 54
58 // media::DemuxerStream implementation. 55 // media::DemuxerStream implementation.
59 void Read(const ReadCB& read_cb) override {} 56 void Read(const ReadCB& read_cb) override {}
60 57
61 media::AudioDecoderConfig audio_decoder_config() override { 58 media::AudioDecoderConfig audio_decoder_config() override {
62 media::AudioDecoderConfig config; 59 media::AudioDecoderConfig config;
63 config.Initialize(media::kCodecAAC, 60 config.Initialize(media::kCodecAAC,
64 media::kSampleFormatU8, 61 media::kSampleFormatU8,
65 media::CHANNEL_LAYOUT_SURROUND, 62 media::CHANNEL_LAYOUT_SURROUND,
66 48000, 63 48000,
67 NULL, 64 NULL,
68 0, 65 0,
69 false, 66 false,
70 false, 67 false,
71 base::TimeDelta(), 68 base::TimeDelta(),
72 0); 69 0);
73 return config; 70 return config;
74 } 71 }
75 72
76 media::VideoDecoderConfig video_decoder_config() override { 73 media::VideoDecoderConfig video_decoder_config() override {
77 NOTREACHED(); 74 NOTREACHED();
78 return media::VideoDecoderConfig(); 75 return media::VideoDecoderConfig();
79 } 76 }
80 77
81 media::DemuxerStream::Type type() override { 78 media::DemuxerStream::Type type() const override {
82 return media::DemuxerStream::AUDIO; 79 return media::DemuxerStream::AUDIO;
83 } 80 }
84 81
85 void EnableBitstreamConverter() override {} 82 void EnableBitstreamConverter() override {}
86 83
87 bool SupportsConfigChanges() override { return true; } 84 bool SupportsConfigChanges() override { return true; }
88 85
89 media::VideoRotation video_rotation() override { 86 media::VideoRotation video_rotation() override {
90 NOTREACHED(); 87 NOTREACHED();
91 return media::VIDEO_ROTATION_0; 88 return media::VIDEO_ROTATION_0;
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
154 media::Renderer::PaintCB(), base::Closure(), 151 media::Renderer::PaintCB(), base::Closure(),
155 base::Bind(&ErrorCallback, &expected_error)); 152 base::Bind(&ErrorCallback, &expected_error));
156 base::MessageLoop::current()->Run(); 153 base::MessageLoop::current()->Run();
157 154
158 // We expect an error during initialization because MojoRendererService 155 // We expect an error during initialization because MojoRendererService
159 // doesn't initialize any decoders, which causes an error. 156 // doesn't initialize any decoders, which causes an error.
160 EXPECT_EQ(PIPELINE_ERROR_COULD_NOT_RENDER, expected_error); 157 EXPECT_EQ(PIPELINE_ERROR_COULD_NOT_RENDER, expected_error);
161 } 158 }
162 159
163 } // namespace media 160 } // namespace media
OLDNEW
« no previous file with comments | « media/mojo/services/demuxer_stream_provider_shim.cc ('k') | media/mojo/services/mojo_demuxer_stream_adapter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698