[ObjC ARC] Converts ios/chrome/browser/ui:ui_internal_arc to ARC.

ios/chrome/browser/ui/fullscreen_controller.mm

 // Copyright 2013 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.
 
 #import "ios/chrome/browser/ui/fullscreen_controller.h"
 
 #include <cmath>
 
 #include "base/logging.h"
-#include "base/mac/objc_property_releaser.h"
+
 #import "ios/chrome/browser/ui/browser_view_controller.h"
 #import "ios/chrome/browser/ui/overscroll_actions/overscroll_actions_controller.h"
 #import "ios/chrome/browser/ui/tabs/tab_strip_controller.h"
 #import "ios/chrome/browser/ui/toolbar/toolbar_controller.h"
 #import "ios/chrome/browser/ui/toolbar/web_toolbar_controller.h"
 #import "ios/chrome/browser/ui/voice/voice_search_notification_names.h"
 #include "ios/web/public/navigation_item.h"
 #import "ios/web/public/navigation_manager.h"
 #include "ios/web/public/ssl_status.h"
 #import "ios/web/public/web_state/ui/crw_web_view_proxy.h"
 #import "ios/web/web_state/ui/crw_web_controller.h"
 
+#if !defined(__has_feature) || !__has_feature(objc_arc)
+#error "This file requires ARC support."
+#endif
+
 NSString* const kSetupForTestingWillCloseAllTabsNotification =
     @"kSetupForTestingWillCloseAllTabsNotification";
 
 using web::NavigationManager;
 
 namespace {
 
 class ScopedIncrementer {
  public:
   explicit ScopedIncrementer(int* value) : value_(value) { ++(*value_); }
@@ skipping 27 matching lines @@
   // Used to detect if the keyboard is visible.
   BOOL keyboardIsVisible_;
   // Used to detect that the OverscrollActionsController is displaying its UI.
   // The FullScreenController is disabled when the OverscrollActionsController's
   // UI is displayed.
   BOOL overscrollActionsInProgress_;
   // Counter used to keep track of the number of actions currently disabling
   // full screen.
   uint fullScreenLock_;
   // CRWWebViewProxy object allows web view manipulations.
-  base::scoped_nsprotocol<id<CRWWebViewProxy>> webViewProxy_;
-  base::mac::ObjCPropertyReleaser propertyReleaser_FullScreenController_;
+  id<CRWWebViewProxy> webViewProxy_;
 }
 
 // Access to the UIWebView's UIScrollView.
-@property(nonatomic, readonly) CRWWebViewScrollViewProxy* scrollViewProxy;
+@property(unsafe_unretained, nonatomic, readonly)
+    CRWWebViewScrollViewProxy* scrollViewProxy;
 // The navigation controller of the page.
 @property(nonatomic, readonly, assign) NavigationManager* navigationManager;
 // The gesture recognizer set on the scrollview to detect tap. Must be readwrite
 // for property releaser to work.
-@property(nonatomic, readwrite, retain)
+@property(nonatomic, readwrite, strong)
     UITapGestureRecognizer* userInteractionGestureRecognizer;
 // The delegate responsible for providing the header height and moving the
 // header.
-@property(nonatomic, readonly) id<FullScreenControllerDelegate> delegate;
+@property(unsafe_unretained, nonatomic, readonly)
+    id<FullScreenControllerDelegate>
+        delegate;
 // Current height of the header, in points. This is a pass-through method that
 // fetches the header height from the FullScreenControllerDelegate.
 @property(nonatomic, readonly) CGFloat headerHeight;
 // |top| field of UIScrollView.contentInset value caused by header.
 // Always 0 for WKWebView, as it does not support contentInset.
 @property(nonatomic, readonly) CGFloat topContentInsetCausedByHeader;
 // Last known y offset of the content in the scroll view during a scroll. Used
 // to infer the direction of the current scroll.
 @property(nonatomic, assign) CGFloat previousContentOffset;
 // Last known y offset requested on the scroll view. In general the same value
@@ skipping 88 matching lines @@
 @synthesize isFullScreenDisabledForLoading = isFullScreenDisabledForLoading_;
 @synthesize skipNextScrollOffsetForHeader = skipNextScrollOffsetForHeader_;
 @synthesize delayedHideHeaderCount = delayedHideHeaderCount_;
 @synthesize sessionID = sessionID_;
 @synthesize userInteractionGestureRecognizer =
     userInteractionGestureRecognizer_;
 
 - (id)initWithDelegate:(id<FullScreenControllerDelegate>)delegate
      navigationManager:(NavigationManager*)navigationManager
              sessionID:(NSString*)sessionID {
   if (!gEnabledForTests) {

[sdefresne, 2017/04/06 11:23:45]

Remove braces as the rest of the file does not use them for simple blocks.

[stkhapugin, 2017/04/07 13:02:09]

Done.

-    propertyReleaser_FullScreenController_.Init(self,
-                                                [FullScreenController class]);
-    [self release];
     return nil;
   }
   if ((self = [super init])) {
-    propertyReleaser_FullScreenController_.Init(self,
-                                                [FullScreenController class]);
     DCHECK(sessionID);
     DCHECK(delegate);
     delegate_ = delegate;
     sessionID_ = [sessionID copy];
     navigationManager_ = navigationManager;
 
     NSNotificationCenter* center = [NSNotificationCenter defaultCenter];
     [center addObserver:self
                selector:@selector(keyboardStart:)
                    name:UIKeyboardWillShowNotification
@@ skipping 97 matching lines @@
 
 - (void)invalidate {
   delegate_ = nil;
   navigationManager_ = NULL;
   [self.scrollViewProxy removeObserver:self];
   [[NSNotificationCenter defaultCenter] removeObserver:self];
 }
 
 - (void)dealloc {
   [[NSNotificationCenter defaultCenter] removeObserver:self];
-  [super dealloc];
 }
 
 - (CRWWebViewScrollViewProxy*)scrollViewProxy {
   return [webViewProxy_ scrollViewProxy];
 }
 
 - (CGFloat)headerHeight {
   return [self.delegate headerHeight];
 }
 
@@ skipping 363 matching lines @@
 
   [self setContentViewTopContentPadding:topInset];
 }
 
 #pragma mark -
 #pragma mark CRWWebControllerObserver methods
 
 - (void)setWebViewProxy:(id<CRWWebViewProxy>)webViewProxy
              controller:(CRWWebController*)webController {
   DCHECK([webViewProxy scrollViewProxy]);
-  webViewProxy_.reset([webViewProxy retain]);
+  webViewProxy_ = webViewProxy;
   [[webViewProxy scrollViewProxy] addObserver:self];
 }
 
 - (void)pageLoaded:(CRWWebController*)webController {
   [self enableFullScreen];
   web::WebState* webState = webController.webState;
   if (webState) {
     BOOL MIMETypeIsPDF = webState->GetContentsMimeType() == "application/pdf";
     [webViewProxy_ setShouldUseInsetForTopPadding:MIMETypeIsPDF];
   }
@@ skipping 99 matching lines @@
   overscrollActionsInProgress_ = NO;
 }
 
 #pragma mark - Used for testing
 
 + (void)setEnabledForTests:(BOOL)enabled {
   gEnabledForTests = enabled;
 }
 
 @end