MediaRecorder: support |timecode| and remove |m_ignoreMutedMedia|.

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}
Committed: https://chromium.googlesource.com/chromium/src/+/ab5ae32c6233fe7938b9a0b53cf1c74918310d7a

[mcasas, 2017-01-06 01:18:28]

Description was changed from

==========
MediaRecorder: support |timecode| and remove |m_ignoreMutedMedia|.



BUG=676211, 678854
==========

to

==========
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
==========

[mcasas, 2017-01-06 02:39:08]

mcasas@chromium.org changed reviewers:
+ emircan@chromium.org

[mcasas, 2017-01-06 02:39:08]

emircan@ ptal

[mcasas, 2017-01-06 02:39:22]

Patchset #1 (id:1) has been deleted

[emircan, 2017-01-06 19:02:55]

lgtm

[mcasas, 2017-01-06 21:42:55]

Patchset #3 (id:60001) has been deleted

[mcasas, 2017-01-06 23:58:29]

Patchset #4 (id:100001) has been deleted

[mcasas, 2017-01-07 00:45:09]

mcasas@chromium.org changed reviewers:
+ foolip@chromium.org

[mcasas, 2017-01-07 00:45:09]

foolip@ please RS:

WebMediaRecorderHandlerClient.h and
the expected.txt updates

[mcasas, 2017-01-09 15:35:29]

mcasas@chromium.org changed reviewers:
+ haraken@chromium.org

[mcasas, 2017-01-09 15:35:30]

haraken@ please RS:

WebMediaRecorderHandlerClient.h and
the ...expected.txt updates

[haraken, 2017-01-09 15:37:52]

LGTM but this will need an approval of an API owner.

[foolip, 2017-01-09 16:50:21]

https://github.com/w3c/mediacapture-record/pull/95 had LGTM by Gecko dev, so
LGTM.

[mcasas, 2017-01-09 17:27:41]

The CQ bit was checked by mcasas@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-09 17:27:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-09 17:30:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-09 17:30:12]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)

[mcasas, 2017-01-09 17:39:58]

Patchset #2 (id:40001) has been deleted

[mcasas, 2017-01-09 17:45:46]

The CQ bit was checked by mcasas@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-09 17:46:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-09 21:33:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-09 21:33:47]

Dry run: This issue passed the CQ dry run.

[mcasas, 2017-01-10 17:21:54]

The CQ bit was checked by mcasas@chromium.org

[mcasas, 2017-01-10 17:21:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
emircan@chromium.org, haraken@chromium.org, foolip@chromium.org
Link to the patchset: https://codereview.chromium.org/2610163006/#ps120001
(title: "Rebase video_capture_device_client.cc")

[commit-bot: I haz the power, 2017-01-10 17:22:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-10 17:26:44]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1484068914537690,
"parent_rev": "7e1be3e34f15577a4f7d500fe547d06af949eb25", "commit_rev":
"ab5ae32c6233fe7938b9a0b53cf1c74918310d7a"}

[commit-bot: I haz the power, 2017-01-10 17:27:21]

Description was changed from

==========
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
==========

to

==========
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}
Committed:
https://chromium.googlesource.com/chromium/src/+/ab5ae32c6233fe7938b9a0b53cf1...
==========

[commit-bot: I haz the power, 2017-01-10 17:27:22]

Committed patchset #3 (id:120001) as
https://chromium.googlesource.com/chromium/src/+/ab5ae32c6233fe7938b9a0b53cf1...