Revert of Adding error output param to SetConfiguration, using new RTCError type.

Revert of Adding error output param to SetConfiguration, using new RTCError type. (patchset #4 id:60001 of https://codereview.webrtc.org/2587133004/ )

Reason for revert:
Broke chromium FYI bot because the chromium mock PC overrides the method whose signature is changing.

Also broke a downstream internal test, which I need to investigate further.

Original issue's description:
> Adding error output param to SetConfiguration, using new RTCError type.
> 
> Most notably, will return "INVALID_MODIFICATION" if a field in the
> configuration was modified and modification of that field isn't supported.
> 
> Also changing RTCError to a class that wraps an enum type, because it will
> eventually need to hold other information (like SDP line number), to match
> the RTCError that was recently added to the spec:
> https://github.com/w3c/webrtc-pc/pull/850
> 
> BUG=webrtc:6916
> 
> Review-Url: https://codereview.webrtc.org/2587133004
> Cr-Commit-Position: refs/heads/master@{#15777}
> Committed: https://chromium.googlesource.com/external/webrtc/+/7a5fa6cd6173adbe32aedc1aedc872478121f5ed

TBR=pthatcher@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6916
Review-Url: https://codereview.webrtc.org/2600813002
Cr-Commit-Position: refs/heads/master@{#15778}
Committed: https://chromium.googlesource.com/external/webrtc/+/1e23461d5ed79b14d918933a0f969603285aad5f

[Taylor Brandstetter, 2016-12-24 09:43:23]

The CQ bit was checked by deadbeef@webrtc.org

[Taylor Brandstetter, 2016-12-24 09:43:23]

Created Revert of Adding error output param to SetConfiguration, using new
RTCError type.

[commit-bot: I haz the power, 2016-12-24 09:43:27]

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

[commit-bot: I haz the power, 2016-12-24 09:43:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482572603149370, "parent_rev":
"7a5fa6cd6173adbe32aedc1aedc872478121f5ed", "commit_rev":
"1e23461d5ed79b14d918933a0f969603285aad5f"}

[commit-bot: I haz the power, 2016-12-24 09:43:35]

Description was changed from

==========
Revert of Adding error output param to SetConfiguration, using new RTCError
type. (patchset #4 id:60001 of https://codereview.webrtc.org/2587133004/ )

Reason for revert:
Broke chromium FYI bot because the chromium mock PC overrides the method whose
signature is changing.

Also broke a downstream internal test, which I need to investigate further.

Original issue's description:
> Adding error output param to SetConfiguration, using new RTCError type.
> 
> Most notably, will return "INVALID_MODIFICATION" if a field in the
> configuration was modified and modification of that field isn't supported.
> 
> Also changing RTCError to a class that wraps an enum type, because it will
> eventually need to hold other information (like SDP line number), to match
> the RTCError that was recently added to the spec:
> https://github.com/w3c/webrtc-pc/pull/850
> 
> BUG=webrtc:6916
> 
> Review-Url: https://codereview.webrtc.org/2587133004
> Cr-Commit-Position: refs/heads/master@{#15777}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/7a5fa6cd6173adbe32aedc1ae...

TBR=pthatcher@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6916
==========

to

==========
Revert of Adding error output param to SetConfiguration, using new RTCError
type. (patchset #4 id:60001 of https://codereview.webrtc.org/2587133004/ )

Reason for revert:
Broke chromium FYI bot because the chromium mock PC overrides the method whose
signature is changing.

Also broke a downstream internal test, which I need to investigate further.

Original issue's description:
> Adding error output param to SetConfiguration, using new RTCError type.
> 
> Most notably, will return "INVALID_MODIFICATION" if a field in the
> configuration was modified and modification of that field isn't supported.
> 
> Also changing RTCError to a class that wraps an enum type, because it will
> eventually need to hold other information (like SDP line number), to match
> the RTCError that was recently added to the spec:
> https://github.com/w3c/webrtc-pc/pull/850
> 
> BUG=webrtc:6916
> 
> Review-Url: https://codereview.webrtc.org/2587133004
> Cr-Commit-Position: refs/heads/master@{#15777}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/7a5fa6cd6173adbe32aedc1ae...

TBR=pthatcher@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6916

Review-Url: https://codereview.webrtc.org/2600813002
Cr-Commit-Position: refs/heads/master@{#15778}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1e23461d5ed79b14d918933a0...
==========

[commit-bot: I haz the power, 2016-12-24 09:43:36]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/1e23461d5ed79b14d918933a0...