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

ios/chrome/browser/ui/omnibox/page_info_view_controller.h

 // Copyright (c) 2012 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.
 
 #ifndef IOS_CHROME_BROWSER_UI_OMNIBOX_PAGE_INFO_VIEW_CONTROLLER_H_
 #define IOS_CHROME_BROWSER_UI_OMNIBOX_PAGE_INFO_VIEW_CONTROLLER_H_
 
 #import <UIKit/UIKit.h>
 
 #include <memory>
@@ skipping 35 matching lines @@
 
   // PageInfoModelObserver implementation.
   void OnPageInfoModelChanged() override;
 
   // Sets the controller.
   void set_controller(PageInfoViewController* controller) {
     controller_ = controller;
   }
 
  private:
   void PerformLayout() {

[rohitrao (ping after 24h), 2017/02/24 13:40:32]

Should we move the body of this method into the .mm file?  Doesn't seem like it
belongs in the header.

[stkhapugin, 2017/03/01 17:43:59]

Done.

     // If the window is animating closed when this is called, the
     // animation could be holding the last reference to |controller_|
     // (and thus |this|).  Pin it until the task is completed.
-    base::scoped_nsobject<PageInfoViewController> keep_alive(
-        [controller_ retain]);
+    base::scoped_nsobject<PageInfoViewController> keep_alive(controller_);
     [controller_ performLayout];
   }
 
   PageInfoViewController* controller_;  // weak

[rohitrao (ping after 24h), 2017/02/24 13:40:32]

This is an objc class.  Does it need to be annotated as __unsafe_unretained?

[stkhapugin, 2017/03/01 17:43:59]

Done.

 
   base::WeakPtrFactory<PageInfoModelBubbleBridge> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PageInfoModelBubbleBridge);
 };
 
 #endif  // IOS_CHROME_BROWSER_UI_OMNIBOX_PAGE_INFO_VIEW_CONTROLLER_H_