Remove attribute boolean ignoreMutedMedia; from MediaRecorder.idl after [2] [1] https://cs.chromium.org/chromium/src/third_party/WebKit/Source/modules/mediarecorder/MediaRecorder.idl?q=mediarecorder.idl&sq=package:chromium&dr [2] https://github.com/w3c/mediacapture-record/issues/99
https://www.chromestatus.com/admin/features/launch/5637418589880320?intent
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/D3Bmuvygci4
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a commit ab5ae32c6233fe7938b9a0b53cf1c74918310d7a Author: mcasas <mcasas@chromium.org> Date: Tue Jan 10 17:26:40 2017 MediaRecorder: support |timecode| and remove |m_ignoreMutedMedia|. This CL - adds |timecode| [1] to the produced BlobEvents (and BlobEventInits) and wires it from the platform. - removes MediaRecorder's ignoreMutedMedia after [2] [1] https://github.com/w3c/mediacapture-record/issues/73 [2] https://github.com/w3c/mediacapture-record/issues/99 BUG= 676211 , 678854 Review-Url: https://codereview.chromium.org/2610163006 Cr-Commit-Position: refs/heads/master@{#442619} [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/content/renderer/media/media_recorder_handler.cc [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/content/renderer/media/media_recorder_handler_unittest.cc [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/LayoutTests/fast/mediarecorder/BlobEvent-basic.html [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/LayoutTests/fast/mediarecorder/MediaRecorder-audio-video.html [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/LayoutTests/fast/mediarecorder/MediaRecorder-requestData.html [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/LayoutTests/platform/linux/virtual/stable/webexposed/global-interface-listing-expected.txt [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/LayoutTests/platform/win/virtual/stable/webexposed/global-interface-listing-expected.txt [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/LayoutTests/virtual/stable/webexposed/global-interface-listing-expected.txt [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/Source/modules/mediarecorder/BlobEvent.cpp [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/Source/modules/mediarecorder/BlobEvent.h [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/Source/modules/mediarecorder/BlobEvent.idl [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/Source/modules/mediarecorder/BlobEventInit.idl [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/Source/modules/mediarecorder/MediaRecorder.cpp [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/Source/modules/mediarecorder/MediaRecorder.h [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/Source/modules/mediarecorder/MediaRecorder.idl [modify] https://crrev.com/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a/third_party/WebKit/public/platform/WebMediaRecorderHandlerClient.h
Bulk move Blink>MediaStream>Recording ---> Blink>MediaRecording
owencm@chromium.org as PM for this.
Removing approval requested as that system is only for type=launch issues. This change is minor enough that it doesn't need a Chrome launch review. It already has blink-dev@ LGTM so proceed.
Comment 1 by mcasas@chromium.org
, Jan 9 2017