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

Side by Side Diff: media/blink/webmediaplayer_impl.h

Issue 2689863002: Change ownership of PipelineImpl (Closed)
Patch Set: Proper rebase Created 3 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 unified diff | Download patch
« no previous file with comments | « media/base/pipeline_impl.cc ('k') | media/blink/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 MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 5 #ifndef MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
6 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 6 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 460 matching lines...) Expand 10 before | Expand all | Expand 10 after
471 MultibufferDataSource::BufferingStrategy buffering_strategy_; 471 MultibufferDataSource::BufferingStrategy buffering_strategy_;
472 472
473 // Task runner for posting tasks on Chrome's main thread. Also used 473 // Task runner for posting tasks on Chrome's main thread. Also used
474 // for DCHECKs so methods calls won't execute in the wrong thread. 474 // for DCHECKs so methods calls won't execute in the wrong thread.
475 const scoped_refptr<base::SingleThreadTaskRunner> main_task_runner_; 475 const scoped_refptr<base::SingleThreadTaskRunner> main_task_runner_;
476 476
477 scoped_refptr<base::SingleThreadTaskRunner> media_task_runner_; 477 scoped_refptr<base::SingleThreadTaskRunner> media_task_runner_;
478 scoped_refptr<base::TaskRunner> worker_task_runner_; 478 scoped_refptr<base::TaskRunner> worker_task_runner_;
479 scoped_refptr<MediaLog> media_log_; 479 scoped_refptr<MediaLog> media_log_;
480 480
481 // |pipeline_controller_| references |pipeline_| and therefore must be 481 // |pipeline_controller_| owns an instance of Pipeline.
482 // constructed after and destructed before |pipeline_|.
483 PipelineImpl pipeline_;
484 PipelineController pipeline_controller_; 482 PipelineController pipeline_controller_;
485 483
486 // The LoadType passed in the |load_type| parameter of the load() call. 484 // The LoadType passed in the |load_type| parameter of the load() call.
487 LoadType load_type_; 485 LoadType load_type_;
488 486
489 // Cache of metadata for answering hasAudio(), hasVideo(), and naturalSize(). 487 // Cache of metadata for answering hasAudio(), hasVideo(), and naturalSize().
490 PipelineMetadata pipeline_metadata_; 488 PipelineMetadata pipeline_metadata_;
491 489
492 // Whether the video is known to be opaque or not. 490 // Whether the video is known to be opaque or not.
493 bool opaque_; 491 bool opaque_;
(...skipping 213 matching lines...) Expand 10 before | Expand all | Expand 10 after
707 // Whether the video requires a user gesture to resume after it was paused in 705 // Whether the video requires a user gesture to resume after it was paused in
708 // the background. Affects the value of ShouldPauseVideoWhenHidden(). 706 // the background. Affects the value of ShouldPauseVideoWhenHidden().
709 bool video_locked_when_paused_when_hidden_ = false; 707 bool video_locked_when_paused_when_hidden_ = false;
710 708
711 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl); 709 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl);
712 }; 710 };
713 711
714 } // namespace media 712 } // namespace media
715 713
716 #endif // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 714 #endif // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
OLDNEW
« no previous file with comments | « media/base/pipeline_impl.cc ('k') | media/blink/webmediaplayer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698