arc: Make sure PlayStore logo image loaded for Arc OptIn OOBE page.

arc: Make sure PlayStore logo image loaded for Arc OptIn OOBE page.

From my investigation, there is the case when Chromebook is loaded
first time. OOBE UI initializedd on early step and internet connection
is not guaranteed at this moment and this may fail image loading.
Fix is to reload image before page is actually shown to user.

BUG=670750
TEST=Manually on device. Restart/Powerwash - in all cases image is
     loaded.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/3408557009dd278f7272ee07b06704f788509b00
Cr-Commit-Position: refs/heads/master@{#437040}

[khmel, 2016-12-07 01:04:38]

Description was changed from

==========
arc: Make sure PlayStore logo image loaded for Arc OptIn OOBE page.

From my investigation, there is the case when Chromebook is loaded
first time. OOBE UI initializedd on early step and internet connection
is not guaranteed at this moment and this may fail image loading.
Fix is to reload image before page is actually shown to user.

BUG=670750
TEST=Manually on device. Restart/Powerwash - in all cases image is
     loaded.
==========

to

==========
arc: Make sure PlayStore logo image loaded for Arc OptIn OOBE page.

From my investigation, there is the case when Chromebook is loaded
first time. OOBE UI initializedd on early step and internet connection
is not guaranteed at this moment and this may fail image loading.
Fix is to reload image before page is actually shown to user.

BUG=670750
TEST=Manually on device. Restart/Powerwash - in all cases image is
     loaded.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[khmel, 2016-12-07 01:05:17]

Hi Xiyuan, PTAL

[khmel, 2016-12-07 16:14:33]

khmel@chromium.org changed reviewers:
+ xiyuan@chromium.org

[khmel, 2016-12-07 16:14:34]

Hi Xiyuan,

PTAL

[xiyuan, 2016-12-07 18:23:05]

lgtm

Yep, we should not access remote resources during oobe page loading since
Internet access might not be there.

[khmel, 2016-12-07 18:23:45]

Thanks for confirmation! :)

[khmel, 2016-12-07 18:23:50]

The CQ bit was checked by khmel@chromium.org

[commit-bot: I haz the power, 2016-12-07 18:24:08]

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-12-07 19:58:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481135030883010, "parent_rev":
"78d88a62b4526fb402ad063fdc50d9c8dda9db2c", "commit_rev":
"02748f99b586dbe88b1e2b22b018505d40ae1367"}

[commit-bot: I haz the power, 2016-12-07 19:58:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-07 20:01:21]

Description was changed from

==========
arc: Make sure PlayStore logo image loaded for Arc OptIn OOBE page.

From my investigation, there is the case when Chromebook is loaded
first time. OOBE UI initializedd on early step and internet connection
is not guaranteed at this moment and this may fail image loading.
Fix is to reload image before page is actually shown to user.

BUG=670750
TEST=Manually on device. Restart/Powerwash - in all cases image is
     loaded.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
arc: Make sure PlayStore logo image loaded for Arc OptIn OOBE page.

From my investigation, there is the case when Chromebook is loaded
first time. OOBE UI initializedd on early step and internet connection
is not guaranteed at this moment and this may fail image loading.
Fix is to reload image before page is actually shown to user.

BUG=670750
TEST=Manually on device. Restart/Powerwash - in all cases image is
     loaded.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/3408557009dd278f7272ee07b06704f788509b00
Cr-Commit-Position: refs/heads/master@{#437040}
==========

[commit-bot: I haz the power, 2016-12-07 20:01:21]

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