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

Issue 2710213005: Revert of Remove |remote| and |readonly| members of MediaStreamTrack (Closed)

Created:
3 years, 10 months ago by Mathieu
Modified:
3 years, 10 months ago
CC:
avayvod+watch_chromium.org, blink-reviews, blink-reviews-api_chromium.org, chromium-apps-reviews_chromium.org, chromium-reviews, darin-cc_chromium.org, dglazkov+blink, einbinder+watch-test-runner_chromium.org, eric.carlson_apple.com, extensions-reviews_chromium.org, feature-media-reviews_chromium.org, haraken, hongchan, imcheng+watch_chromium.org, isheriff+watch_chromium.org, jam, jasonroberts+watch_google.com, jochen+watch_chromium.org, mcasas+watch+vc_chromium.org, mcasas+watch+mediastream_chromium.org, miu+watch_chromium.org, mlamouri+watch-content_chromium.org, mlamouri+watch-test-runner_chromium.org, mlamouri+watch-blink_chromium.org, Peter Beverloo, posciak+watch_chromium.org, Raymond Toy, Srirama, tommyw+watchlist_chromium.org, xjz+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Remove |remote| and |readonly| members of MediaStreamTrack (patchset #10 id:200001 of https://codereview.chromium.org/2425703002/ ) Reason for revert: Failure on webkit_tests: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty/builds/23714 external/wpt/mediacapture-streams/MediaStreamTrack-init.https.html Original issue's description: > Remove |remote| and |readonly| members of MediaStreamTrack. > > Spec reference: > http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1 > > removal was in the February 22, 2016'. > [#321] Remove track attributes "remote" and "readonly" : > https://github.com/w3c/mediacapture-main/pull/321 > > Intent to Deprecate and Remove : > https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/d20ECb2sWzI > > BUG=598704 > > Review-Url: https://codereview.chromium.org/2425703002 > Cr-Commit-Position: refs/heads/master@{#452851} > Committed: https://chromium.googlesource.com/chromium/src/+/a72385a6addeb6194d3568beda3f352e07c97260 TBR=jochen@chromium.org,bbudge@chromium.org,drott@chromium.org,esprehn@chromium.org,hongchan@chromium.org,hta@chromium.org,mcasas@chromium.org,miu@chromium.org,peter@chromium.org,scottmg@chromium.org,peary2@gmail.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=598704 Review-Url: https://codereview.chromium.org/2710213005 Cr-Commit-Position: refs/heads/master@{#452862} Committed: https://chromium.googlesource.com/chromium/src/+/705b13eab5b3969e6f351cd23bb536ec204a78fc

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+261 lines, -54 lines) Patch
M chrome/renderer/extensions/cast_streaming_native_handler.cc View 1 chunk +6 lines, -2 lines 0 comments Download
M content/public/renderer/media_stream_utils.h View 2 chunks +7 lines, -0 lines 0 comments Download
M content/public/renderer/media_stream_utils.cc View 4 chunks +12 lines, -5 lines 0 comments Download
M content/renderer/media/capturefromelement/canvas_capture_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/media/capturefromelement/html_audio_element_capturer_source_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/external_media_stream_audio_source.h View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/external_media_stream_audio_source.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M content/renderer/media/media_stream_audio_source.h View 1 chunk +0 lines, -1 line 0 comments Download
M content/renderer/media/media_stream_audio_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/media_stream_video_capturer_source_unittest.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M content/renderer/media/media_stream_video_renderer_sink_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/media_stream_video_source_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/media_stream_video_track_unittest.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M content/renderer/media/mock_media_stream_registry.cc View 2 chunks +5 lines, -4 lines 0 comments Download
M content/renderer/media/pepper_to_video_track_adapter.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/remote_media_stream_impl.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/rtc_peer_connection_handler_unittest.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M content/renderer/media/speech_recognition_audio_sink_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/user_media_client_impl.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/webmediaplayer_ms_compositor.cc View 2 chunks +6 lines, -2 lines 0 comments Download
M content/renderer/media/webrtc/media_stream_remote_video_source_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/webrtc/processed_local_audio_source_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/webrtc/webrtc_media_stream_adapter_unittest.cc View 3 chunks +11 lines, -5 lines 0 comments Download
M content/renderer/media/webrtc_local_audio_source_provider_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media_recorder/audio_track_recorder_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/pepper/pepper_media_stream_video_track_host.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/renderer_blink_platform_impl.cc View 2 chunks +6 lines, -2 lines 0 comments Download
M content/shell/renderer/layout_test/blink_test_runner.cc View 2 chunks +4 lines, -0 lines 0 comments Download
M content/shell/test_runner/mock_web_user_media_client.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M content/shell/test_runner/mock_webrtc_peer_connection_handler.cc View 2 chunks +2 lines, -2 lines 0 comments Download
A third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection-remotestreams.html View 1 chunk +86 lines, -0 lines 0 comments Download
A third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection-remotestreams-expected.txt View 1 chunk +27 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/platform/mac/virtual/stable/webexposed/global-interface-listing-expected.txt View 1 chunk +1 line, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/platform/win/virtual/stable/webexposed/global-interface-listing-expected.txt View 1 chunk +1 line, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt View 1 chunk +1 line, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.h View 1 chunk +1 line, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp View 1 chunk +4 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.idl View 1 chunk +4 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/webaudio/MediaStreamAudioDestinationNode.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/platform/exported/WebMediaStreamSource.cpp View 2 chunks +14 lines, -1 line 0 comments Download
M third_party/WebKit/Source/platform/mediastream/MediaStreamSource.h View 2 chunks +4 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/platform/mediastream/MediaStreamSource.cpp View 1 chunk +5 lines, -1 line 0 comments Download
M third_party/WebKit/public/platform/WebMediaStreamSource.h View 1 chunk +6 lines, -1 line 0 comments Download

Messages

Total messages: 6 (3 generated)
Mathieu
Created Revert of Remove |remote| and |readonly| members of MediaStreamTrack
3 years, 10 months ago (2017-02-24 17:44:07 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2710213005/1
3 years, 10 months ago (2017-02-24 17:44:29 UTC) #3
commit-bot: I haz the power
3 years, 10 months ago (2017-02-24 17:46:24 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/705b13eab5b3969e6f351cd23bb5...

Powered by Google App Engine
This is Rietveld 408576698