Fix window.close() in Athena

Fix window.close() in Athena

This patch makes AthenaWebView handle content::WebContentsDelegate::CloseContents().

BUG=412188
TEST=manual test

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

[yoshiki, 2014-10-16 18:40:15]

yoshiki@chromium.org changed reviewers:
+ oshima@chromium.org

[yoshiki, 2014-10-16 18:42:29]

yoshiki@chromium.org changed required reviewers:
+ oshima@chromium.org

[yoshiki, 2014-10-16 18:42:29]

Oshima-san, PTAL. Thanks.

[oshima, 2014-10-16 18:44:48]

https://codereview.chromium.org/659113002/diff/1/athena/content/web_activity.cc
File athena/content/web_activity.cc (right):

https://codereview.chromium.org/659113002/diff/1/athena/content/web_activity....
athena/content/web_activity.cc:374: GetWidget()->Close();
This may not delete activity. Does the following work?

Acvitity::Delete(this)

[yoshiki, 2014-10-16 18:49:58]

https://codereview.chromium.org/659113002/diff/1/athena/content/web_activity.cc
File athena/content/web_activity.cc (right):

https://codereview.chromium.org/659113002/diff/1/athena/content/web_activity....
athena/content/web_activity.cc:374: GetWidget()->Close();
On 2014/10/16 18:44:48, oshima wrote:
> This may not delete activity. Does the following work?
> 
> Acvitity::Delete(this)
> 
> 
I'll check whether it works. BTW, should we change L139 as well?

[oshima, 2014-10-16 18:51:49]

On 2014/10/16 18:49:58, yoshiki wrote:
>
https://codereview.chromium.org/659113002/diff/1/athena/content/web_activity.cc
> File athena/content/web_activity.cc (right):
> 
>
https://codereview.chromium.org/659113002/diff/1/athena/content/web_activity....
> athena/content/web_activity.cc:374: GetWidget()->Close();
> On 2014/10/16 18:44:48, oshima wrote:
> > This may not delete activity. Does the following work?
> > 
> > Acvitity::Delete(this)
> > 
> > 
> I'll check whether it works. BTW, should we change L139 as well?

Yes please.

[yoshiki, 2014-10-16 23:20:23]

Oshima san, PTAL. Thanks.

[oshima, 2014-10-17 16:41:03]

lgtm

[yoshiki, 2014-10-17 17:21:23]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-10-17 17:21:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/659113002/40001

[commit-bot: I haz the power, 2014-10-17 17:45:09]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-10-17 17:45:53]

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