[Remoting iOS] Basic viewport manipulation support

remoting/client/ios/display/gl_display_handler.h

Index: remoting/client/ios/display/gl_display_handler.h
diff --git a/remoting/client/ios/display/gl_display_handler.h b/remoting/client/ios/display/gl_display_handler.h
index d6e7fe301b702051cb49901ee003620fb0ae25cf..a0fc57ebe84cb110deab7e58c6fa222c4727cf92 100644
--- a/remoting/client/ios/display/gl_display_handler.h
+++ b/remoting/client/ios/display/gl_display_handler.h
@@ -7,10 +7,11 @@
 
 #import <Foundation/Foundation.h>
 #import <GLKit/GLKit.h>
-
-#import "remoting/client/display/sys_opengl.h"
+#include <array>

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

new line between #import and include
 
Where are you using array in the .h file?

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

Removed. Not used anymore.

 
 #include "base/memory/ptr_util.h"
+#import "remoting/client/display/sys_opengl.h"

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

sort the includes, #imports first

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

Done.

+#include "remoting/client/view_matrix.h"
 
 namespace remoting {
 
@@ -24,17 +25,28 @@ class CursorShapeStub;
 }  // namespace protocol
 }  // namespace remoting
 
+@protocol GlDisplayHandlerDelegate<NSObject>

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

Can you also add a comment describing the what the goal of the delegate is?

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

Done.

+
+// Notifies the delegate that the size of the desktop image has changed.
+- (void)canvasSizeChanged:(CGSize)size;
+
+@end
+
 @interface GlDisplayHandler : NSObject {
 }
 
 - (void)stop;
 
+- (void)setDelegate:(id<GlDisplayHandlerDelegate>)delegate;

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

Better to use @property delegate, for example:
https://cs.chromium.org/chromium/src/remoting/client/ios/key_input.h?type=cs&...

And then you can still call setDelegate but you really should use
object.delegate = someDelegate;

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

Done. Manually synthesized with trick to disable the getter. ObjC doesn't
support write-only @property...

+
 // Called once the GLKView created.
 - (void)onSurfaceCreated:(GLKView*)view;
 
 // Called every time the GLKView dimension is initialized or changed.
 - (void)onSurfaceChanged:(const CGRect&)frame;
 
+- (void)onPixelTransformationChanged:(const remoting::ViewMatrix&)matrix;
+
 - (std::unique_ptr<remoting::protocol::VideoRenderer>)CreateVideoRenderer;
 - (std::unique_ptr<remoting::protocol::CursorShapeStub>)CreateCursorShapeStub;