[CRD iOS] Add the host setup instruction page

[CRD iOS] Add the host setup instruction page

This CL adds the host setup view to inform the user to setup the host when
there is no host in the list. Please see this screenshot:
https://drive.google.com/file/d/0BytzIZKeM8nBdzBoWlN3R3hGSWM/view?usp=sharing

This CL also fixes some color scheme issue. Previously the host collection list
is using opacity to lighten the color.

BUG=736932
Review-Url: https://codereview.chromium.org/2957943002
Cr-Commit-Position: refs/heads/master@{#482832}
Committed: https://chromium.googlesource.com/chromium/src/+/e3872416cbc8896a54bb571539202bf0e24fcac2

[Yuwei, 2017-06-26 23:31:12]

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

[Yuwei, 2017-06-26 23:31:13]

PTAL thanks!

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_c...
File remoting/ios/app/host_collection_view_controller.mm (left):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_c...
remoting/ios/app/host_collection_view_controller.mm:144: MDCShadowLayer*
shadowLayer = [MDCShadowLayer layer];
This logic has been moved to RemotingViewController to be shared with
HostSetupVC

[nicholss, 2017-06-27 16:36:02]

lgtm, some minor suggestions on cleanup

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
File remoting/ios/app/host_setup_footer_view.mm (right):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
remoting/ios/app/host_setup_footer_view.mm:33: [raisedButton
setTitleColor:UIColor.whiteColor forState:UIControlStateNormal];
whiteColor should come from RemotingTheme

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
remoting/ios/app/host_setup_footer_view.mm:48: NSLog(@"Tapped email
instructions.");
Here you can do what I did in the settings menu and delegate back to the app
delegate to do the email work and then you can add the TODO in the app delegate.

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
File remoting/ios/app/host_setup_view_cell.mm (right):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
remoting/ios/app/host_setup_view_cell.mm:49: _numberLabel.textColor =
UIColor.whiteColor;
Colors should come from RemotingTheme too

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
File remoting/ios/app/host_setup_view_controller.mm (right):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
remoting/ios/app/host_setup_view_controller.mm:35: @"complete the setup",
you do not need to add the extra @ to the second line of the string btw, without
it it helps to make it look like a single @"" vs @"", @""

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/remoti...
File remoting/ios/app/remoting_theme.mm (right):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/remoti...
remoting/ios/app/remoting_theme.mm:58: color = [UIColor colorWithRed:0.40f
green:0.75f blue:0.40f alpha:1.f];
Oh good, you got rid of that horrible green color :)

[Yuwei, 2017-06-27 20:46:46]

Thanks!

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
File remoting/ios/app/host_setup_footer_view.mm (right):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
remoting/ios/app/host_setup_footer_view.mm:33: [raisedButton
setTitleColor:UIColor.whiteColor forState:UIControlStateNormal];
On 2017/06/27 16:36:01, nicholss wrote:
> whiteColor should come from RemotingTheme

Done.

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
remoting/ios/app/host_setup_footer_view.mm:48: NSLog(@"Tapped email
instructions.");
On 2017/06/27 16:36:01, nicholss wrote:
> Here you can do what I did in the settings menu and delegate back to the app
> delegate to do the email work and then you can add the TODO in the app
delegate.

Done.

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
File remoting/ios/app/host_setup_view_cell.mm (right):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
remoting/ios/app/host_setup_view_cell.mm:49: _numberLabel.textColor =
UIColor.whiteColor;
On 2017/06/27 16:36:02, nicholss wrote:
> Colors should come from RemotingTheme too

Done.

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
File remoting/ios/app/host_setup_view_controller.mm (right):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/host_s...
remoting/ios/app/host_setup_view_controller.mm:35: @"complete the setup",
On 2017/06/27 16:36:02, nicholss wrote:
> you do not need to add the extra @ to the second line of the string btw,
without
> it it helps to make it look like a single @"" vs @"", @""

Done. Didn't know that you can drop the second @...

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/remoti...
File remoting/ios/app/remoting_theme.mm (right):

https://codereview.chromium.org/2957943002/diff/20001/remoting/ios/app/remoti...
remoting/ios/app/remoting_theme.mm:58: color = [UIColor colorWithRed:0.40f
green:0.75f blue:0.40f alpha:1.f];
On 2017/06/27 16:36:02, nicholss wrote:
> Oh good, you got rid of that horrible green color :)

This is basically to get rid of the opacity setting. Not sure whether it still
looks horrible though X_X

[Yuwei, 2017-06-27 20:46:50]

The CQ bit was checked by yuweih@chromium.org

[Yuwei, 2017-06-27 20:46:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
nicholss@chromium.org
Link to the patchset: https://codereview.chromium.org/2957943002/#ps40001
(title: "Resolve Feedback")

[commit-bot: I haz the power, 2017-06-27 20:47:12]

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-27 22:26:44]

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

[commit-bot: I haz the power, 2017-06-27 22:26:45]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Yuwei, 2017-06-27 22:30:14]

The CQ bit was checked by yuweih@chromium.org

[commit-bot: I haz the power, 2017-06-27 22:30:39]

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-28 01:03:40]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1498602614334090,
"parent_rev": "98cf368ffef110f3a00dabe56289cee021b72c7e", "commit_rev":
"e3872416cbc8896a54bb571539202bf0e24fcac2"}

[commit-bot: I haz the power, 2017-06-28 01:03:57]

Description was changed from

==========
[CRD iOS] Add the host setup instruction page

This CL adds the host setup view to inform the user to setup the host when
there is no host in the list. Please see this screenshot:
https://drive.google.com/file/d/0BytzIZKeM8nBdzBoWlN3R3hGSWM/view?usp=sharing

This CL also fixes some color scheme issue. Previously the host collection list
is using opacity to lighten the color.

BUG=736932
==========

to

==========
[CRD iOS] Add the host setup instruction page

This CL adds the host setup view to inform the user to setup the host when
there is no host in the list. Please see this screenshot:
https://drive.google.com/file/d/0BytzIZKeM8nBdzBoWlN3R3hGSWM/view?usp=sharing

This CL also fixes some color scheme issue. Previously the host collection list
is using opacity to lighten the color.

BUG=736932

Review-Url: https://codereview.chromium.org/2957943002
Cr-Commit-Position: refs/heads/master@{#482832}
Committed:
https://chromium.googlesource.com/chromium/src/+/e3872416cbc8896a54bb57153920...
==========

[commit-bot: I haz the power, 2017-06-28 01:03:58]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/e3872416cbc8896a54bb57153920...