iOS stability improvement for device switching

iOS stability improvement for device switching, including BT devices

BUG=webrtc:5058
Committed: https://crrev.com/30166cb1a8d7ca2d67981927d056e60aa58cb1ae
Cr-Commit-Position: refs/heads/master@{#11168}

[henrika_webrtc, 2016-01-04 15:49:21]

Description was changed from

==========
iOS stability improvement for device switching

BUG=
==========

to

==========
iOS stability improvement for device switching, including BT devices

BUG=webrtc:5058
==========

[henrika_webrtc, 2016-01-04 15:58:50]

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

[henrika_webrtc, 2016-01-04 15:58:51]

Adding new modifications which ensures that we can perform device switching
between different types of devices (BT, speaker, wired headset) on latest iOS
9.2.

https://codereview.webrtc.org/1554163002/diff/1/webrtc/examples/objc/AppRTCDe...
File webrtc/examples/objc/AppRTCDemo/ios/ARDMainViewController.m (right):

https://codereview.webrtc.org/1554163002/diff/1/webrtc/examples/objc/AppRTCDe...
webrtc/examples/objc/AppRTCDemo/ios/ARDMainViewController.m:30: // [self
dismissViewControllerAnimated:NO completion:nil];
This part will not be part of the final CL, hence not checked in.

[henrika_webrtc, 2016-01-04 16:03:12]

Zeke, do you mind checking this CL? I will run tests on more devices and using
additional BT headsets but its seems to work well as is.

I wanted to ensure that basic device switching works before we start doing more
work on your larger CL. To be able to understand if possible degradations comes
from your change or not.

The main reasons why these changes are needed seems to relate to a slightly
different "style" in how device switching is reported in iOS 9.2. I can't say
for sure exactly when this change was introduced but all work I have done here
has been on iOS 9.2 iPhone 6S and 6.

[tkchin_webrtc, 2016-01-04 17:17:11]

On 2016/01/04 16:03:12, henrika_webrtc wrote:
> Zeke, do you mind checking this CL? I will run tests on more devices and using
> additional BT headsets but its seems to work well as is.
> 
> I wanted to ensure that basic device switching works before we start doing
more
> work on your larger CL. To be able to understand if possible degradations
comes
> from your change or not.
> 
> The main reasons why these changes are needed seems to relate to a slightly
> different "style" in how device switching is reported in iOS 9.2. I can't say
> for sure exactly when this change was introduced but all work I have done here
> has been on iOS 9.2 iPhone 6S and 6.

Seems easy enough to merge this into my change, so lgtm.

[henrika_webrtc, 2016-01-05 08:16:00]

Thanks, just want to run some more tests first. Expect this one to land on
Thursday (due to two holidays) if nothing strange is found during testing.

[henrika_webrtc, 2016-01-07 13:44:05]

Tried more devices and it seems to work well so far. Landing as is.

[henrika_webrtc, 2016-01-07 13:44:09]

The CQ bit was checked by henrika@webrtc.org

[henrika_webrtc, 2016-01-07 13:44:09]

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/1554163002/#ps40001 (title:
"nit")

[commit-bot: I haz the power, 2016-01-07 13:44:14]

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

[commit-bot: I haz the power, 2016-01-07 15:23:04]

Description was changed from

==========
iOS stability improvement for device switching, including BT devices

BUG=webrtc:5058
==========

to

==========
iOS stability improvement for device switching, including BT devices

BUG=webrtc:5058
==========

[commit-bot: I haz the power, 2016-01-07 15:23:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-07 15:23:16]

Description was changed from

==========
iOS stability improvement for device switching, including BT devices

BUG=webrtc:5058
==========

to

==========
iOS stability improvement for device switching, including BT devices

BUG=webrtc:5058

Committed: https://crrev.com/30166cb1a8d7ca2d67981927d056e60aa58cb1ae
Cr-Commit-Position: refs/heads/master@{#11168}
==========

[commit-bot: I haz the power, 2016-01-07 15:23:16]

Patchset 3 (id:??) landed as
https://crrev.com/30166cb1a8d7ca2d67981927d056e60aa58cb1ae
Cr-Commit-Position: refs/heads/master@{#11168}