Converts parts of ios/web to ARC

ios/web/navigation/navigation_manager_impl.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.
 
 #include "ios/web/navigation/navigation_manager_impl.h"
 
 #include <stddef.h>
 
 #include <utility>
 
@@ skipping 66 matching lines @@
 
 NavigationManagerImpl::~NavigationManagerImpl() {
   // The facade layer should be deleted before this object.
   DCHECK(!facade_delegate_);
 
   [session_controller_ setNavigationManager:nullptr];
 }
 
 void NavigationManagerImpl::SetSessionController(
     CRWSessionController* session_controller) {
-  session_controller_.reset([session_controller retain]);
+  session_controller_.reset(session_controller);
   [session_controller_ setNavigationManager:this];
 }
 
 void NavigationManagerImpl::InitializeSession(NSString* window_name,
                                               NSString* opener_id,
                                               BOOL opened_by_dom,
                                               int opener_navigation_index) {
-  SetSessionController([[[CRWSessionController alloc]
+  SetSessionController([[CRWSessionController alloc]
          initWithWindowName:window_name
                    openerId:opener_id
                 openedByDOM:opened_by_dom
       openerNavigationIndex:opener_navigation_index
-               browserState:browser_state_] autorelease]);
+               browserState:browser_state_]);
 }
 
 void NavigationManagerImpl::ReplaceSessionHistory(
     ScopedVector<web::NavigationItem> items,
     int current_index) {
-  SetSessionController([[[CRWSessionController alloc]
+  SetSessionController([[CRWSessionController alloc]
       initWithNavigationItems:std::move(items)
                  currentIndex:current_index
-                 browserState:browser_state_] autorelease]);
+                 browserState:browser_state_]);
 }
 
 void NavigationManagerImpl::SetFacadeDelegate(
     NavigationManagerFacadeDelegate* facade_delegate) {
   facade_delegate_ = facade_delegate;
 }
 
 NavigationManagerFacadeDelegate* NavigationManagerImpl::GetFacadeDelegate()
     const {
   return facade_delegate_;
@@ skipping 195 matching lines @@
 NavigationManagerImpl::GetTransientURLRewriters() {
   return std::move(transient_url_rewriters_);
 }
 
 void NavigationManagerImpl::RemoveTransientURLRewriters() {
   transient_url_rewriters_.reset();
 }
 
 void NavigationManagerImpl::CopyState(
     NavigationManagerImpl* navigation_manager) {
-  SetSessionController(
-      [[navigation_manager->GetSessionController() copy] autorelease]);
+  SetSessionController([navigation_manager->GetSessionController() copy]);

[Eugene But (OOO till 7-30), 2016/08/01 21:13:48]

This looks like a leak too.

[stkhapugin, 2016/08/18 17:09:36]

Same here, ARC always balances retains and releases (aside for new- alloc- copy-
and NS_RETURNS_RETAINED methods)

 }
 
 }  // namespace web