New issue
Advanced search Search tips

Issue 873031 link

Starred by 1 user
Status: Fixed
Owner:
tbarzic@chromium.org
Closed: Aug 20
Cc:
agawronska@chromium.org
r...@chromium.org
ovanieva@chromium.org
michae...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

[Demo mode] Cros component installer should attempt loading demo mode resources from preinstalled location

Project Member Reported by tbarzic@chromium.org, Aug 10 
When registering components, component installer first attempts to find existing installed components. It should be able to detect demo mode resources (and possibly other components) pre-installed on the stateful partition (so it can find the component without requiring download, provided that the demo mode reosources component is actually pre-installed) - this seems like a better alternative compared to demo mode code loading the pre-installed component directly if component installer fails to locate a version installed by component updater.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 16 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4098a1226af22637d1ff3332e431e19bb11258bf

commit 4098a1226af22637d1ff3332e431e19bb11258bf
Author: Toni Barzic <tbarzic@google.com>
Date: Thu Aug 16 19:49:25 2018

Load preinstalled components from stateful partition on Chrome OS

Sets /mnt/stateful_partition/unencrypted as root from which to load
preinstalled components on Chrome OS. (cros component installer will look
for components in /mnt/stateful_partition/cros-components/).

BUG= 873031 

Change-Id: Ifcf42c5b9a2af5463a6cb62c1f2e4e860c72cd7b
Reviewed-on: https://chromium-review.googlesource.com/1170347
Commit-Queue: Toni Baržić <tbarzic@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583777}
[modify] https://crrev.com/4098a1226af22637d1ff3332e431e19bb11258bf/chrome/app/chrome_main_delegate.cc
[modify] https://crrev.com/4098a1226af22637d1ff3332e431e19bb11258bf/chromeos/chromeos_paths.cc
[modify] https://crrev.com/4098a1226af22637d1ff3332e431e19bb11258bf/chromeos/chromeos_paths.h

Comment 2 by tbarzic@chromium.org, Aug 20

Status: Fixed (was: Started)
►

Sign in to add a comment