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

Side by Side Diff: content/renderer/renderer_blink_platform_impl.cc

Issue 2425703002: Remove |remote| and |readonly| members of MediaStreamTrack (Closed)
Patch Set: Fixed webkit_tests Created 3 years, 9 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/renderer_blink_platform_impl.h" 5 #include "content/renderer/renderer_blink_platform_impl.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 840 matching lines...) Expand 10 before | Expand all | Expand 10 after
851 851
852 void RendererBlinkPlatformImpl::createHTMLVideoElementCapturer( 852 void RendererBlinkPlatformImpl::createHTMLVideoElementCapturer(
853 WebMediaStream* web_media_stream, 853 WebMediaStream* web_media_stream,
854 WebMediaPlayer* web_media_player) { 854 WebMediaPlayer* web_media_player) {
855 #if BUILDFLAG(ENABLE_WEBRTC) 855 #if BUILDFLAG(ENABLE_WEBRTC)
856 DCHECK(web_media_stream); 856 DCHECK(web_media_stream);
857 DCHECK(web_media_player); 857 DCHECK(web_media_player);
858 AddVideoTrackToMediaStream( 858 AddVideoTrackToMediaStream(
859 HtmlVideoElementCapturerSource::CreateFromWebMediaPlayerImpl( 859 HtmlVideoElementCapturerSource::CreateFromWebMediaPlayerImpl(
860 web_media_player, content::RenderThread::Get()->GetIOTaskRunner()), 860 web_media_player, content::RenderThread::Get()->GetIOTaskRunner()),
861 false, // is_remote
862 false, // is_readonly
863 web_media_stream); 861 web_media_stream);
864 #endif 862 #endif
865 } 863 }
866 864
867 void RendererBlinkPlatformImpl::createHTMLAudioElementCapturer( 865 void RendererBlinkPlatformImpl::createHTMLAudioElementCapturer(
868 WebMediaStream* web_media_stream, 866 WebMediaStream* web_media_stream,
869 WebMediaPlayer* web_media_player) { 867 WebMediaPlayer* web_media_player) {
870 #if BUILDFLAG(ENABLE_WEBRTC) 868 #if BUILDFLAG(ENABLE_WEBRTC)
871 DCHECK(web_media_stream); 869 DCHECK(web_media_stream);
872 DCHECK(web_media_player); 870 DCHECK(web_media_player);
873 871
874 blink::WebMediaStreamSource web_media_stream_source; 872 blink::WebMediaStreamSource web_media_stream_source;
875 blink::WebMediaStreamTrack web_media_stream_track; 873 blink::WebMediaStreamTrack web_media_stream_track;
876 const WebString track_id = WebString::fromUTF8(base::GenerateGUID()); 874 const WebString track_id = WebString::fromUTF8(base::GenerateGUID());
877 875
878 web_media_stream_source.initialize(track_id, 876 web_media_stream_source.initialize(
879 blink::WebMediaStreamSource::TypeAudio, 877 track_id, blink::WebMediaStreamSource::TypeAudio, track_id);
880 track_id,
881 false /* is_remote */);
882 web_media_stream_track.initialize(web_media_stream_source); 878 web_media_stream_track.initialize(web_media_stream_source);
883 879
884 MediaStreamAudioSource* const media_stream_source = 880 MediaStreamAudioSource* const media_stream_source =
885 HtmlAudioElementCapturerSource::CreateFromWebMediaPlayerImpl( 881 HtmlAudioElementCapturerSource::CreateFromWebMediaPlayerImpl(
886 web_media_player); 882 web_media_player);
887 883
888 // Takes ownership of |media_stream_source|. 884 // Takes ownership of |media_stream_source|.
889 web_media_stream_source.setExtraData(media_stream_source); 885 web_media_stream_source.setExtraData(media_stream_source);
890 886
891 media_stream_source->ConnectToTrack(web_media_stream_track); 887 media_stream_source->ConnectToTrack(web_media_stream_track);
(...skipping 371 matching lines...) Expand 10 before | Expand all | Expand 10 after
1263 //------------------------------------------------------------------------------ 1259 //------------------------------------------------------------------------------
1264 void RendererBlinkPlatformImpl::requestPurgeMemory() { 1260 void RendererBlinkPlatformImpl::requestPurgeMemory() {
1265 // TODO(tasak|bashi): We should use ChildMemoryCoordinator here, but 1261 // TODO(tasak|bashi): We should use ChildMemoryCoordinator here, but
1266 // ChildMemoryCoordinator isn't always available as it's only initialized 1262 // ChildMemoryCoordinator isn't always available as it's only initialized
1267 // when kMemoryCoordinatorV0 is enabled. 1263 // when kMemoryCoordinatorV0 is enabled.
1268 // Use ChildMemoryCoordinator when memory coordinator is always enabled. 1264 // Use ChildMemoryCoordinator when memory coordinator is always enabled.
1269 base::MemoryCoordinatorClientRegistry::GetInstance()->PurgeMemory(); 1265 base::MemoryCoordinatorClientRegistry::GetInstance()->PurgeMemory();
1270 } 1266 }
1271 1267
1272 } // namespace content 1268 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698