Revert of [ObjC ARC] Converts crw_web_controller to ARC.

Revert of [ObjC ARC] Converts crw_web_controller to ARC. (patchset #8 id:140001 of https://codereview.chromium.org/2434853002/ )

Reason for revert:
This caused multiple Chrome for iOS unit tests to fail when creating tabs.

It resulted in tests crashing when checking if currently on the web thread.

Original issue's description:
> [ObjC ARC] Converts crw_web_controller to ARC.
>
> Notable changes:
> * crw_web_controller.mm used to have a dictionary storing selectors just
> for keeping the KVO code clean. ARC cannot call a selector from a string
> like this, because it cannot guarantee correct memory management in
> this case. So it was replaced with an array of observed key paths to
> keep some of the code cleanness, while the KVO callback has a huge
> switch.
>
> BUG=624365
> TEST=None
>
> Committed: https://crrev.com/9b6f44b09855a803914efd9877fddb471a97cbf1
> Cr-Commit-Position: refs/heads/master@{#430304}

TBR=sdefresne@chromium.org,eugenebut@chromium.org,stkhapugin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624365
Committed: https://crrev.com/85d802e4f2fc5ee1c7480f8f99e83455035f65c8
Cr-Commit-Position: refs/heads/master@{#430443}

[baxley, 2016-11-08 00:33:10]

The CQ bit was checked by baxley@chromium.org

[baxley, 2016-11-08 00:33:11]

Created Revert of [ObjC ARC] Converts crw_web_controller to ARC.

[commit-bot: I haz the power, 2016-11-08 00:34:01]

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-11-08 00:37:12]

Description was changed from

==========
Revert of [ObjC ARC] Converts crw_web_controller to ARC. (patchset #8 id:140001
of https://codereview.chromium.org/2434853002/ )

Reason for revert:
This caused multiple Chrome for iOS unit tests to fail when creating tabs.

It resulted in tests crashing when checking if currently on the web thread.

Original issue's description:
> [ObjC ARC] Converts crw_web_controller to ARC.
> 
> Notable changes:
> * crw_web_controller.mm used to have a dictionary storing selectors just
> for keeping the KVO code clean. ARC cannot call a selector from a string
> like this, because it cannot guarantee correct memory management in
> this case. So it was replaced with an array of observed key paths to
> keep some of the code cleanness, while the KVO callback has a huge
> switch.
> 
> BUG=624365
> TEST=None
> 
> Committed: https://crrev.com/9b6f44b09855a803914efd9877fddb471a97cbf1
> Cr-Commit-Position: refs/heads/master@{#430304}

TBR=sdefresne@chromium.org,eugenebut@chromium.org,stkhapugin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624365
==========

to

==========
Revert of [ObjC ARC] Converts crw_web_controller to ARC. (patchset #8 id:140001
of https://codereview.chromium.org/2434853002/ )

Reason for revert:
This caused multiple Chrome for iOS unit tests to fail when creating tabs.

It resulted in tests crashing when checking if currently on the web thread.

Original issue's description:
> [ObjC ARC] Converts crw_web_controller to ARC.
> 
> Notable changes:
> * crw_web_controller.mm used to have a dictionary storing selectors just
> for keeping the KVO code clean. ARC cannot call a selector from a string
> like this, because it cannot guarantee correct memory management in
> this case. So it was replaced with an array of observed key paths to
> keep some of the code cleanness, while the KVO callback has a huge
> switch.
> 
> BUG=624365
> TEST=None
> 
> Committed: https://crrev.com/9b6f44b09855a803914efd9877fddb471a97cbf1
> Cr-Commit-Position: refs/heads/master@{#430304}

TBR=sdefresne@chromium.org,eugenebut@chromium.org,stkhapugin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624365
==========

[commit-bot: I haz the power, 2016-11-08 00:37:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-08 00:52:45]

Description was changed from

==========
Revert of [ObjC ARC] Converts crw_web_controller to ARC. (patchset #8 id:140001
of https://codereview.chromium.org/2434853002/ )

Reason for revert:
This caused multiple Chrome for iOS unit tests to fail when creating tabs.

It resulted in tests crashing when checking if currently on the web thread.

Original issue's description:
> [ObjC ARC] Converts crw_web_controller to ARC.
> 
> Notable changes:
> * crw_web_controller.mm used to have a dictionary storing selectors just
> for keeping the KVO code clean. ARC cannot call a selector from a string
> like this, because it cannot guarantee correct memory management in
> this case. So it was replaced with an array of observed key paths to
> keep some of the code cleanness, while the KVO callback has a huge
> switch.
> 
> BUG=624365
> TEST=None
> 
> Committed: https://crrev.com/9b6f44b09855a803914efd9877fddb471a97cbf1
> Cr-Commit-Position: refs/heads/master@{#430304}

TBR=sdefresne@chromium.org,eugenebut@chromium.org,stkhapugin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624365
==========

to

==========
Revert of [ObjC ARC] Converts crw_web_controller to ARC. (patchset #8 id:140001
of https://codereview.chromium.org/2434853002/ )

Reason for revert:
This caused multiple Chrome for iOS unit tests to fail when creating tabs.

It resulted in tests crashing when checking if currently on the web thread.

Original issue's description:
> [ObjC ARC] Converts crw_web_controller to ARC.
>
> Notable changes:
> * crw_web_controller.mm used to have a dictionary storing selectors just
> for keeping the KVO code clean. ARC cannot call a selector from a string
> like this, because it cannot guarantee correct memory management in
> this case. So it was replaced with an array of observed key paths to
> keep some of the code cleanness, while the KVO callback has a huge
> switch.
>
> BUG=624365
> TEST=None
>
> Committed: https://crrev.com/9b6f44b09855a803914efd9877fddb471a97cbf1
> Cr-Commit-Position: refs/heads/master@{#430304}

TBR=sdefresne@chromium.org,eugenebut@chromium.org,stkhapugin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624365

Committed: https://crrev.com/85d802e4f2fc5ee1c7480f8f99e83455035f65c8
Cr-Commit-Position: refs/heads/master@{#430443}
==========

[commit-bot: I haz the power, 2016-11-08 00:52:46]

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

[sdefresne, 2016-11-08 08:41:41]

lgtm

[stkhapugin, 2016-11-08 14:25:57]

On 2016/11/08 08:41:41, sdefresne wrote:
> lgtm

lgtm. In the future when creating a revert, please include at least one example
failure log.