[Autofill] Close mojo interface bindings of autofill/password agent in time.

[Autofill] Close mojo interface bindings of autofill/password agent in time.

As for AutofillAgent, Password{Autofill,Generation}Agent, their destructions
are asynchronous with destructions of corresponding render frame.
Thus, even after corresponding render frame has been deleted, they can still
keep alive for some while and still able to receive mojo messages which maybe
lead to use-after-delete access on the render frame.

To solve above problem, this CL closes the mojo interface bindings immediatelly
when render frame got destroyed.

BUG=641156
Committed: https://crrev.com/7e8c85b79831bb59427ffc5b229bc08470f619ef
Cr-Commit-Position: refs/heads/master@{#417286}

[leonhsl(Using Gerrit), 2016-09-08 12:16:50]

The CQ bit was checked by leon.han@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-08 12:17:05]

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

[leonhsl(Using Gerrit), 2016-09-08 12:24:04]

leon.han@intel.com changed reviewers:
+ vabr@chromium.org

[leonhsl(Using Gerrit), 2016-09-08 12:24:05]

I suppose this CL should be able to solve bug 641156, could we just land this to
see effect? I think at least this CL is doing correct work ;-)

[vabr (Chromium), 2016-09-08 12:26:25]

LGTM, this looks like a very good idea!

Thanks!
Vaclav

[commit-bot: I haz the power, 2016-09-08 13:26:39]

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

[commit-bot: I haz the power, 2016-09-08 13:26:40]

Dry run: This issue passed the CQ dry run.

[leonhsl(Using Gerrit), 2016-09-08 14:20:39]

On 2016/09/08 12:26:25, vabr (Chromium) wrote:
> LGTM, this looks like a very good idea!
> 
> Thanks!
> Vaclav

Thanks! Landing now.

[leonhsl(Using Gerrit), 2016-09-08 14:20:46]

The CQ bit was checked by leon.han@intel.com

[commit-bot: I haz the power, 2016-09-08 14:21:07]

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-08 14:25:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-08 14:27:23]

Description was changed from

==========
[Autofill] Close mojo interface bindings of autofill/password agent in time.

As for AutofillAgent, Password{Autofill,Generation}Agent, their destructions
are asynchronous with destructions of corresponding render frame.
Thus, even after corresponding render frame has been deleted, they can still
keep alive for some while and still able to receive mojo messages which maybe
lead to use-after-delete access on the render frame.

To solve above problem, this CL closes the mojo interface bindings immediatelly
when render frame got destroyed.

BUG=641156
==========

to

==========
[Autofill] Close mojo interface bindings of autofill/password agent in time.

As for AutofillAgent, Password{Autofill,Generation}Agent, their destructions
are asynchronous with destructions of corresponding render frame.
Thus, even after corresponding render frame has been deleted, they can still
keep alive for some while and still able to receive mojo messages which maybe
lead to use-after-delete access on the render frame.

To solve above problem, this CL closes the mojo interface bindings immediatelly
when render frame got destroyed.

BUG=641156

Committed: https://crrev.com/7e8c85b79831bb59427ffc5b229bc08470f619ef
Cr-Commit-Position: refs/heads/master@{#417286}
==========

[commit-bot: I haz the power, 2016-09-08 14:27:24]

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