Remove unused SignalMediaError and infrastructure.

Remove unused SignalMediaError and infrastructure.

BUG=webrtc:4690
Committed: https://crrev.com/5b14b42e93f17d0ea57f1f8b3e8224082c514946
Cr-Commit-Position: refs/heads/master@{#10133}

[the sun, 2015-09-25 11:16:36]

solenberg@webrtc.org changed reviewers:
+ pthatcher@webrtc.org

[the sun, 2015-09-25 11:16:37]

Nobody is hooked up to listen to [Voice|Video|Data]Channel::SignalMediaError, so
they can be safely removed. 

A funky side effect is that SrtpFilter::SignalSrtpError is no longer attached.
Should I remove that signal too?

[pthatcher1, 2015-09-25 17:18:52]

I think we should leave the SignalSrtpError for now.  We might actually want
that around in the future.

https://codereview.webrtc.org/1362913004/diff/20001/talk/session/media/channe...
File talk/session/media/channel_unittest.cc (right):

https://codereview.webrtc.org/1362913004/diff/20001/talk/session/media/channe...
talk/session/media/channel_unittest.cc:1704: // TODO(solenberg): !!!!!
You're going to fix this right?

[the sun, 2015-09-28 10:34:23]

On 2015/09/25 17:18:52, pthatcher1 wrote:
> I think we should leave the SignalSrtpError for now.  We might actually want
> that around in the future.
> 
>
https://codereview.webrtc.org/1362913004/diff/20001/talk/session/media/channe...
> File talk/session/media/channel_unittest.cc (right):
> 
>
https://codereview.webrtc.org/1362913004/diff/20001/talk/session/media/channe...
> talk/session/media/channel_unittest.cc:1704: // TODO(solenberg): !!!!!
> You're going to fix this right?

Yes, and now it's done.

[pthatcher1, 2015-09-30 22:34:49]

lgtm

[the sun, 2015-10-01 08:58:37]

The CQ bit was checked by solenberg@webrtc.org

[the sun, 2015-10-01 08:58:37]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1362913004/#ps60001 (title:
"rebase")

[commit-bot: I haz the power, 2015-10-01 08:58:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1362913004/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1362913004/60001

[commit-bot: I haz the power, 2015-10-01 09:01:45]

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

[commit-bot: I haz the power, 2015-10-01 09:01:46]

Try jobs failed on following builders:
  ios_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_rel/builds/8442)

[the sun, 2015-10-01 09:34:45]

The CQ bit was checked by solenberg@webrtc.org

[the sun, 2015-10-01 09:34:46]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1362913004/#ps80001 (title:
"rebase")

[commit-bot: I haz the power, 2015-10-01 09:34:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1362913004/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1362913004/80001

[commit-bot: I haz the power, 2015-10-01 11:10:36]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-10-01 11:10:48]

Patchset 5 (id:??) landed as
https://crrev.com/5b14b42e93f17d0ea57f1f8b3e8224082c514946
Cr-Commit-Position: refs/heads/master@{#10133}