Revert of Fix getUserMedia so that failure is reported for invalid audio sources.

Revert of Fix getUserMedia so that failure is reported for invalid audio sources. (patchset #2 id:20001 of https://codereview.chromium.org/2623443002/ )

Reason for revert:
Revert due to WebRTC tests failing:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/56312

[4832:364:0109/131325.673:4351226:ERROR:browser_test_utils.cc(147)] Cannot communicate with DOMMessageQueue.
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(66): error: Value of: ExecuteScriptAndExtractString(shell(), javascript, &result)
  Actual: false
Expected: true
Failed to execute javascript callAndRenegotiateToVideo({audio: true}, {audio: true, video:true});.
From javascript: (nothing)
When executing 'callAndRenegotiateToVideo({audio: true}, {audio: true, video:true});'
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(90): error: Failed

and

[4704:4156:0109/131352.840:4378402:ERROR:browser_test_utils.cc(147)] Cannot communicate with DOMMessageQueue.
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(66): error: Value of: ExecuteScriptAndExtractString(shell(), javascript, &result)
  Actual: false
Expected: true
Failed to execute javascript getUserMediaInIframeAndCloseInSuccessCb({audio: true});.
From javascript: (nothing)
When executing 'getUserMediaInIframeAndCloseInSuccessCb({audio: true});'
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(90): error: Failed

Original issue's description:
> Fix getUserMedia so that failure is reported for invalid audio sources.
> This changes getUserMedia to wait for initialization of local
> audio sources before issuing the completion callback (either success or
> failure).
> Previously, if an error occurs between attempting to start a local
> audio source and the render side OnStreamCreated callback, getUserMedia
> would report successful completion with an audio track but no audio
> would actually be delivered for that track.
> 
> BUG=679210
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Review-Url: https://codereview.chromium.org/2623443002
> Cr-Commit-Position: refs/heads/master@{#442323}
> Committed: https://chromium.googlesource.com/chromium/src/+/81cb66acf3f8099e416c7fc5ecbcaabf6d5c3c04

TBR=jochen@chromium.org,guidou@chromium.org,xhwang@chromium.org,tommi@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679210
Review-Url: https://codereview.chromium.org/2626533002
Cr-Commit-Position: refs/heads/master@{#442373}
Committed: https://chromium.googlesource.com/chromium/src/+/f05caaa1349e44abd44a00f88506bf140a0bc887

[dewittj, 2017-01-09 22:23:12]

The CQ bit was checked by dewittj@chromium.org

[dewittj, 2017-01-09 22:23:12]

Created Revert of Fix getUserMedia so that failure is reported for invalid audio
sources.

[commit-bot: I haz the power, 2017-01-09 22:24:23]

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 22:25:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484000592155030, "parent_rev":
"39a260f9aff94ecd6d518778e2ea900f58933636", "commit_rev":
"f05caaa1349e44abd44a00f88506bf140a0bc887"}

[commit-bot: I haz the power, 2017-01-09 22:26:25]

Description was changed from

==========
Revert of Fix getUserMedia so that failure is reported for invalid audio
sources. (patchset #2 id:20001 of https://codereview.chromium.org/2623443002/ )

Reason for revert:
Revert due to WebRTC tests failing:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

[4832:364:0109/131325.673:4351226:ERROR:browser_test_utils.cc(147)] Cannot
communicate with DOMMessageQueue.
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(66):
error: Value of: ExecuteScriptAndExtractString(shell(), javascript, &result)
  Actual: false
Expected: true
Failed to execute javascript callAndRenegotiateToVideo({audio: true}, {audio:
true, video:true});.
From javascript: (nothing)
When executing 'callAndRenegotiateToVideo({audio: true}, {audio: true,
video:true});'
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(90):
error: Failed

and

[4704:4156:0109/131352.840:4378402:ERROR:browser_test_utils.cc(147)] Cannot
communicate with DOMMessageQueue.
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(66):
error: Value of: ExecuteScriptAndExtractString(shell(), javascript, &result)
  Actual: false
Expected: true
Failed to execute javascript getUserMediaInIframeAndCloseInSuccessCb({audio:
true});.
From javascript: (nothing)
When executing 'getUserMediaInIframeAndCloseInSuccessCb({audio: true});'
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(90):
error: Failed

Original issue's description:
> Fix getUserMedia so that failure is reported for invalid audio sources.
> This changes getUserMedia to wait for initialization of local
> audio sources before issuing the completion callback (either success or
> failure).
> Previously, if an error occurs between attempting to start a local
> audio source and the render side OnStreamCreated callback, getUserMedia
> would report successful completion with an audio track but no audio
> would actually be delivered for that track.
> 
> BUG=679210
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Review-Url: https://codereview.chromium.org/2623443002
> Cr-Commit-Position: refs/heads/master@{#442323}
> Committed:
https://chromium.googlesource.com/chromium/src/+/81cb66acf3f8099e416c7fc5ecbc...

TBR=jochen@chromium.org,guidou@chromium.org,xhwang@chromium.org,tommi@chromiu...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679210
==========

to

==========
Revert of Fix getUserMedia so that failure is reported for invalid audio
sources. (patchset #2 id:20001 of https://codereview.chromium.org/2623443002/ )

Reason for revert:
Revert due to WebRTC tests failing:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win7%20Tests%20%...

[4832:364:0109/131325.673:4351226:ERROR:browser_test_utils.cc(147)] Cannot
communicate with DOMMessageQueue.
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(66):
error: Value of: ExecuteScriptAndExtractString(shell(), javascript, &result)
  Actual: false
Expected: true
Failed to execute javascript callAndRenegotiateToVideo({audio: true}, {audio:
true, video:true});.
From javascript: (nothing)
When executing 'callAndRenegotiateToVideo({audio: true}, {audio: true,
video:true});'
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(90):
error: Failed

and

[4704:4156:0109/131352.840:4378402:ERROR:browser_test_utils.cc(147)] Cannot
communicate with DOMMessageQueue.
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(66):
error: Value of: ExecuteScriptAndExtractString(shell(), javascript, &result)
  Actual: false
Expected: true
Failed to execute javascript getUserMediaInIframeAndCloseInSuccessCb({audio:
true});.
From javascript: (nothing)
When executing 'getUserMediaInIframeAndCloseInSuccessCb({audio: true});'
c:\c\win\src\contentrowser\webrtc\webrtc_content_browsertest_base.cc(90):
error: Failed

Original issue's description:
> Fix getUserMedia so that failure is reported for invalid audio sources.
> This changes getUserMedia to wait for initialization of local
> audio sources before issuing the completion callback (either success or
> failure).
> Previously, if an error occurs between attempting to start a local
> audio source and the render side OnStreamCreated callback, getUserMedia
> would report successful completion with an audio track but no audio
> would actually be delivered for that track.
> 
> BUG=679210
>
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Review-Url: https://codereview.chromium.org/2623443002
> Cr-Commit-Position: refs/heads/master@{#442323}
> Committed:
https://chromium.googlesource.com/chromium/src/+/81cb66acf3f8099e416c7fc5ecbc...

TBR=jochen@chromium.org,guidou@chromium.org,xhwang@chromium.org,tommi@chromiu...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=679210

Review-Url: https://codereview.chromium.org/2626533002
Cr-Commit-Position: refs/heads/master@{#442373}
Committed:
https://chromium.googlesource.com/chromium/src/+/f05caaa1349e44abd44a00f88506...
==========

[commit-bot: I haz the power, 2017-01-09 22:26:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/f05caaa1349e44abd44a00f88506...

[tommi (sloooow) - chröme, 2017-01-10 10:41:10]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2622643006/ by tommi@chromium.org.

The reason for reverting is: Preparing reland.