[ObjC ARC] Converts ios/web:web to ARC.

ios/web/webui/web_ui_mojo_inttest.mm

 // Copyright 2016 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 <memory>
 
 #include "base/memory/ptr_util.h"
 #include "base/run_loop.h"
 #include "base/threading/thread_task_runner_handle.h"
 #import "ios/testing/wait_util.h"
@@ skipping 119 matching lines @@
  private:
   // UI handler class which communicates with test WebUI page.
   TestUIHandler* ui_handler_;
 };
 }  // namespace
 
 // A test fixture for verifying mojo comminication for WebUI.
 class WebUIMojoTest : public WebIntTest {
  protected:
   void SetUp() override {
-    WebIntTest::SetUp();
-    ui_handler_ = base::MakeUnique<TestUIHandler>();
-    web::WebState::CreateParams params(GetBrowserState());
-    web_state_ = base::MakeUnique<web::WebStateImpl>(params);
-    web_state_->GetNavigationManagerImpl().InitializeSession();
-    WebUIIOSControllerFactory::RegisterFactory(
-        new TestWebUIControllerFactory(ui_handler_.get()));
+    @autoreleasepool {
+      WebIntTest::SetUp();
+      ui_handler_ = base::MakeUnique<TestUIHandler>();
+      web::WebState::CreateParams params(GetBrowserState());
+      web_state_ = base::MakeUnique<web::WebStateImpl>(params);
+      web_state_->GetNavigationManagerImpl().InitializeSession();
+      WebUIIOSControllerFactory::RegisterFactory(
+          new TestWebUIControllerFactory(ui_handler_.get()));
+    }
+  }
+
+  void TearDown() override {
+    @autoreleasepool {
+      web_state_.reset();
+      WebIntTest::TearDown();
+    }
   }
 
   // Returns WebState which loads test WebUI page.
   WebStateImpl* web_state() { return web_state_.get(); }
   // Returns UI handler which communicates with WebUI page.
   TestUIHandler* test_ui_handler() { return ui_handler_.get(); }
 
  private:
   std::unique_ptr<WebStateImpl> web_state_;
   std::unique_ptr<TestUIHandler> ui_handler_;
 };
 
 // Tests that JS can send messages to the native code and vice versa.
 // TestUIHandler is used for communication and test suceeds only when
 // |TestUIHandler| sucessfully receives "ack" message from WebUI page.
 #if TARGET_IPHONE_SIMULATOR
 #define MAYBE_MessageExchange MessageExchange
 #else
 #define MAYBE_MessageExchange FLAKY_MessageExchange
 #endif
 // TODO(crbug.com/720098): Enable this test on device.
 TEST_F(WebUIMojoTest, MAYBE_MessageExchange) {
-  web_state()->SetWebUsageEnabled(true);
-  web_state()->GetView();  // WebState won't load URL without view.
-  GURL url(
-      url::SchemeHostPort(kTestWebUIScheme, kTestWebUIURLHost, 0).Serialize());
-  NavigationManager::WebLoadParams load_params(url);
-  web_state()->GetNavigationManager()->LoadURLWithParams(load_params);
+  @autoreleasepool {

[Eugene But (OOO till 7-30), 2017/06/14 01:27:59]

Would ScopedAutoreleasePool work here to avoid indentations?

[stkhapugin, 2017/06/14 17:00:59]

We should probably not use ScopedAutoreleasePool and use @autoreleasepool
instead. If the identations bother you, maybe we could not indent this
specifically, like with namespace{} ?

[Eugene But (OOO till 7-30), 2017/06/14 23:10:18]

Indentation is fine. Just out of curiosity, why we should avoid
ScopedAutoreleasePool?

+    web_state()->SetWebUsageEnabled(true);
+    web_state()->GetView();  // WebState won't load URL without view.
+    GURL url(url::SchemeHostPort(kTestWebUIScheme, kTestWebUIURLHost, 0)
+                 .Serialize());
+    NavigationManager::WebLoadParams load_params(url);
+    web_state()->GetNavigationManager()->LoadURLWithParams(load_params);
 
-  // Wait until |TestUIHandler| receives "fin" message from WebUI page.
-  bool fin_received = testing::WaitUntilConditionOrTimeout(kMessageTimeout, ^{
-    // Flush any pending tasks. Don't RunUntilIdle() because
-    // RunUntilIdle() is incompatible with mojo::SimpleWatcher's
-    // automatic arming behavior, which Mojo JS still depends upon.
-    //
-    // TODO(crbug.com/701875): Introduce the full watcher API to JS and get rid
-    // of this hack.
-    base::RunLoop loop;
-    base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
-                                                  loop.QuitClosure());
-    loop.Run();
-    return test_ui_handler()->IsFinReceived();
-  });
+    // Wait until |TestUIHandler| receives "fin" message from WebUI page.
+    bool fin_received = testing::WaitUntilConditionOrTimeout(kMessageTimeout, ^{
+      // Flush any pending tasks. Don't RunUntilIdle() because
+      // RunUntilIdle() is incompatible with mojo::SimpleWatcher's
+      // automatic arming behavior, which Mojo JS still depends upon.
+      //
+      // TODO(crbug.com/701875): Introduce the full watcher API to JS and get
+      // rid of this hack.
+      base::RunLoop loop;
+      base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
+                                                    loop.QuitClosure());
+      loop.Run();
+      return test_ui_handler()->IsFinReceived();
+    });
 
-  ASSERT_TRUE(fin_received);
-  EXPECT_FALSE(web_state()->IsLoading());
-  EXPECT_EQ(url, web_state()->GetLastCommittedURL());
+    ASSERT_TRUE(fin_received);
+    EXPECT_FALSE(web_state()->IsLoading());
+    EXPECT_EQ(url, web_state()->GetLastCommittedURL());
+  }
 }
 
 }  // namespace web