Fix the bug that "Open" button doesn't show after WebAPK installation is complete on Jellybean.

Fix the bug that "Open" button doesn't show after WebAPK installation is complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer. Therefore, a wrong package name is given to the
infobar and the infobar believes the WebAPK hasn't been installed yet.

BUG=646844
Committed: https://crrev.com/14565db671121404c1d00c0bd959d847f0f97b0d
Cr-Commit-Position: refs/heads/master@{#418610}

[Xi Han, 2016-09-14 13:42:10]

Description was changed from

==========
Fix the bug that "Open" button doesn't show after WebAPK installation is
complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer.
==========

to

==========
Fix the bug that "Open" button doesn't show after WebAPK installation is
complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer. Therefore, a wrong package name is given to the
infobar and the infobar believes the WebAPK hasn't been installed yet.
==========

[Xi Han, 2016-09-14 13:45:06]

hanxi@chromium.org changed reviewers:
+ pkotwicz@chromium.org

[Xi Han, 2016-09-14 13:45:06]

Hi Peter, please take a look, thanks!

[Xi Han, 2016-09-14 13:48:59]

Description was changed from

==========
Fix the bug that "Open" button doesn't show after WebAPK installation is
complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer. Therefore, a wrong package name is given to the
infobar and the infobar believes the WebAPK hasn't been installed yet.
==========

to

==========
Fix the bug that "Open" button doesn't show after WebAPK installation is
complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer. Therefore, a wrong package name is given to the
infobar and the infobar believes the WebAPK hasn't been installed yet.

BUG=646844
==========

[pkotwicz, 2016-09-14 14:14:43]

lgtm

[Xi Han, 2016-09-14 14:16:27]

hanxi@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[Xi Han, 2016-09-14 14:16:28]

Hi Dan, I need OWNERS review, please take a look, thanks!

[gone, 2016-09-14 17:01:25]

lgtm

[Xi Han, 2016-09-14 17:02:02]

The CQ bit was checked by hanxi@chromium.org

[commit-bot: I haz the power, 2016-09-14 17:02:27]

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-14 18:13:00]

Description was changed from

==========
Fix the bug that "Open" button doesn't show after WebAPK installation is
complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer. Therefore, a wrong package name is given to the
infobar and the infobar believes the WebAPK hasn't been installed yet.

BUG=646844
==========

to

==========
Fix the bug that "Open" button doesn't show after WebAPK installation is
complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer. Therefore, a wrong package name is given to the
infobar and the infobar believes the WebAPK hasn't been installed yet.

BUG=646844
==========

[commit-bot: I haz the power, 2016-09-14 18:13:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-14 18:14:55]

Description was changed from

==========
Fix the bug that "Open" button doesn't show after WebAPK installation is
complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer. Therefore, a wrong package name is given to the
infobar and the infobar believes the WebAPK hasn't been installed yet.

BUG=646844
==========

to

==========
Fix the bug that "Open" button doesn't show after WebAPK installation is
complete on Jellybean.

This is caused in WebApkInstaller::OnSucess() which uses a local variable after
deleting the "this" pointer. Therefore, a wrong package name is given to the
infobar and the infobar believes the WebAPK hasn't been installed yet.

BUG=646844

Committed: https://crrev.com/14565db671121404c1d00c0bd959d847f0f97b0d
Cr-Commit-Position: refs/heads/master@{#418610}
==========

[commit-bot: I haz the power, 2016-09-14 18:14:56]

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