Added audioSessionDidActivate and audioSessionDidDectivate

Added audioSessionDidActivate and audioSessionDidDectivate so that RTCAudioSession can be notified when the audio session is activated or deactivated by iOS outside of the app.

BUG=webrtc:7446
Review-Url: https://codereview.webrtc.org/2813493004
Cr-Commit-Position: refs/heads/master@{#17743}
Committed: https://chromium.googlesource.com/external/webrtc/+/3c9a6c0aff7b7d154b04390961072966252139d0

[jtt_webrtc, 2017-04-13 20:08:18]

Somehow the CL changed. Here is the up to date one.

[jtt_webrtc, 2017-04-13 20:09:33]

jtteh@webrtc.org changed reviewers:
+ tkchin@webrtc.org

[jtt_webrtc, 2017-04-13 20:09:33]

resending

[jtt_webrtc, 2017-04-13 20:12:11]

Description was changed from

==========
Don't call unconfigureWebRTCSession if setConfiguration fails. webrtc:7471

BUG=webrtc:7471
==========

to

==========
Added audioSessionDidActivate and audioSessionDidDectivate so that
RTCAudioSession can be notified when the audio session is activated or
deactivated by iOS outside of the app.

BUG=webrtc:7446
==========

[tkchin_webrtc, 2017-04-13 20:17:30]

lgtm

https://codereview.webrtc.org/2813493004/diff/100001/webrtc/sdk/objc/Framewor...
File webrtc/sdk/objc/Framework/UnitTests/RTCAudioSessionTest.mm (right):

https://codereview.webrtc.org/2813493004/diff/100001/webrtc/sdk/objc/Framewor...
webrtc/sdk/objc/Framework/UnitTests/RTCAudioSessionTest.mm:27: 
it'd be nice to have a unit test here that captures the previous scenario in
which we ended up with a negative count

[jtt_webrtc, 2017-04-13 20:27:58]

The CQ bit was checked by jtteh@webrtc.org

[commit-bot: I haz the power, 2017-04-13 20:28:09]

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-04-13 20:31:38]

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

[commit-bot: I haz the power, 2017-04-13 20:31:39]

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

[jtt_webrtc, 2017-04-13 20:48:07]

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

[commit-bot: I haz the power, 2017-04-13 20:48:17]

Dry run: 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-04-13 20:59:47]

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

[commit-bot: I haz the power, 2017-04-13 20:59:47]

Dry run: Try jobs failed on following builders:
  win_x64_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_rel/builds/24706)

[jtt_webrtc, 2017-04-13 21:00:06]

Had to move the audio session test to iOS only in BUILD.gn

[jtt_webrtc, 2017-04-13 21:35:55]

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

[commit-bot: I haz the power, 2017-04-13 21:35:57]

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

[jtt_webrtc, 2017-04-13 21:36:25]

Moved tests to RTCAudioSessionTest.mm.

[commit-bot: I haz the power, 2017-04-13 21:53:07]

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

[commit-bot: I haz the power, 2017-04-13 21:53:09]

Dry run: Try jobs failed on following builders:
  win_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_rel/builds/25290)

[jtt_webrtc, 2017-04-17 18:07:31]

Bump. Need this to be checked in before I can cl the fix for
configureWebRTCSession

[henrika_webrtc, 2017-04-18 08:19:04]

lgtm. Thanks for improvements.

[jtt_webrtc, 2017-04-18 15:46:16]

The CQ bit was checked by jtteh@webrtc.org

[jtt_webrtc, 2017-04-18 15:46:17]

The patchset sent to the CQ was uploaded after l-g-t-m from tkchin@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2813493004/#ps160001 (title:
"remove code not for this cl")

[commit-bot: I haz the power, 2017-04-18 15:46:26]

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-04-18 16:09:36]

CQ is committing da patch.

Bot data: {"patchset_id": 160001, "attempt_start_ts": 1492530376780730,
"parent_rev": "0d335c7756e172c38f5a86287aec7b62ff288be8", "commit_rev":
"3c9a6c0aff7b7d154b04390961072966252139d0"}

[commit-bot: I haz the power, 2017-04-18 16:09:40]

Description was changed from

==========
Added audioSessionDidActivate and audioSessionDidDectivate so that
RTCAudioSession can be notified when the audio session is activated or
deactivated by iOS outside of the app.

BUG=webrtc:7446
==========

to

==========
Added audioSessionDidActivate and audioSessionDidDectivate so that
RTCAudioSession can be notified when the audio session is activated or
deactivated by iOS outside of the app.

BUG=webrtc:7446

Review-Url: https://codereview.webrtc.org/2813493004
Cr-Commit-Position: refs/heads/master@{#17743}
Committed:
https://chromium.googlesource.com/external/webrtc/+/3c9a6c0aff7b7d154b0439096...
==========

[commit-bot: I haz the power, 2017-04-18 16:09:42]

Committed patchset #9 (id:160001) as
https://chromium.googlesource.com/external/webrtc/+/3c9a6c0aff7b7d154b0439096...