Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(36)

Unified Diff: chrome/browser/component_updater/cros_component_installer.cc

Issue 2898063002: Hold cros_component_installers in browser_process. (Closed)
Patch Set: new line between 5/6 Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/component_updater/cros_component_installer.cc
diff --git a/chrome/browser/component_updater/cros_component_installer.cc b/chrome/browser/component_updater/cros_component_installer.cc
index b68cd03f14e51a67945e8b6a09eaa450bfb6cfc5..faf0ca2113a24460f643b05d6f6653744c56bf70 100644
--- a/chrome/browser/component_updater/cros_component_installer.cc
+++ b/chrome/browser/component_updater/cros_component_installer.cc
@@ -95,7 +95,9 @@ CrOSComponentInstallerTraits::OnCustomInstall(
void CrOSComponentInstallerTraits::ComponentReady(
const base::Version& version,
const base::FilePath& path,
- std::unique_ptr<base::DictionaryValue> manifest) {}
+ std::unique_ptr<base::DictionaryValue> manifest) {
+ g_browser_process->platform_part()->AddCompatibleCrOSComponent(GetName());
+}
bool CrOSComponentInstallerTraits::VerifyInstallation(
const base::DictionaryValue& manifest,
@@ -153,7 +155,7 @@ bool CrOSComponent::InstallCrOSComponent(
auto* const cus = g_browser_process->component_updater();
const ConfigMap components = {
{"epson-inkjet-printer-escpr",
- {{"env_version", "0.0"},
+ {{"env_version", "2.0"},
waffles 2017/05/24 21:14:35 Did you mean to change this line?
xiaochu 2017/05/24 21:35:37 Done. Sorry about that. I'm saving this to next C
{"sha2hashstr",
"1913a5e0a6cad30b6f03e176177e0d7ed62c5d6700a9c66da556d7c3f5d6a47e"}}}};
if (name.empty()) {

Powered by Google App Engine
This is Rietveld 408576698