Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome
is running, WebApkValidator returns false for a valid WebAPK since
the WebApkValidator hasn't been initialized.

BUG=609122
Committed: https://crrev.com/53cac0649b0bcac6c135928287136050ad296dad
Cr-Commit-Position: refs/heads/master@{#399544}

[Xi Han, 2016-06-10 17:57:30]

Description was changed from

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This cl fixes a bug that when a WebAPK is launched without Chrome,
WebApkValidator returns false for a valid WebAPK since the
WebApkValidator hasn't been initialized.

BUG=609122
==========

to

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome,
WebApkValidator returns false for a valid WebAPK since the
WebApkValidator hasn't been initialized.

BUG=609122
==========

[Xi Han, 2016-06-10 17:57:54]

Description was changed from

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome,
WebApkValidator returns false for a valid WebAPK since the
WebApkValidator hasn't been initialized.

BUG=609122
==========

to

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome
is running, WebApkValidator returns false for a valid WebAPK since the
WebApkValidator hasn't been initialized.

BUG=609122
==========

[Xi Han, 2016-06-10 17:58:08]

Description was changed from

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome
is running, WebApkValidator returns false for a valid WebAPK since the
WebApkValidator hasn't been initialized.

BUG=609122
==========

to

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome
is running, WebApkValidator returns false for a valid WebAPK since
the WebApkValidator hasn't been initialized.

BUG=609122
==========

[Xi Han, 2016-06-10 17:59:16]

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

[Xi Han, 2016-06-10 17:59:17]

Hi Peter, could you please take a look? Thanks!

[pkotwicz, 2016-06-10 21:34:14]

LGTM with nit

This is probably the cause of what I observed in
https://codereview.chromium.org/2049843004/

https://codereview.chromium.org/2056383002/diff/1/chrome/android/webapk/libs/...
File
chrome/android/webapk/libs/client/src/org/chromium/webapk/lib/client/WebApkValidator.java
(right):

https://codereview.chromium.org/2056383002/diff/1/chrome/android/webapk/libs/...
chrome/android/webapk/libs/client/src/org/chromium/webapk/lib/client/WebApkValidator.java:122:
if (sIsInitialized) {
Can we return if sExpectedSignature != null ?

[Xi Han, 2016-06-13 20:10:53]

This fixes exactly the issue that you mentioned in my previous CL. Since the
changes are irreverent, so I moved them to this following up CL.

https://codereview.chromium.org/2056383002/diff/1/chrome/android/webapk/libs/...
File
chrome/android/webapk/libs/client/src/org/chromium/webapk/lib/client/WebApkValidator.java
(right):

https://codereview.chromium.org/2056383002/diff/1/chrome/android/webapk/libs/...
chrome/android/webapk/libs/client/src/org/chromium/webapk/lib/client/WebApkValidator.java:122:
if (sIsInitialized) {
On 2016/06/10 21:34:14, pkotwicz wrote:
> Can we return if sExpectedSignature != null ?

Done.

[Xi Han, 2016-06-13 20:11:20]

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

[Xi Han, 2016-06-13 20:11:21]

Hi Dan, could you please take a look? Thanks!

[gone, 2016-06-13 20:35:48]

lgtm

[Xi Han, 2016-06-13 20:36:39]

The CQ bit was checked by hanxi@chromium.org

[Xi Han, 2016-06-13 20:36:40]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkotwicz@chromium.org
Link to the patchset: https://codereview.chromium.org/2056383002/#ps20001
(title: "Nits.")

[commit-bot: I haz the power, 2016-06-13 20:37:25]

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

[commit-bot: I haz the power, 2016-06-13 21:24:24]

Description was changed from

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome
is running, WebApkValidator returns false for a valid WebAPK since
the WebApkValidator hasn't been initialized.

BUG=609122
==========

to

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome
is running, WebApkValidator returns false for a valid WebAPK since
the WebApkValidator hasn't been initialized.

BUG=609122
==========

[commit-bot: I haz the power, 2016-06-13 21:24:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-13 21:24:53]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-13 21:26:09]

Description was changed from

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome
is running, WebApkValidator returns false for a valid WebAPK since
the WebApkValidator hasn't been initialized.

BUG=609122
==========

to

==========
Upstream: Initialize ChromeWebApkHost in WebappLaunchActivity.

This CL fixes a bug that when a WebAPK is launched without Chrome
is running, WebApkValidator returns false for a valid WebAPK since
the WebApkValidator hasn't been initialized.

BUG=609122

Committed: https://crrev.com/53cac0649b0bcac6c135928287136050ad296dad
Cr-Commit-Position: refs/heads/master@{#399544}
==========

[commit-bot: I haz the power, 2016-06-13 21:26:10]

Patchset 2 (id:??) landed as
https://crrev.com/53cac0649b0bcac6c135928287136050ad296dad
Cr-Commit-Position: refs/heads/master@{#399544}