New issue
Advanced search Search tips

Issue 879013 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocked on:
issue 878660

Blocking:
issue 812040



Sign in to add a comment

Land demo resources in demo resources image

Project Member Reported by michae...@chromium.org, Aug 30

Issue description

Add demo resources to demo_mode_resources to support offline Demo Mode.

Most of these resources will be used in the CrOS component that Chrome downloads when enrolling into Demo Mode.

All of these resources will be used in the offline Demo Mode Resources image installed in the factory flow.
 
Blockedon: 878660
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 30

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/overlays/chromeos-overlay/+/1c6ce49f83866ac78eb9b26f130f7224b31f60dc

commit 1c6ce49f83866ac78eb9b26f130f7224b31f60dc
Author: Michael Giuffrida <michaelpg@chromium.org>
Date: Thu Aug 30 21:42:04 2018

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 28

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/55f2f2ba1986725cfbc8155e5fc233b05ba77d7e

commit 55f2f2ba1986725cfbc8155e5fc233b05ba77d7e
Author: Michael Giuffrida <michaelpg@chromium.org>
Date: Fri Sep 28 02:45:20 2018

cros_component: Update demo_mode_resources directories

We should include media in the component (for photos). We shouldn't
include chrome_apps (since they will be downloaded from the web store),
demo_extensions.json (only needed to import chrome apps), or policy
(only used in offline-enrolled Demo Mode).

BUG= chromium:879013 
TEST=build chromeos-base/demo_mode_resources component, it should have:
       - media/
       - android_demo_apps.squash
       - manifest.json

Change-Id: I50639b25e2daad1aa73ba3b918c0225adf42a6f4
Reviewed-on: https://chromium-review.googlesource.com/1244902
Commit-Ready: Michael Giuffrida <michaelpg@chromium.org>
Tested-by: Michael Giuffrida <michaelpg@chromium.org>
Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>

[modify] https://crrev.com/55f2f2ba1986725cfbc8155e5fc233b05ba77d7e/cros_component/cros_components.json

Cc: -tbarzic@chromium.org
Labels: M-73

Comment 5 by michae...@chromium.org, Jan 19 (3 days ago)

Status: Fixed (was: Started)

Sign in to add a comment