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

Side by Side Diff: content/renderer/media/android/webmediaplayer_android.cc

Issue 2338223003: Unassociating MediaSession from media players (in blink & content) (Closed)
Patch Set: fixed tests Created 4 years, 3 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
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 #include "content/renderer/media/android/webmediaplayer_android.h" 5 #include "content/renderer/media/android/webmediaplayer_android.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <limits> 9 #include <limits>
10 10
11 #include "base/android/build_info.h" 11 #include "base/android/build_info.h"
12 #include "base/bind.h" 12 #include "base/bind.h"
13 #include "base/callback_helpers.h" 13 #include "base/callback_helpers.h"
14 #include "base/command_line.h" 14 #include "base/command_line.h"
15 #include "base/files/file_path.h" 15 #include "base/files/file_path.h"
16 #include "base/logging.h" 16 #include "base/logging.h"
17 #include "base/metrics/histogram_macros.h" 17 #include "base/metrics/histogram_macros.h"
18 #include "base/single_thread_task_runner.h" 18 #include "base/single_thread_task_runner.h"
19 #include "base/strings/string_number_conversions.h" 19 #include "base/strings/string_number_conversions.h"
20 #include "base/strings/utf_string_conversions.h" 20 #include "base/strings/utf_string_conversions.h"
21 #include "cc/blink/web_layer_impl.h" 21 #include "cc/blink/web_layer_impl.h"
22 #include "cc/layers/video_layer.h" 22 #include "cc/layers/video_layer.h"
23 #include "content/public/common/content_client.h" 23 #include "content/public/common/content_client.h"
24 #include "content/public/common/content_switches.h" 24 #include "content/public/common/content_switches.h"
25 #include "content/public/common/renderer_preferences.h" 25 #include "content/public/common/renderer_preferences.h"
26 #include "content/public/renderer/render_frame.h" 26 #include "content/public/renderer/render_frame.h"
27 #include "content/renderer/media/android/renderer_demuxer_android.h" 27 #include "content/renderer/media/android/renderer_demuxer_android.h"
28 #include "content/renderer/media/android/renderer_media_player_manager.h" 28 #include "content/renderer/media/android/renderer_media_player_manager.h"
29 #include "content/renderer/media/android/webmediasession_android.h"
30 #include "content/renderer/render_frame_impl.h" 29 #include "content/renderer/render_frame_impl.h"
31 #include "content/renderer/render_thread_impl.h" 30 #include "content/renderer/render_thread_impl.h"
32 #include "content/renderer/render_view_impl.h" 31 #include "content/renderer/render_view_impl.h"
33 #include "gpu/GLES2/gl2extchromium.h" 32 #include "gpu/GLES2/gl2extchromium.h"
34 #include "gpu/command_buffer/client/gles2_interface.h" 33 #include "gpu/command_buffer/client/gles2_interface.h"
35 #include "gpu/command_buffer/common/constants.h" 34 #include "gpu/command_buffer/common/constants.h"
36 #include "gpu/command_buffer/common/mailbox_holder.h" 35 #include "gpu/command_buffer/common/mailbox_holder.h"
37 #include "media/base/android/media_codec_util.h" 36 #include "media/base/android/media_codec_util.h"
38 #include "media/base/android/media_common_android.h" 37 #include "media/base/android/media_common_android.h"
39 #include "media/base/android/media_player_android.h" 38 #include "media/base/android/media_player_android.h"
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 delegate_(delegate), 145 delegate_(delegate),
147 delegate_id_(0), 146 delegate_id_(0),
148 defer_load_cb_(params.defer_load_cb()), 147 defer_load_cb_(params.defer_load_cb()),
149 buffered_(static_cast<size_t>(1)), 148 buffered_(static_cast<size_t>(1)),
150 media_task_runner_(params.media_task_runner()), 149 media_task_runner_(params.media_task_runner()),
151 ignore_metadata_duration_change_(false), 150 ignore_metadata_duration_change_(false),
152 pending_seek_(false), 151 pending_seek_(false),
153 seeking_(false), 152 seeking_(false),
154 did_loading_progress_(false), 153 did_loading_progress_(false),
155 player_manager_(player_manager), 154 player_manager_(player_manager),
156 media_session_id_(params.media_session()
157 ? static_cast<const WebMediaSessionAndroid*>(
158 params.media_session())
159 ->media_session_id()
160 : blink::WebMediaSession::DefaultID),
161 network_state_(WebMediaPlayer::NetworkStateEmpty), 155 network_state_(WebMediaPlayer::NetworkStateEmpty),
162 ready_state_(WebMediaPlayer::ReadyStateHaveNothing), 156 ready_state_(WebMediaPlayer::ReadyStateHaveNothing),
163 texture_id_(0), 157 texture_id_(0),
164 stream_id_(0), 158 stream_id_(0),
165 is_player_initialized_(false), 159 is_player_initialized_(false),
166 is_playing_(false), 160 is_playing_(false),
167 is_play_pending_(false), 161 is_play_pending_(false),
168 needs_establish_peer_(true), 162 needs_establish_peer_(true),
169 has_size_info_(false), 163 has_size_info_(false),
170 // Threaded compositing isn't enabled universally yet. 164 // Threaded compositing isn't enabled universally yet.
(...skipping 924 matching lines...) Expand 10 before | Expand all | Expand 10 after
1095 void WebMediaPlayerAndroid::InitializePlayer( 1089 void WebMediaPlayerAndroid::InitializePlayer(
1096 const GURL& url, 1090 const GURL& url,
1097 const GURL& first_party_for_cookies, 1091 const GURL& first_party_for_cookies,
1098 bool allow_stored_credentials, 1092 bool allow_stored_credentials,
1099 int demuxer_client_id) { 1093 int demuxer_client_id) {
1100 ReportHLSMetrics(); 1094 ReportHLSMetrics();
1101 1095
1102 allow_stored_credentials_ = allow_stored_credentials; 1096 allow_stored_credentials_ = allow_stored_credentials;
1103 player_manager_->Initialize( 1097 player_manager_->Initialize(
1104 player_type_, player_id_, url, first_party_for_cookies, demuxer_client_id, 1098 player_type_, player_id_, url, first_party_for_cookies, demuxer_client_id,
1105 frame_->document().url(), allow_stored_credentials, delegate_id_, 1099 frame_->document().url(), allow_stored_credentials, delegate_id_);
1106 media_session_id_);
1107 is_player_initialized_ = true; 1100 is_player_initialized_ = true;
1108 1101
1109 if (is_fullscreen_) 1102 if (is_fullscreen_)
1110 player_manager_->EnterFullscreen(player_id_); 1103 player_manager_->EnterFullscreen(player_id_);
1111 1104
1112 if (cdm_context_) 1105 if (cdm_context_)
1113 SetCdmInternal(base::Bind(&media::IgnoreCdmAttached)); 1106 SetCdmInternal(base::Bind(&media::IgnoreCdmAttached));
1114 } 1107 }
1115 1108
1116 void WebMediaPlayerAndroid::Pause(bool is_media_related_action) { 1109 void WebMediaPlayerAndroid::Pause(bool is_media_related_action) {
(...skipping 496 matching lines...) Expand 10 before | Expand all | Expand 10 after
1613 if (base::CommandLine::ForCurrentProcess()->HasSwitch( 1606 if (base::CommandLine::ForCurrentProcess()->HasSwitch(
1614 switches::kDisableMediaSuspend)) { 1607 switches::kDisableMediaSuspend)) {
1615 return false; 1608 return false;
1616 } 1609 }
1617 1610
1618 return base::FeatureList::IsEnabled(media::kResumeBackgroundVideo) && 1611 return base::FeatureList::IsEnabled(media::kResumeBackgroundVideo) &&
1619 hasAudio() && !isRemote() && delegate_ && delegate_->IsHidden(); 1612 hasAudio() && !isRemote() && delegate_ && delegate_->IsHidden();
1620 } 1613 }
1621 1614
1622 } // namespace content 1615 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/android/webmediaplayer_android.h ('k') | content/renderer/render_frame_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698