Delete class MockCongestionController.

Delete class MockCongestionController.

It became unused in cl https://codereview.webrtc.org/2516983004.

BUG=None

Review-Url: https://codereview.webrtc.org/2762023004
Cr-Commit-Position: refs/heads/master@{#17325}
Committed: https://chromium.googlesource.com/external/webrtc/+/d19bcb7116d527b5d0b7426468490e72b03358ca

[nisse-webrtc, 2017-03-21 12:23:18]

The CQ bit was checked by nisse@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-21 12:23:24]

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

[nisse-webrtc, 2017-03-21 12:23:36]

nisse@webrtc.org changed reviewers:
+ stefan@webrtc.org

[nisse-webrtc, 2017-03-21 12:23:36]

PTAL

[commit-bot: I haz the power, 2017-03-21 12:54:13]

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

[commit-bot: I haz the power, 2017-03-21 12:54:13]

Dry run: This issue passed the CQ dry run.

[stefan-webrtc, 2017-03-21 15:28:57]

lgtm

https://codereview.webrtc.org/2762023004/diff/20001/webrtc/modules/congestion...
File
webrtc/modules/congestion_controller/include/mock/mock_congestion_controller.h
(right):

https://codereview.webrtc.org/2762023004/diff/20001/webrtc/modules/congestion...
webrtc/modules/congestion_controller/include/mock/mock_congestion_controller.h:12:
#define
WEBRTC_MODULES_CONGESTION_CONTROLLER_INCLUDE_MOCK_MOCK_CONGESTION_CONTROLLER_H_
Maybe rename the file to mock_congestion_observer.h?

[nisse-webrtc, 2017-03-21 15:31:45]

https://codereview.webrtc.org/2762023004/diff/20001/webrtc/modules/congestion...
File
webrtc/modules/congestion_controller/include/mock/mock_congestion_controller.h
(right):

https://codereview.webrtc.org/2762023004/diff/20001/webrtc/modules/congestion...
webrtc/modules/congestion_controller/include/mock/mock_congestion_controller.h:12:
#define
WEBRTC_MODULES_CONGESTION_CONTROLLER_INCLUDE_MOCK_MOCK_CONGESTION_CONTROLLER_H_
On 2017/03/21 15:28:57, stefan-webrtc wrote:
> Maybe rename the file to mock_congestion_observer.h?

Hmm. It's used in only one place, webrtc/audio/audio_send_stream_unittest.cc.

So maybe even better to move the definition there, and delete this file.

[nisse-webrtc, 2017-03-21 15:36:53]

On 2017/03/21 15:31:45, nisse-webrtc wrote:
>
https://codereview.webrtc.org/2762023004/diff/20001/webrtc/modules/congestion...
> File
> webrtc/modules/congestion_controller/include/mock/mock_congestion_controller.h
> (right):
> 
>
https://codereview.webrtc.org/2762023004/diff/20001/webrtc/modules/congestion...
>
webrtc/modules/congestion_controller/include/mock/mock_congestion_controller.h:12:
> #define
>
WEBRTC_MODULES_CONGESTION_CONTROLLER_INCLUDE_MOCK_MOCK_CONGESTION_CONTROLLER_H_
> On 2017/03/21 15:28:57, stefan-webrtc wrote:
> > Maybe rename the file to mock_congestion_observer.h?
> 
> Hmm. It's used in only one place, webrtc/audio/audio_send_stream_unittest.cc.
> 
> So maybe even better to move the definition there, and delete this file.

Ooops, it's used in one more place. I'll rename the file. 

It's also not mentioned in any BUILD.gn file, I'll see if I can guess an
appropriate target.

[nisse-webrtc, 2017-03-21 15:47:31]

Renaming done.

Finding a suitable gn target was more difficult than I expected, so not doing
that now.

[nisse-webrtc, 2017-03-21 15:48:00]

The CQ bit was checked by nisse@webrtc.org

[nisse-webrtc, 2017-03-21 15:48:00]

The patchset sent to the CQ was uploaded after l-g-t-m from stefan@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2762023004/#ps40001 (title:
"Renamed mock_congestion_controller.h --> mock_congestion_observer.h.")

[commit-bot: I haz the power, 2017-03-21 15:48:04]

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, 2017-03-21 15:49:33]

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

[commit-bot: I haz the power, 2017-03-21 15:49:34]

Try jobs failed on following builders:
  ios64_sim_ios10_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios64_sim_ios10_dbg/bui...)
  ios_arm64_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_arm64_dbg/builds/18789)
  ios_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/ios_rel/builds/22870)

[nisse-webrtc, 2017-03-21 15:52:15]

The CQ bit was checked by nisse@webrtc.org

[nisse-webrtc, 2017-03-21 15:52:16]

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

[commit-bot: I haz the power, 2017-03-21 15:52:24]

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, 2017-03-21 16:41:07]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490111535901710,
"parent_rev": "1182afedecc572718af4e68ba7cd594dc771456c", "commit_rev":
"d19bcb7116d527b5d0b7426468490e72b03358ca"}

[commit-bot: I haz the power, 2017-03-21 16:41:10]

Description was changed from

==========
Delete class MockCongestionController.

It became unused in cl https://codereview.webrtc.org/2516983004.

BUG=None
==========

to

==========
Delete class MockCongestionController.

It became unused in cl https://codereview.webrtc.org/2516983004.

BUG=None

Review-Url: https://codereview.webrtc.org/2762023004
Cr-Commit-Position: refs/heads/master@{#17325}
Committed:
https://chromium.googlesource.com/external/webrtc/+/d19bcb7116d527b5d0b742646...
==========

[commit-bot: I haz the power, 2017-03-21 16:41:13]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/external/webrtc/+/d19bcb7116d527b5d0b742646...

[skvlad, 2017-03-21 20:15:06]

On 2017/03/21 16:41:13, commit-bot: I haz the power wrote:
> Committed patchset #4 (id:60001) as
>
https://chromium.googlesource.com/external/webrtc/+/d19bcb7116d527b5d0b742646...

This is breaking downstream build. Reverting until a fix lands.

[skvlad, 2017-03-21 20:15:47]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.webrtc.org/2762133003/ by skvlad@webrtc.org.

The reason for reverting is: This is breaking downstream build..