[ARC] Converts part of ios/web/ui to ARC.

Converts part of ios/web/ui to ARC.
Notable changes:
* ios/web/webui/crw_web_ui_page_builder.mm has a block that had a
   __block reference to itself. Before ARC __block means "do not retain
   when being captured in a block", but with ARC it is captured. So I
   added an additional variable of __weak __block type instead.

* One WeakNSObject in mojo_facade.h could not be removed because it is
  included from non-ARC code. WeakNSObject is functional in ARC but is
  not preferred. Filed crbug.com/639326 to track future removal of
  this.

BUG=639327
TEST=None

Committed: https://crrev.com/ae7aa8958c5981d3ced5b179f490b5ea9e00ba64
Cr-Commit-Position: refs/heads/master@{#419458}

[stkhapugin, 2016-09-15 16:51:24]

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

[stkhapugin, 2016-09-15 16:51:24]

PTAL. I hope this is sufficiently small. Please tell me if I should make smaller
patches!

[Eugene But (OOO till 7-30), 2016-09-15 18:04:19]

eugenebut@chromium.org changed reviewers:
+ jyquinn@chromium.org

[Eugene But (OOO till 7-30), 2016-09-15 18:04:19]

Thanks the size if perfect. Adding Jackie to review this as well. lgtm

[Jackie Quinn, 2016-09-16 20:12:13]

On 2016/09/15 18:04:19, Eugene But wrote:
> Thanks the size if perfect. Adding Jackie to review this as well. lgtm

lgtm, thanks! Notable changes list in the CL description is very helpful :-)

[stkhapugin, 2016-09-19 08:31:30]

Description was changed from

==========
Converts part of ios/web/ui to ARC.

Converts part of ios/web/ui to ARC.
Notable changes:
* ios/web/webui/crw_web_ui_page_builder.mm has a block that had a
   __block reference to itself. Before ARC __block means "do not retain
   when being captured in a block", but with ARC it is captured. So I
   added an additional variable of __weak __block type instead.

* One WeakNSObject in mojo_facade.h could not be removed because it is
  included from non-ARC code. WeakNSObject is functional in ARC but is
  not preferred. Filed crbug.com/639326 to track future removal of
  this.

BUG=639327
TEST=None
==========

to

==========
Converts part of ios/web/ui to ARC.
Notable changes:
* ios/web/webui/crw_web_ui_page_builder.mm has a block that had a
   __block reference to itself. Before ARC __block means "do not retain
   when being captured in a block", but with ARC it is captured. So I
   added an additional variable of __weak __block type instead.

* One WeakNSObject in mojo_facade.h could not be removed because it is
  included from non-ARC code. WeakNSObject is functional in ARC but is
  not preferred. Filed crbug.com/639326 to track future removal of
  this.

BUG=639327
TEST=None
==========

[stkhapugin, 2016-09-19 08:32:07]

The CQ bit was checked by stkhapugin@chromium.org

[commit-bot: I haz the power, 2016-09-19 08:32:18]

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-09-19 08:59:25]

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

[commit-bot: I haz the power, 2016-09-19 08:59:26]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[stkhapugin, 2016-09-19 14:43:28]

The CQ bit was checked by stkhapugin@chromium.org

[stkhapugin, 2016-09-19 14:43:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
eugenebut@chromium.org, jyquinn@chromium.org
Link to the patchset: https://codereview.chromium.org/2340343002/#ps20001
(title: "Fixes unit test")

[commit-bot: I haz the power, 2016-09-19 14:43:38]

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-09-19 14:59:18]

Description was changed from

==========
Converts part of ios/web/ui to ARC.
Notable changes:
* ios/web/webui/crw_web_ui_page_builder.mm has a block that had a
   __block reference to itself. Before ARC __block means "do not retain
   when being captured in a block", but with ARC it is captured. So I
   added an additional variable of __weak __block type instead.

* One WeakNSObject in mojo_facade.h could not be removed because it is
  included from non-ARC code. WeakNSObject is functional in ARC but is
  not preferred. Filed crbug.com/639326 to track future removal of
  this.

BUG=639327
TEST=None
==========

to

==========
Converts part of ios/web/ui to ARC.
Notable changes:
* ios/web/webui/crw_web_ui_page_builder.mm has a block that had a
   __block reference to itself. Before ARC __block means "do not retain
   when being captured in a block", but with ARC it is captured. So I
   added an additional variable of __weak __block type instead.

* One WeakNSObject in mojo_facade.h could not be removed because it is
  included from non-ARC code. WeakNSObject is functional in ARC but is
  not preferred. Filed crbug.com/639326 to track future removal of
  this.

BUG=639327
TEST=None
==========

[commit-bot: I haz the power, 2016-09-19 14:59:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-19 15:01:12]

Description was changed from

==========
Converts part of ios/web/ui to ARC.
Notable changes:
* ios/web/webui/crw_web_ui_page_builder.mm has a block that had a
   __block reference to itself. Before ARC __block means "do not retain
   when being captured in a block", but with ARC it is captured. So I
   added an additional variable of __weak __block type instead.

* One WeakNSObject in mojo_facade.h could not be removed because it is
  included from non-ARC code. WeakNSObject is functional in ARC but is
  not preferred. Filed crbug.com/639326 to track future removal of
  this.

BUG=639327
TEST=None
==========

to

==========
Converts part of ios/web/ui to ARC.
Notable changes:
* ios/web/webui/crw_web_ui_page_builder.mm has a block that had a
   __block reference to itself. Before ARC __block means "do not retain
   when being captured in a block", but with ARC it is captured. So I
   added an additional variable of __weak __block type instead.

* One WeakNSObject in mojo_facade.h could not be removed because it is
  included from non-ARC code. WeakNSObject is functional in ARC but is
  not preferred. Filed crbug.com/639326 to track future removal of
  this.

BUG=639327
TEST=None

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

[commit-bot: I haz the power, 2016-09-19 15:01:17]

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