[CRD iOS] Use UINavigationController for view stack manipulation

[CRD iOS] Use UINavigationController for view stack manipulation

Previously we use modal popup for the host connection flow and we experienced
UI churn when we dismiss the ClientConnectionView and open the HostView. The
host list will flash onto the screen.

This CL introduces a UINavigationController on the root level, which allows us
to:

* Replace the topmost view on the stack within one frame; no churn caused by
  the host list being rendered intermediately.
* Get nice right-to-left and left-to-right transition effect for the host
  connection flow.

Other calls to presentViewController are not affected.

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

[Yuwei, 2017-06-05 20:44:23]

Description was changed from

==========
[CRD iOS] Use UINavigationController for history manipulation
==========

to

==========
[CRD iOS] Use UINavigationController for view stack manipulation

Previously we use modal popup for the host connection flow and we experienced
UI churn when we dismiss the ClientConnectionView and open the HostView. The
host list will flash onto the screen.

This CL introduces a UINavigationController on the root level, which allows us
to:

* Replace the topmost view on the stack within one frame; no churn caused by
  the host list being rendered intermediately.
* Get nice right-to-left and left-to-right transition effect for the host
  connection flow.
==========

[Yuwei, 2017-06-05 20:45:03]

Description was changed from

==========
[CRD iOS] Use UINavigationController for view stack manipulation

Previously we use modal popup for the host connection flow and we experienced
UI churn when we dismiss the ClientConnectionView and open the HostView. The
host list will flash onto the screen.

This CL introduces a UINavigationController on the root level, which allows us
to:

* Replace the topmost view on the stack within one frame; no churn caused by
  the host list being rendered intermediately.
* Get nice right-to-left and left-to-right transition effect for the host
  connection flow.
==========

to

==========
[CRD iOS] Use UINavigationController for view stack manipulation

Previously we use modal popup for the host connection flow and we experienced
UI churn when we dismiss the ClientConnectionView and open the HostView. The
host list will flash onto the screen.

This CL introduces a UINavigationController on the root level, which allows us
to:

* Replace the topmost view on the stack within one frame; no churn caused by
  the host list being rendered intermediately.
* Get nice right-to-left and left-to-right transition effect for the host
  connection flow.

Other calls to presentViewController are not affected.
==========

[Yuwei, 2017-06-05 20:52:17]

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

[Yuwei, 2017-06-05 20:52:18]

PTAL thanks!

Note that this doesn't fix the green flash. It was caused by the delay in
receiving the desktop frame and it's not trivial to fix. In official build that
color will become black and might mitigate the problem.

[nicholss, 2017-06-05 21:34:04]

lgtm

Nice work, the demo looked nice.

[Yuwei, 2017-06-05 21:35:37]

Thanks! Checking-in :)

[Yuwei, 2017-06-05 21:35:45]

The CQ bit was checked by yuweih@chromium.org

[commit-bot: I haz the power, 2017-06-05 21:36:16]

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-05 22:10:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496698545448840,
"parent_rev": "593f82d4b2351c8f7a3eb4081839f8171ed04b36", "commit_rev":
"9acd0160c7449a806fa3a51bed5d2fb7aa9b350f"}

[commit-bot: I haz the power, 2017-06-05 22:10:38]

Description was changed from

==========
[CRD iOS] Use UINavigationController for view stack manipulation

Previously we use modal popup for the host connection flow and we experienced
UI churn when we dismiss the ClientConnectionView and open the HostView. The
host list will flash onto the screen.

This CL introduces a UINavigationController on the root level, which allows us
to:

* Replace the topmost view on the stack within one frame; no churn caused by
  the host list being rendered intermediately.
* Get nice right-to-left and left-to-right transition effect for the host
  connection flow.

Other calls to presentViewController are not affected.
==========

to

==========
[CRD iOS] Use UINavigationController for view stack manipulation

Previously we use modal popup for the host connection flow and we experienced
UI churn when we dismiss the ClientConnectionView and open the HostView. The
host list will flash onto the screen.

This CL introduces a UINavigationController on the root level, which allows us
to:

* Replace the topmost view on the stack within one frame; no churn caused by
  the host list being rendered intermediately.
* Get nice right-to-left and left-to-right transition effect for the host
  connection flow.

Other calls to presentViewController are not affected.

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

[commit-bot: I haz the power, 2017-06-05 22:10:39]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/9acd0160c7449a806fa3a51bed5d...