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

Side by Side Diff: content/renderer/media/webmediaplayer_impl.h

Issue 343053003: Remove WebMediaPlayerImpl::starting_ as it's redundant. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | content/renderer/media/webmediaplayer_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #ifndef CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_IMPL_H_ 5 #ifndef CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_IMPL_H_
6 #define CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_IMPL_H_ 6 #define CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_IMPL_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after
305 bool incremented_externally_allocated_memory_; 305 bool incremented_externally_allocated_memory_;
306 306
307 // Factories for supporting video accelerators. May be null. 307 // Factories for supporting video accelerators. May be null.
308 scoped_refptr<media::GpuVideoAcceleratorFactories> gpu_factories_; 308 scoped_refptr<media::GpuVideoAcceleratorFactories> gpu_factories_;
309 309
310 // Routes audio playback to either AudioRendererSink or WebAudio. 310 // Routes audio playback to either AudioRendererSink or WebAudio.
311 scoped_refptr<WebAudioSourceProviderImpl> audio_source_provider_; 311 scoped_refptr<WebAudioSourceProviderImpl> audio_source_provider_;
312 312
313 bool supports_save_; 313 bool supports_save_;
314 314
315 bool starting_;
316
317 // These two are mutually exclusive: 315 // These two are mutually exclusive:
318 // |data_source_| is used for regular resource loads. 316 // |data_source_| is used for regular resource loads.
319 // |chunk_demuxer_| is used for Media Source resource loads. 317 // |chunk_demuxer_| is used for Media Source resource loads.
320 // 318 //
321 // |demuxer_| will contain the appropriate demuxer based on which resource 319 // |demuxer_| will contain the appropriate demuxer based on which resource
322 // load strategy we're using. 320 // load strategy we're using.
323 scoped_ptr<BufferedDataSource> data_source_; 321 scoped_ptr<BufferedDataSource> data_source_;
324 scoped_ptr<media::Demuxer> demuxer_; 322 scoped_ptr<media::Demuxer> demuxer_;
325 media::ChunkDemuxer* chunk_demuxer_; 323 media::ChunkDemuxer* chunk_demuxer_;
326 324
(...skipping 23 matching lines...) Expand all
350 WebContentDecryptionModuleImpl* web_cdm_; 348 WebContentDecryptionModuleImpl* web_cdm_;
351 349
352 media::DecryptorReadyCB decryptor_ready_cb_; 350 media::DecryptorReadyCB decryptor_ready_cb_;
353 351
354 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl); 352 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl);
355 }; 353 };
356 354
357 } // namespace content 355 } // namespace content
358 356
359 #endif // CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_IMPL_H_ 357 #endif // CONTENT_RENDERER_MEDIA_WEBMEDIAPLAYER_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | content/renderer/media/webmediaplayer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698