[ObjC ARC] Converts ios/web/shell/test to ARC.

[ObjC ARC] Converts ios/web/shell/test to ARC.

Notable changes:none

BUG=624365
Committed: https://crrev.com/bd8522e26c5b1c19576d2cf0dbd9783f6cf76832
Cr-Commit-Position: refs/heads/master@{#427731}

[stkhapugin, 2016-10-25 14:44:53]

The CQ bit was checked by stkhapugin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-25 14:45:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-25 15:48:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-25 15:48:16]

Dry run: This issue passed the CQ dry run.

[stkhapugin, 2016-10-25 16:11:07]

stkhapugin@chromium.org changed reviewers:
+ baxley@chromium.org

[stkhapugin, 2016-10-25 16:11:08]

ptal

[baxley, 2016-10-25 16:16:29]

One question about files in ios/web/shell/test (and not subdirectories)

https://codereview.chromium.org/2446893004/diff/1/ios/web/shell/test/BUILD.gn
File ios/web/shell/test/BUILD.gn (right):

https://codereview.chromium.org/2446893004/diff/1/ios/web/shell/test/BUILD.gn...
ios/web/shell/test/BUILD.gn:14: "pdf_egtest.mm",
Will you convert this to arc in a separate CL? The tests have a couple of
allocs.

[stkhapugin, 2016-10-26 15:51:19]

https://codereview.chromium.org/2446893004/diff/1/ios/web/shell/test/BUILD.gn
File ios/web/shell/test/BUILD.gn (right):

https://codereview.chromium.org/2446893004/diff/1/ios/web/shell/test/BUILD.gn...
ios/web/shell/test/BUILD.gn:14: "pdf_egtest.mm",
On 2016/10/25 16:16:29, baxley wrote:
> Will you convert this to arc in a separate CL? The tests have a couple of
> allocs.

See comment below

https://codereview.chromium.org/2446893004/diff/1/ios/web/shell/test/BUILD.gn...
ios/web/shell/test/BUILD.gn:37: configs += [
"//build/config/compiler:enable_arc" ]
These files were already ARC :)

[baxley, 2016-10-26 16:04:16]

On 2016/10/26 15:51:19, stkhapugin wrote:
> https://codereview.chromium.org/2446893004/diff/1/ios/web/shell/test/BUILD.gn
> File ios/web/shell/test/BUILD.gn (right):
> 
>
https://codereview.chromium.org/2446893004/diff/1/ios/web/shell/test/BUILD.gn...
> ios/web/shell/test/BUILD.gn:14: "pdf_egtest.mm",
> On 2016/10/25 16:16:29, baxley wrote:
> > Will you convert this to arc in a separate CL? The tests have a couple of
> > allocs.
> 
> See comment below
> 
>
https://codereview.chromium.org/2446893004/diff/1/ios/web/shell/test/BUILD.gn...
> ios/web/shell/test/BUILD.gn:37: configs += [
> "//build/config/compiler:enable_arc" ]
> These files were already ARC :)

:)
LGTM

[stkhapugin, 2016-10-26 16:18:17]

The CQ bit was checked by stkhapugin@chromium.org

[commit-bot: I haz the power, 2016-10-26 16:18:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-26 17:30:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-26 17:50:52]

Description was changed from

==========
[ObjC ARC] Converts ios/web/shell/test to ARC.

Notable changes:none

BUG=624365
==========

to

==========
[ObjC ARC] Converts ios/web/shell/test to ARC.

Notable changes:none

BUG=624365

Committed: https://crrev.com/bd8522e26c5b1c19576d2cf0dbd9783f6cf76832
Cr-Commit-Position: refs/heads/master@{#427731}
==========

[commit-bot: I haz the power, 2016-10-26 17:50:53]

Patchset 1 (id:??) landed as
https://crrev.com/bd8522e26c5b1c19576d2cf0dbd9783f6cf76832
Cr-Commit-Position: refs/heads/master@{#427731}