Convert remoting/ios to use Bind() for blocks

With https://crrev.com/c/1070158, base::Bind() supports
block so convert uses of base::BindBlockArc() to instead
use base::BindOnce() or base::BindRepeating().

Bug: 701275
Change-Id: Ib251d93328858adfee6ccaddafebaa91124a8a24
Reviewed-on: https://chromium-review.googlesource.com/1087060Reviewed-by: Lambros Lambrou <lambroslambrou@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564833}

remoting/ios/app/remoting_view_controller.mm

@@ -11,7 +11,7 @@
 #include <SystemConfiguration/SystemConfiguration.h>
 #include <netinet/in.h>
 
-#import "base/mac/bind_objc_block.h"
+#import "base/bind.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"
@@ -206,11 +206,11 @@ using remoting::HostListService;
 
   __weak __typeof(self) weakSelf = self;
   _hostListStateSubscription =
-      _hostListService->RegisterHostListStateCallback(base::BindBlockArc(^{
+      _hostListService->RegisterHostListStateCallback(base::BindRepeating(^{
         [weakSelf hostListStateDidChange];
       }));
   _hostListFetchFailureSubscription =
-      _hostListService->RegisterFetchFailureCallback(base::BindBlockArc(^{
+      _hostListService->RegisterFetchFailureCallback(base::BindRepeating(^{
         [weakSelf hostListFetchDidFail];
       }));
 }

remoting/ios/display/gl_display_handler.mm

@@ -11,7 +11,7 @@
 
 #import "remoting/ios/display/gl_display_handler.h"
 
-#import "base/mac/bind_objc_block.h"
+#import "base/bind.h"
 #import "remoting/client/display/sys_opengl.h"
 #import "remoting/ios/display/eagl_view.h"
 #import "remoting/ios/display/gl_demo_screen.h"
@@ -177,7 +177,7 @@ void Core::OnFrameRendered() {
   // block to dereference |this|, which is thread unsafe because it doesn't
   // support ARC.
   __weak id<GlDisplayHandlerDelegate> handler_delegate = handler_delegate_;
-  runtime_->ui_task_runner()->PostTask(FROM_HERE, base::BindBlockArc(^{
+  runtime_->ui_task_runner()->PostTask(FROM_HERE, base::BindOnce(^{
                                          [handler_delegate rendererTicked];
                                        }));
 }
@@ -186,7 +186,7 @@ void Core::OnSizeChanged(int width, int height) {
   DCHECK(runtime_->display_task_runner()->BelongsToCurrentThread());
   __weak id<GlDisplayHandlerDelegate> handler_delegate = handler_delegate_;
   runtime_->ui_task_runner()->PostTask(
-      FROM_HERE, base::BindBlockArc(^{
+      FROM_HERE, base::BindOnce(^{
         [handler_delegate canvasSizeChanged:CGSizeMake(width, height)];
       }));
 }
@@ -200,7 +200,7 @@ void Core::CreateRendererContext(EAGLView* view) {
   }
   view_ = view;
 
-  runtime_->ui_task_runner()->PostTask(FROM_HERE, base::BindBlockArc(^{
+  runtime_->ui_task_runner()->PostTask(FROM_HERE, base::BindOnce(^{
                                          [view startWithContext:eagl_context_];
                                        }));
 
@@ -224,7 +224,7 @@ void Core::DestroyRendererContext() {
 
   renderer_->OnSurfaceDestroyed();
   __weak EAGLView* view = view_;
-  runtime_->ui_task_runner()->PostTask(FROM_HERE, base::BindBlockArc(^{
+  runtime_->ui_task_runner()->PostTask(FROM_HERE, base::BindOnce(^{
                                          [view stop];
                                        }));
   view_ = nil;

remoting/ios/facade/remoting_oauth_authentication.mm

@@ -11,7 +11,7 @@
 #import <Foundation/Foundation.h>
 #import <Security/Security.h>
 
-#import "base/mac/bind_objc_block.h"
+#import "base/bind.h"
 #import "ios/third_party/material_components_ios/src/components/Snackbar/src/MaterialSnackbar.h"
 #import "remoting/ios/facade/host_info.h"
 #import "remoting/ios/facade/host_list_fetcher.h"
@@ -127,7 +127,7 @@ RemotingAuthenticationStatus oauthStatusToRemotingAuthenticationStatus(
   __weak RemotingOAuthAuthentication* weakSelf = self;
   _tokenGetter = CreateOAuthTokenGetterWithAuthorizationCode(
       std::string(base::SysNSStringToUTF8(authorizationCode)),
-      base::BindBlockArc(
+      base::BindRepeating(
           ^(const std::string& user_email, const std::string& refresh_token) {
             VLOG(1) << "New Creds: " << user_email << " " << refresh_token;
             UserInfo* user = [[UserInfo alloc] init];
@@ -169,7 +169,7 @@ RemotingAuthenticationStatus oauthStatusToRemotingAuthenticationStatus(
   // Be careful here since a failure to reset onAccessToken will end up with
   // retain cycle and memory leakage.
   if (_tokenGetter) {
-    _tokenGetter->CallWithToken(base::BindBlockArc(
+    _tokenGetter->CallWithToken(base::BindRepeating(
         ^(remoting::OAuthTokenGetter::Status status,
           const std::string& user_email, const std::string& access_token) {
           onAccessToken(oauthStatusToRemotingAuthenticationStatus(status),