[Remoting iOS] Basic viewport manipulation support

remoting/client/ios/app/host_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/host_view_controller.h"
 
+#include <memory>
+
 #import <GLKit/GLKit.h>
 
+#include "remoting/client/gesture_interpreter.h"
+
 #import "ios/third_party/material_components_ios/src/components/Buttons/src/MaterialButtons.h"
-#import "remoting/client/ios/display/gl_display_handler.h"
+#import "remoting/client/ios/client_gestures.h"
 #import "remoting/client/ios/session/remoting_client.h"
 
 static const CGFloat kFabInset = 15.f;
 
 @interface HostViewController () {
   RemotingClient* _client;
   MDCFloatingButton* _floatingButton;
+
+  ClientGestures* _clientGestures;
 }
 @end
 
 @implementation HostViewController
 
 - (id)initWithClient:(RemotingClient*)client {
   self = [super init];
   if (self) {
     _client = client;
   }
@@ skipping 37 matching lines @@
   [_client.displayHandler onSurfaceCreated:glView];
 
   // viewDidLayoutSubviews may be called before viewDidAppear, in which case
   // the surface is not ready and onSurfaceChanged will be no-op.
   // Call onSurfaceChanged here to cover that case.
   [_client.displayHandler onSurfaceChanged:glView.frame];
 }
 
 - (void)viewWillAppear:(BOOL)animated {
   [super viewWillAppear:animated];
-  ((GLKView*)self.view).enableSetNeedsDisplay = true;
+
+  _client.displayHandler.delegate = self;
+
+  _clientGestures =
+      [[ClientGestures alloc] initWithView:self.view client:_client];
 }
 
 - (void)viewWillDisappear:(BOOL)animated {
   [super viewWillDisappear:animated];
+
+  _clientGestures = nil;
 }
 
 - (void)viewDidLayoutSubviews {
   [super viewDidLayoutSubviews];
 
-  [_client.displayHandler onSurfaceChanged:((GLKView*)self.view).frame];
+  CGRect surfaceFrame = self.view.frame;
+  [_client.displayHandler onSurfaceChanged:surfaceFrame];

[nicholss, 2017/05/03 22:47:35]

Maybe we could tell the client that the surface changed and then the client can
tell the display handler and the gesture interpreter  the frame has changed size
rather than having the host view controller do this.

[Yuwei, 2017/05/04 00:28:07]

Done.

+  _client.gestureInterpreter->OnSurfaceSizeChanged(surfaceFrame.size.width,
+                                                   surfaceFrame.size.height);
 
-  const CGSize& btnSize = _floatingButton.frame.size;
+  CGSize btnSize = _floatingButton.frame.size;
   _floatingButton.frame =
       CGRectMake(self.view.frame.size.width - btnSize.width - kFabInset,
                  self.view.frame.size.height - btnSize.height - kFabInset,
                  btnSize.width, btnSize.height);
 }
 
+#pragma mark - GlDisplayHandlerDelegate
+
+- (void)canvasSizeChanged:(CGSize)size {
+  _client.gestureInterpreter->OnDesktopSizeChanged(size.width, size.height);

[nicholss, 2017/05/03 22:47:35]

Do we still need the host view controller to tell the gesture interpreter this?
Seems like the _client can do it instead.

[Yuwei, 2017/05/04 00:28:07]

Nope. Move delegate to RemotingClient instead.

+}
+
 #pragma mark - Private
 
 - (void)didTap:(id)sender {
   // TODO(nicholss): The FAB is being used to close the window at the moment
   // just as a demo  as the integration continues. This will not be the case
   // in the final app.
   [self dismissViewControllerAnimated:YES completion:nil];
 }
 
 @end