Zero-out crw_web_controller's container view delegate in dealloc.

Zero-out crw_web_controller's container view delegate in dealloc.

Exposes delegate setter on CRWWebControllerContainerView and sets
it to zero in CRWWebController dealloc.

BUG=None
TEST=None

Committed: https://crrev.com/8f3eb56c71f307254488a00cbe09a7561149ed41
Cr-Commit-Position: refs/heads/master@{#430253}

[stkhapugin, 2016-10-25 16:06:10]

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

[commit-bot: I haz the power, 2016-10-25 16:06:37]

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

[stkhapugin, 2016-10-25 16:09:29]

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

[stkhapugin, 2016-10-25 16:09:29]

ptal

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

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

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

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-10-25 18:11:12]

Thanks for a separate CL.

https://codereview.chromium.org/2453473002/diff/1/ios/web/web_state/ui/crw_we...
File ios/web/web_state/ui/crw_web_controller_container_view.mm (right):

https://codereview.chromium.org/2453473002/diff/1/ios/web/web_state/ui/crw_we...
ios/web/web_state/ui/crw_web_controller_container_view.mm:155: -
(id<CRWWebControllerContainerViewDelegate>)delegate {
nit: could you please keep the same methods order as in the order of header's
properties.

[Eugene But (OOO till 7-30), 2016-10-25 18:24:34]

Could you please update CL description and explain *why* this change was made.
lgtm

https://codereview.chromium.org/2453473002/diff/1/ios/web/web_state/ui/crw_we...
File ios/web/web_state/ui/crw_web_controller.mm (right):

https://codereview.chromium.org/2453473002/diff/1/ios/web/web_state/ui/crw_we...
ios/web/web_state/ui/crw_web_controller.mm:1117: [_containerView
setDelegate:nil];
Is this change temporary? If so, could you please add TODO to remove it.
Otherwise could you please add a comment which explains why this is necessary.

[stkhapugin, 2016-11-07 10:38:12]

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

[commit-bot: I haz the power, 2016-11-07 10:38:18]

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

[stkhapugin, 2016-11-07 10:43:42]

Comments

[stkhapugin, 2016-11-07 11:40:37]

The CQ bit was checked by stkhapugin@chromium.org

[stkhapugin, 2016-11-07 11:40:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
eugenebut@chromium.org
Link to the patchset: https://codereview.chromium.org/2453473002/#ps40001
(title: "Comments")

[commit-bot: I haz the power, 2016-11-07 11:40:46]

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-07 11:49:58]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-07 11:52:58]

Description was changed from

==========
Zero-out crw_web_controller's container view delegate in dealloc.

Exposes delegate setter on CRWWebControllerContainerView and sets
it to zero in CRWWebController dealloc.

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

to

==========
Zero-out crw_web_controller's container view delegate in dealloc.

Exposes delegate setter on CRWWebControllerContainerView and sets
it to zero in CRWWebController dealloc.

BUG=None
TEST=None

Committed: https://crrev.com/8f3eb56c71f307254488a00cbe09a7561149ed41
Cr-Commit-Position: refs/heads/master@{#430253}
==========

[commit-bot: I haz the power, 2016-11-07 11:52:58]

Patchset 3 (id:??) landed as
https://crrev.com/8f3eb56c71f307254488a00cbe09a7561149ed41
Cr-Commit-Position: refs/heads/master@{#430253}