Provide HostView with data required for creating a new and session, and render for CRD iOS.

remoting/client/ios/app/remoting_view_controller.mm

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 #import "remoting/client/ios/app/remoting_view_controller.h"
 
+#import "base/mac/bind_objc_block.h"
 #import "ios/third_party/material_components_ios/src/components/AnimationTiming/src/MaterialAnimationTiming.h"
 #import "ios/third_party/material_components_ios/src/components/AppBar/src/MaterialAppBar.h"
 #import "ios/third_party/material_components_ios/src/components/Dialogs/src/MaterialDialogs.h"
 #import "ios/third_party/material_components_ios/src/components/Snackbar/src/MaterialSnackbar.h"
 #import "remoting/client/ios/app/host_collection_view_controller.h"
 #import "remoting/client/ios/app/host_view_controller.h"
 #import "remoting/client/ios/app/pin_entry_view_controller.h"
 #import "remoting/client/ios/app/remoting_settings_view_controller.h"
 #import "remoting/client/ios/facade/remoting_service.h"
+#import "remoting/client/ios/session/remoting_client.h"
+
+#include "base/strings/sys_string_conversions.h"
+#include "remoting/base/oauth_token_getter.h"
+#include "remoting/client/connect_to_host_info.h"
 
 static CGFloat kHostInset = 5.f;
 
 @interface RemotingViewController ()<HostCollectionViewControllerDelegate,
                                      UIViewControllerAnimatedTransitioning,
                                      UIViewControllerTransitioningDelegate> {
   bool _isAuthenticated;
   MDCDialogTransitionController* _dialogTransitionController;
   MDCAppBar* _appBar;
   HostCollectionViewController* _collectionViewController;
@@ skipping 55 matching lines @@
 - (void)viewDidLoad {
   [super viewDidLoad];
   [_appBar addSubviewsToParent];
   [self presentStatus];
 }
 
 - (void)viewDidAppear:(BOOL)animated {
   if (!_isAuthenticated) {
     // TODO(nicholss): This is used as a demo of the app functionality for the
     // moment but the real app will force the login flow if unauthenticated.
-    //[self didSelectSettings];
-    [self didSelectRefresh];
+    [self didSelectSettings];
+    // [self didSelectRefresh];
     MDCSnackbarMessage* message = [[MDCSnackbarMessage alloc] init];
     message.text = @"Please login.";
     [MDCSnackbarManager showMessage:message];
   }
 }
 
 #pragma mark - RemotingAuthenticationDelegate
 
 - (void)nowAuthenticated:(BOOL)authenticated {
   if (authenticated) {
@@ skipping 16 matching lines @@
 // TODO(nicholss): these need to be a stats change like "none, loading,
 // updated"...
 - (void)hostListUpdated {
   [_collectionViewController.collectionView reloadData];
 }
 
 #pragma mark - HostCollectionViewControllerDelegate
 
 - (void)didSelectCell:(HostCollectionViewCell*)cell
            completion:(void (^)())completionBlock {
-  HostViewController* hostVC = [HostViewController new];
-  [self presentViewController:hostVC animated:YES completion:nil];
+  RemotingClient* client = [[RemotingClient alloc] init];
+
+  [_remotingService
+      callbackWithAccessToken:base::BindBlockArc(^(

[Jamie, 2017/04/24 21:54:11]

Can you pull this block out into a variable? It's hard to read like this,
particularly given ObjC's strange function call syntax.

[nicholss, 2017/04/24 22:03:07]

The blind blocks are suppose to be used inline like this, it uses cell from the
parent method context. I don't think it would help readability to define it and
then use it rather than just use it in place.

[Jamie, 2017/04/24 22:07:42]

One advantage is that the name of the variable makes it clear what the callback
is supposed to do, but I don't feel strongly about it.

+                                  remoting::OAuthTokenGetter::Status status,
+                                  const std::string& user_email,
+                                  const std::string& access_token) {
+        // TODO(nicholss): Check status.
+        HostInfo* hostInfo = cell.hostInfo;
+        DCHECK(hostInfo);
+        DCHECK(hostInfo.jabberId);
+        DCHECK(hostInfo.hostId);
+        DCHECK(hostInfo.publicKey);
+
+        remoting::ConnectToHostInfo info;
+        info.username = user_email;
+        info.auth_token = access_token;
+        info.host_jid = base::SysNSStringToUTF8(hostInfo.jabberId);
+        info.host_id = base::SysNSStringToUTF8(hostInfo.hostId);
+        info.host_pubkey = base::SysNSStringToUTF8(hostInfo.publicKey);
+        // TODO(nicholss): If iOS supports pairing, pull the stored data and
+        // insert it here.
+        info.pairing_id = "";
+        info.pairing_secret = "";
+
+        // TODO(nicholss): I am not sure about the following fields yet.
+        // info.capabilities =
+        // info.flags =
+        // info.host_version =
+        // info.host_os =
+        // info.host_os_version =
+        [client connectToHost:info];
+      })];
+
+  HostViewController* hostViewController =
+      [[HostViewController alloc] initWithClient:client];
+
+  // TODO(nicholss): Add feedback on status of request.
+  [self presentViewController:hostViewController animated:YES completion:nil];
+
+  completionBlock();
 }
 
 - (NSInteger)getHostCount {
   NSArray<HostInfo*>* hosts = [_remotingService getHosts];
   return [hosts count];
 }
 
 - (HostInfo*)getHostAtIndexPath:(NSIndexPath*)path {
   NSArray<HostInfo*>* hosts = [_remotingService getHosts];
   return hosts[path.row];
@@ skipping 52 matching lines @@
   MDCSnackbarMessage* message = [[MDCSnackbarMessage alloc] init];
   if (_isAuthenticated) {
     UserInfo* user = [_remotingService getUser];
     message.text = [NSString
         stringWithFormat:@"Currently signed in as %@.", [user userEmail]];
     [MDCSnackbarManager showMessage:message];
   }
 }
 
 @end