[CRD iOS] Don't attempt to connect to an offline host

[CRD iOS] Don't attempt to connect to an offline host

Previously when the user clicks on an offline host, the client will still
attempts to connect to it, then it crashes.

This CL prevents the client from attempting to connect to an offline host.
When the offline host is selected, the client will just show a snack bar
message and do nothing else.

Review-Url: https://codereview.chromium.org/2914233003
Cr-Commit-Position: refs/heads/master@{#476455}
Committed: https://chromium.googlesource.com/chromium/src/+/9c1f5b800d16648826954266f25767a23d75f276

[Yuwei, 2017-06-01 21:13:29]

yuweih@chromium.org changed reviewers:
+ nicholss@chromium.org

[Yuwei, 2017-06-01 21:13:29]

PTAL thanks!

[nicholss, 2017-06-01 21:56:57]

lgtm

https://codereview.chromium.org/2914233003/diff/1/remoting/ios/app/remoting_v...
File remoting/ios/app/remoting_view_controller.mm (right):

https://codereview.chromium.org/2914233003/diff/1/remoting/ios/app/remoting_v...
remoting/ios/app/remoting_view_controller.mm:208: message.text = @"Host is
offline.";
Might be overkill but for now yeah, lets use it.

[Yuwei, 2017-06-01 22:06:49]

Thanks!

https://codereview.chromium.org/2914233003/diff/1/remoting/ios/app/remoting_v...
File remoting/ios/app/remoting_view_controller.mm (right):

https://codereview.chromium.org/2914233003/diff/1/remoting/ios/app/remoting_v...
remoting/ios/app/remoting_view_controller.mm:208: message.text = @"Host is
offline.";
On 2017/06/01 21:56:57, nicholss wrote:
> Might be overkill but for now yeah, lets use it.

Okay. BTW this is what we do on Android #_#

[Yuwei, 2017-06-01 22:06:54]

The CQ bit was checked by yuweih@chromium.org

[commit-bot: I haz the power, 2017-06-01 22:07:55]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-01 22:23:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496354814945430, "parent_rev":
"3b3d3bc478458dfd88e632b11c3ec319df0a4a21", "commit_rev":
"9c1f5b800d16648826954266f25767a23d75f276"}

[commit-bot: I haz the power, 2017-06-01 22:23:20]

Description was changed from

==========
[CRD iOS] Don't attempt to connect to an offline host

Previously when the user clicks on an offline host, the client will still
attempts to connect to it, then it crashes.

This CL prevents the client from attempting to connect to an offline host.
When the offline host is selected, the client will just show a snack bar
message and do nothing else.
==========

to

==========
[CRD iOS] Don't attempt to connect to an offline host

Previously when the user clicks on an offline host, the client will still
attempts to connect to it, then it crashes.

This CL prevents the client from attempting to connect to an offline host.
When the offline host is selected, the client will just show a snack bar
message and do nothing else.

Review-Url: https://codereview.chromium.org/2914233003
Cr-Commit-Position: refs/heads/master@{#476455}
Committed:
https://chromium.googlesource.com/chromium/src/+/9c1f5b800d16648826954266f257...
==========

[commit-bot: I haz the power, 2017-06-01 22:23:21]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/9c1f5b800d16648826954266f257...