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

ios/chrome/browser/ui/first_run/welcome_to_chrome_view.mm

 // Copyright 2015 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/first_run/welcome_to_chrome_view.h"
 
 #include "base/i18n/rtl.h"
-#import "base/ios/weak_nsobject.h"
 #include "base/logging.h"
-#import "base/mac/scoped_nsobject.h"
 #include "base/strings/sys_string_conversions.h"
 #import "ios/chrome/browser/ui/UIView+SizeClassSupport.h"
 #include "ios/chrome/browser/ui/fancy_ui/primary_action_button.h"
 #include "ios/chrome/browser/ui/first_run/first_run_util.h"
 #include "ios/chrome/browser/ui/ui_util.h"
 #import "ios/chrome/browser/ui/uikit_ui_util.h"
 #import "ios/chrome/browser/ui/util/CRUILabel+AttributeUtils.h"
 #import "ios/chrome/browser/ui/util/label_link_controller.h"
 #include "ios/chrome/common/string_util.h"
 #include "ios/chrome/grit/ios_chromium_strings.h"
 #include "ios/chrome/grit/ios_strings.h"
 #import "ios/third_party/material_roboto_font_loader_ios/src/src/MaterialRobotoFontLoader.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "url/gurl.h"
 
+#if !defined(__has_feature) || !__has_feature(objc_arc)
+#error "This file requires ARC support."
+#endif
+
 namespace {
 
 // Accessibility identifier for the checkbox button.
 NSString* const kUMAMetricsButtonAccessibilityIdentifier =
     @"UMAMetricsButtonAccessibilityIdentifier";
 
 // Color of "Terms of Service" link text.
 const int kLinkColorRGB = 0x5D9AFF;
 
 // The width of the container view for a REGULAR width size class.
@@ skipping 28 matching lines @@
 
 // Image names.
 NSString* const kAppLogoImageName = @"launchscreen_app_logo";
 NSString* const kCheckBoxImageName = @"checkbox";
 NSString* const kCheckBoxCheckedImageName = @"checkbox_checked";
 
 }  // namespace
 
 @interface WelcomeToChromeView () {
   // Backing objects for properties of the same name.
-  base::WeakNSProtocol<id<WelcomeToChromeViewDelegate>> _delegate;
-  base::scoped_nsobject<UIView> _containerView;
-  base::scoped_nsobject<UILabel> _titleLabel;
-  base::scoped_nsobject<UIImageView> _imageView;
-  base::scoped_nsobject<UILabel> _TOSLabel;
-  base::scoped_nsobject<LabelLinkController> _TOSLabelLinkController;
-  base::scoped_nsobject<UIButton> _checkBoxButton;
-  base::scoped_nsobject<UILabel> _optInLabel;
-  base::scoped_nsobject<PrimaryActionButton> _OKButton;
+  __weak id<WelcomeToChromeViewDelegate> _delegate;

[bzanotti, 2017/02/06 18:35:31]

You should be able to remove this entirely and use the auto-synthesized
property.

[lody, 2017/02/07 15:46:50]

Done.

+  UIView* _containerView;
+  UILabel* _titleLabel;
+  UIImageView* _imageView;
+  UILabel* _TOSLabel;
+  LabelLinkController* _TOSLabelLinkController;
+  UIButton* _checkBoxButton;
+  UILabel* _optInLabel;
+  PrimaryActionButton* _OKButton;
 }
 
 // Subview properties are lazily instantiated upon their first use.
 
 // A container view used to layout and center subviews.
-@property(nonatomic, readonly) UIView* containerView;
+@property(weak, nonatomic, readonly) UIView* containerView;

[stkhapugin, 2017/02/06 18:07:20]

Here and below, since those were backed with scoped_nsobject, make these strong

[lody, 2017/02/07 15:46:50]

Done.

 // The "Welcome to Chrome" label that appears at the top of the view.
-@property(nonatomic, readonly) UILabel* titleLabel;
+@property(weak, nonatomic, readonly) UILabel* titleLabel;
 // The Chrome logo image view.
-@property(nonatomic, readonly) UIImageView* imageView;
+@property(weak, nonatomic, readonly) UIImageView* imageView;
 // The "Terms of Service" label.
-@property(nonatomic, readonly) UILabel* TOSLabel;
+@property(weak, nonatomic, readonly) UILabel* TOSLabel;
 // The stats reporting opt-in label.
-@property(nonatomic, readonly) UILabel* optInLabel;
+@property(weak, nonatomic, readonly) UILabel* optInLabel;
 // The stats reporting opt-in checkbox button.
-@property(nonatomic, readonly) UIButton* checkBoxButton;
+@property(weak, nonatomic, readonly) UIButton* checkBoxButton;
 // The "Accept & Continue" button.
-@property(nonatomic, readonly) PrimaryActionButton* OKButton;
+@property(weak, nonatomic, readonly) PrimaryActionButton* OKButton;
 
 // Subview layout methods.  They must be called in the order declared here, as
 // subsequent subview layouts depend on the layouts that precede them.
 - (void)layoutTitleLabel;
 - (void)layoutImageView;
 - (void)layoutTOSLabel;
 - (void)layoutOptInLabel;
 - (void)layoutCheckBoxButton;
 - (void)layoutContainerView;
 - (void)layoutOKButton;
@@ skipping 44 matching lines @@
 
   // Get final location of logo based on result from previously run
   // layoutSubviews.
   CGRect finalLogoFrame = self.imageView.frame;
   // Ensure that frame position is valid and that layoutSubviews ran
   // before this method.
   DCHECK(finalLogoFrame.origin.x >= 0 && finalLogoFrame.origin.y >= 0);
   self.imageView.center = CGPointMake(CGRectGetMidX(self.containerView.bounds),
                                       CGRectGetMidY(self.containerView.bounds));
 
-  base::WeakNSObject<WelcomeToChromeView> weakSelf(self);
+  __weak WelcomeToChromeView* weakSelf = self;
   [UIView animateWithDuration:kAnimationDuration
                         delay:kAnimationDelay
                       options:UIViewAnimationCurveEaseInOut
                    animations:^{
                      [weakSelf imageView].frame = finalLogoFrame;
                      [weakSelf titleLabel].alpha = 1.0;
                      [weakSelf TOSLabel].alpha = 1.0;
                      [weakSelf optInLabel].alpha = 1.0;
                      [weakSelf checkBoxButton].alpha = 1.0;
                      [weakSelf OKButton].alpha = 1.0;
                    }
                    completion:nil];
 }
 
 #pragma mark - Accessors
 
 - (id<WelcomeToChromeViewDelegate>)delegate {
   return _delegate;
 }
 
 - (void)setDelegate:(id<WelcomeToChromeViewDelegate>)delegate {
-  _delegate.reset(delegate);
+  _delegate = delegate;

[stkhapugin, 2017/02/06 18:07:20]

Remove this, as this can now be synthesized. Same for getter above. 

[lody, 2017/02/07 15:46:50]

Done.

 }
 
 - (BOOL)isCheckBoxSelected {
   return self.checkBoxButton.selected;
 }
 
 - (void)setCheckBoxSelected:(BOOL)checkBoxSelected {
   if (checkBoxSelected != self.checkBoxButton.selected)
     [self checkBoxButtonWasTapped];
 }
 
 - (UIView*)containerView {
   if (!_containerView) {
-    _containerView.reset([[UIView alloc] initWithFrame:CGRectZero]);
+    _containerView = [[UIView alloc] initWithFrame:CGRectZero];
     [_containerView setBackgroundColor:[UIColor whiteColor]];
   }
-  return _containerView.get();
+  return _containerView;
 }
 
 - (UILabel*)titleLabel {
   if (!_titleLabel) {
-    _titleLabel.reset([[UILabel alloc] initWithFrame:CGRectZero]);
+    _titleLabel = [[UILabel alloc] initWithFrame:CGRectZero];
     [_titleLabel setBackgroundColor:[UIColor whiteColor]];
     [_titleLabel setNumberOfLines:0];
     [_titleLabel setLineBreakMode:NSLineBreakByWordWrapping];
     [_titleLabel setBaselineAdjustment:UIBaselineAdjustmentAlignBaselines];
     [_titleLabel
         setText:l10n_util::GetNSString(IDS_IOS_FIRSTRUN_WELCOME_TO_CHROME)];
   }
-  return _titleLabel.get();
+  return _titleLabel;
 }
 
 - (UIImageView*)imageView {
   if (!_imageView) {
     UIImage* image = [UIImage imageNamed:kAppLogoImageName];
-    _imageView.reset([[UIImageView alloc] initWithImage:image]);
+    _imageView = [[UIImageView alloc] initWithImage:image];
     [_imageView setBackgroundColor:[UIColor whiteColor]];
   }
-  return _imageView.get();
+  return _imageView;
 }
 
 - (UILabel*)TOSLabel {
   if (!_TOSLabel) {
-    _TOSLabel.reset([[UILabel alloc] initWithFrame:CGRectZero]);
+    _TOSLabel = [[UILabel alloc] initWithFrame:CGRectZero];
     [_TOSLabel setNumberOfLines:0];
     [_TOSLabel setTextAlignment:NSTextAlignmentCenter];
   }
-  return _TOSLabel.get();
+  return _TOSLabel;
 }
 
 - (UILabel*)optInLabel {
   if (!_optInLabel) {
-    _optInLabel.reset([[UILabel alloc] initWithFrame:CGRectZero]);
+    _optInLabel = [[UILabel alloc] initWithFrame:CGRectZero];
     [_optInLabel setNumberOfLines:0];
     [_optInLabel
         setText:l10n_util::GetNSString(IDS_IOS_FIRSTRUN_NEW_OPT_IN_LABEL)];
     [_optInLabel setTextAlignment:NSTextAlignmentNatural];
   }
-  return _optInLabel.get();
+  return _optInLabel;
 }
 
 - (UIButton*)checkBoxButton {
   if (!_checkBoxButton) {
-    _checkBoxButton.reset([[UIButton alloc] initWithFrame:CGRectZero]);
+    _checkBoxButton = [[UIButton alloc] initWithFrame:CGRectZero];
     [_checkBoxButton setBackgroundColor:[UIColor clearColor]];
     [_checkBoxButton addTarget:self
                         action:@selector(checkBoxButtonWasTapped)
               forControlEvents:UIControlEventTouchUpInside];
     SetA11yLabelAndUiAutomationName(_checkBoxButton,
                                     IDS_IOS_FIRSTRUN_NEW_OPT_IN_LABEL,
                                     kUMAMetricsButtonAccessibilityIdentifier);
     [_checkBoxButton
         setAccessibilityValue:l10n_util::GetNSString(IDS_IOS_SETTING_OFF)];
     [_checkBoxButton setImage:[UIImage imageNamed:kCheckBoxImageName]
                      forState:UIControlStateNormal];
     [_checkBoxButton setImage:[UIImage imageNamed:kCheckBoxCheckedImageName]
                      forState:UIControlStateSelected];
   }
-  return _checkBoxButton.get();
+  return _checkBoxButton;
 }
 
 - (PrimaryActionButton*)OKButton {
   if (!_OKButton) {
-    _OKButton.reset([[PrimaryActionButton alloc] initWithFrame:CGRectZero]);
+    _OKButton = [[PrimaryActionButton alloc] initWithFrame:CGRectZero];
     [_OKButton addTarget:self
                   action:@selector(OKButtonWasTapped)
         forControlEvents:UIControlEventTouchUpInside];
     NSString* acceptAndContinue =
         l10n_util::GetNSString(IDS_IOS_FIRSTRUN_OPT_IN_ACCEPT_BUTTON);
     [_OKButton setTitle:acceptAndContinue forState:UIControlStateNormal];
     [_OKButton setTitle:acceptAndContinue forState:UIControlStateHighlighted];
     // UIAutomation tests look for the Accept button to skip through the
     // First Run UI when it shows up.
     SetA11yLabelAndUiAutomationName(
         _OKButton, IDS_IOS_FIRSTRUN_OPT_IN_ACCEPT_BUTTON, @"Accept & Continue");
   }
-  return _OKButton.get();
+  return _OKButton;
 }
 
 #pragma mark - Layout
 
 - (void)willMoveToSuperview:(nullable UIView*)newSuperview {
   [super willMoveToSuperview:newSuperview];
 
   // Early return if the view hierarchy is already built.
   if (self.containerView.superview) {
     DCHECK_EQ(self, self.containerView.superview);
@@ skipping 57 matching lines @@
   self.TOSLabel.text = strippedText;
   if (ios_internal::FixOrphanWord(self.TOSLabel)) {
     // If a newline is inserted, check whether it was added mid-link and adjust
     // |linkTextRange| accordingly.
     NSRange newlineRange =
         [self.TOSLabel.text rangeOfString:@"\n" options:0 range:linkTextRange];
     if (newlineRange.length)
       linkTextRange.length++;
   }
 
-  base::WeakNSObject<WelcomeToChromeView> weakSelf(self);
+  __weak WelcomeToChromeView* weakSelf = self;
   ProceduralBlockWithURL action = ^(const GURL& url) {
-    base::scoped_nsobject<WelcomeToChromeView> strongSelf([weakSelf retain]);
+    WelcomeToChromeView* strongSelf = weakSelf;
     if (!strongSelf)
       return;
     [[strongSelf delegate] welcomeToChromeViewDidTapTOSLink:strongSelf];
   };
 
-  _TOSLabelLinkController.reset(
-      [[LabelLinkController alloc] initWithLabel:_TOSLabel action:action]);
+  _TOSLabelLinkController =
+      [[LabelLinkController alloc] initWithLabel:_TOSLabel action:action];
   [_TOSLabelLinkController
       addLinkWithRange:linkTextRange
                    url:GURL("internal://terms-of-service")];
   [_TOSLabelLinkController setLinkColor:UIColorFromRGB(kLinkColorRGB)];
 
   CGSize TOSLabelSize = [self.TOSLabel sizeThatFits:containerSize];
   CGFloat TOSLabelTopPadding = kTOSLabelTopPadding[self.cr_heightSizeClass];
   self.TOSLabel.frame = AlignRectOriginAndSizeToPixels(
       CGRectMake((containerSize.width - TOSLabelSize.width) / 2.0,
                  CGRectGetMaxY(self.imageView.frame) + TOSLabelTopPadding,
@@ skipping 148 matching lines @@
 
 - (void)OKButtonWasTapped {
   [self.delegate welcomeToChromeViewDidTapOKButton:self];
 }
 
 - (void)TOSLinkWasTapped {
   [self.delegate welcomeToChromeViewDidTapTOSLink:self];
 }
 
 @end