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

Side by Side Diff: chrome/browser/component_updater/default_component_installer.cc

Issue 385013002: Componentize component_updater: Replace content::BrowserThread usage with task runners (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/bind.h" 5 #include "base/bind.h"
6 #include "base/file_util.h" 6 #include "base/file_util.h"
7 #include "base/files/file_enumerator.h" 7 #include "base/files/file_enumerator.h"
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/location.h"
10 #include "base/message_loop/message_loop_proxy.h"
11 #include "base/sequenced_task_runner.h"
9 #include "base/values.h" 12 #include "base/values.h"
10 #include "base/version.h" 13 #include "base/version.h"
11 // TODO(ddorwin): Find a better place for ReadManifest. 14 // TODO(ddorwin): Find a better place for ReadManifest.
12 #include "chrome/browser/component_updater/component_unpacker.h" 15 #include "chrome/browser/component_updater/component_unpacker.h"
13 #include "chrome/browser/component_updater/default_component_installer.h" 16 #include "chrome/browser/component_updater/default_component_installer.h"
14 #include "content/public/browser/browser_thread.h"
15 17
16 namespace component_updater { 18 namespace component_updater {
17 19
18 namespace { 20 namespace {
19 // Version "0" corresponds to no installed version. By the server's conventions, 21 // Version "0" corresponds to no installed version. By the server's conventions,
20 // we represent it as a dotted quad. 22 // we represent it as a dotted quad.
21 const char kNullVersion[] = "0.0.0.0"; 23 const char kNullVersion[] = "0.0.0.0";
22 } // namespace 24 } // namespace
23 25
24 ComponentInstallerTraits::~ComponentInstallerTraits() { 26 ComponentInstallerTraits::~ComponentInstallerTraits() {
25 } 27 }
26 28
27 DefaultComponentInstaller::DefaultComponentInstaller( 29 DefaultComponentInstaller::DefaultComponentInstaller(
28 scoped_ptr<ComponentInstallerTraits> installer_traits) 30 scoped_ptr<ComponentInstallerTraits> installer_traits,
29 : current_version_(kNullVersion) { 31 scoped_refptr<base::SequencedTaskRunner> task_runner)
32 : current_version_(kNullVersion),
33 task_runner_(task_runner),
34 main_loop_(base::MessageLoopProxy::current()) {
30 installer_traits_ = installer_traits.Pass(); 35 installer_traits_ = installer_traits.Pass();
31 } 36 }
32 37
33 DefaultComponentInstaller::~DefaultComponentInstaller() { 38 DefaultComponentInstaller::~DefaultComponentInstaller() {
39 DCHECK(thread_checker_.CalledOnValidThread());
34 } 40 }
35 41
36 void DefaultComponentInstaller::Register(ComponentUpdateService* cus) { 42 void DefaultComponentInstaller::Register(ComponentUpdateService* cus) {
43 DCHECK(thread_checker_.CalledOnValidThread());
37 if (!installer_traits_) { 44 if (!installer_traits_) {
38 NOTREACHED() << "A DefaultComponentInstaller has been created but " 45 NOTREACHED() << "A DefaultComponentInstaller has been created but "
39 << "has no installer traits."; 46 << "has no installer traits.";
40 return; 47 return;
41 } 48 }
42 content::BrowserThread::PostBlockingPoolTask( 49 task_runner_->PostTask(
43 FROM_HERE, 50 FROM_HERE,
44 base::Bind(&DefaultComponentInstaller::StartRegistration, 51 base::Bind(&DefaultComponentInstaller::StartRegistration,
45 base::Unretained(this), 52 base::Unretained(this),
46 cus)); 53 cus));
47 } 54 }
48 55
49 void DefaultComponentInstaller::OnUpdateError(int error) { 56 void DefaultComponentInstaller::OnUpdateError(int error) {
50 NOTREACHED() << "Component update error: " << error; 57 NOTREACHED() << "Component update error: " << error;
51 } 58 }
52 59
53 bool DefaultComponentInstaller::InstallHelper( 60 bool DefaultComponentInstaller::InstallHelper(
54 const base::DictionaryValue& manifest, 61 const base::DictionaryValue& manifest,
55 const base::FilePath& unpack_path, 62 const base::FilePath& unpack_path,
56 const base::FilePath& install_path) { 63 const base::FilePath& install_path) {
57 if (!base::Move(unpack_path, install_path)) 64 if (!base::Move(unpack_path, install_path))
58 return false; 65 return false;
59 if (!installer_traits_->OnCustomInstall(manifest, install_path)) 66 if (!installer_traits_->OnCustomInstall(manifest, install_path))
60 return false; 67 return false;
61 if (!installer_traits_->VerifyInstallation(install_path)) 68 if (!installer_traits_->VerifyInstallation(install_path))
62 return false; 69 return false;
63 return true; 70 return true;
64 } 71 }
65 72
66 bool DefaultComponentInstaller::Install(const base::DictionaryValue& manifest, 73 bool DefaultComponentInstaller::Install(const base::DictionaryValue& manifest,
67 const base::FilePath& unpack_path) { 74 const base::FilePath& unpack_path) {
75 DCHECK(task_runner_->RunsTasksOnCurrentThread());
68 std::string manifest_version; 76 std::string manifest_version;
69 manifest.GetStringASCII("version", &manifest_version); 77 manifest.GetStringASCII("version", &manifest_version);
70 base::Version version(manifest_version.c_str()); 78 base::Version version(manifest_version.c_str());
71 if (!version.IsValid()) 79 if (!version.IsValid())
72 return false; 80 return false;
73 if (current_version_.CompareTo(version) > 0) 81 if (current_version_.CompareTo(version) > 0)
74 return false; 82 return false;
75 base::FilePath install_path = 83 base::FilePath install_path =
76 installer_traits_->GetBaseDirectory().AppendASCII(version.GetString()); 84 installer_traits_->GetBaseDirectory().AppendASCII(version.GetString());
77 if (base::PathExists(install_path)) { 85 if (base::PathExists(install_path)) {
78 if (!base::DeleteFile(install_path, true)) 86 if (!base::DeleteFile(install_path, true))
79 return false; 87 return false;
80 } 88 }
81 if (!InstallHelper(manifest, unpack_path, install_path)) { 89 if (!InstallHelper(manifest, unpack_path, install_path)) {
82 base::DeleteFile(install_path, true); 90 base::DeleteFile(install_path, true);
83 return false; 91 return false;
84 } 92 }
85 current_version_ = version; 93 current_version_ = version;
86 // TODO(ddorwin): Change the parameter to scoped_ptr<base::DictionaryValue> 94 // TODO(ddorwin): Change the parameter to scoped_ptr<base::DictionaryValue>
87 // so we can avoid this DeepCopy. 95 // so we can avoid this DeepCopy.
88 current_manifest_.reset(manifest.DeepCopy()); 96 current_manifest_.reset(manifest.DeepCopy());
89 scoped_ptr<base::DictionaryValue> manifest_copy( 97 scoped_ptr<base::DictionaryValue> manifest_copy(
90 current_manifest_->DeepCopy()); 98 current_manifest_->DeepCopy());
91 content::BrowserThread::PostTask( 99 main_loop_->PostTask(FROM_HERE,
92 content::BrowserThread::UI, 100 base::Bind(&ComponentInstallerTraits::ComponentReady,
93 FROM_HERE, 101 base::Unretained(installer_traits_.get()),
94 base::Bind(&ComponentInstallerTraits::ComponentReady, 102 current_version_,
95 base::Unretained(installer_traits_.get()), 103 GetInstallDirectory(),
96 current_version_, 104 base::Passed(&manifest_copy)));
97 GetInstallDirectory(),
98 base::Passed(&manifest_copy)));
99 return true; 105 return true;
100 } 106 }
101 107
102 bool DefaultComponentInstaller::GetInstalledFile( 108 bool DefaultComponentInstaller::GetInstalledFile(
103 const std::string& file, 109 const std::string& file,
104 base::FilePath* installed_file) { 110 base::FilePath* installed_file) {
105 if (current_version_.Equals(base::Version(kNullVersion))) 111 if (current_version_.Equals(base::Version(kNullVersion)))
106 return false; // No component has been installed yet. 112 return false; // No component has been installed yet.
107 113
108 *installed_file = installer_traits_->GetBaseDirectory() 114 *installed_file = installer_traits_->GetBaseDirectory()
109 .AppendASCII(current_version_.GetString()) 115 .AppendASCII(current_version_.GetString())
110 .AppendASCII(file); 116 .AppendASCII(file);
111 return true; 117 return true;
112 } 118 }
113 119
114 void DefaultComponentInstaller::StartRegistration(ComponentUpdateService* cus) { 120 void DefaultComponentInstaller::StartRegistration(ComponentUpdateService* cus) {
121 DCHECK(task_runner_->RunsTasksOnCurrentThread());
115 base::FilePath base_dir = installer_traits_->GetBaseDirectory(); 122 base::FilePath base_dir = installer_traits_->GetBaseDirectory();
116 if (!base::PathExists(base_dir) && !base::CreateDirectory(base_dir)) { 123 if (!base::PathExists(base_dir) && !base::CreateDirectory(base_dir)) {
117 NOTREACHED() << "Could not create the base directory for " 124 NOTREACHED() << "Could not create the base directory for "
118 << installer_traits_->GetName() << " (" 125 << installer_traits_->GetName() << " ("
119 << base_dir.MaybeAsASCII() << ")."; 126 << base_dir.MaybeAsASCII() << ").";
120 return; 127 return;
121 } 128 }
122 129
123 base::FilePath latest_dir; 130 base::FilePath latest_dir;
124 base::Version latest_version(kNullVersion); 131 base::Version latest_version(kNullVersion);
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
169 } 176 }
170 177
171 // Remove older versions of the component. None should be in use during 178 // Remove older versions of the component. None should be in use during
172 // browser startup. 179 // browser startup.
173 for (std::vector<base::FilePath>::iterator iter = older_dirs.begin(); 180 for (std::vector<base::FilePath>::iterator iter = older_dirs.begin();
174 iter != older_dirs.end(); 181 iter != older_dirs.end();
175 ++iter) { 182 ++iter) {
176 base::DeleteFile(*iter, true); 183 base::DeleteFile(*iter, true);
177 } 184 }
178 185
179 content::BrowserThread::PostTask( 186 main_loop_->PostTask(
180 content::BrowserThread::UI,
181 FROM_HERE, 187 FROM_HERE,
182 base::Bind(&DefaultComponentInstaller::FinishRegistration, 188 base::Bind(&DefaultComponentInstaller::FinishRegistration,
183 base::Unretained(this), 189 base::Unretained(this),
184 cus)); 190 cus));
185 } 191 }
186 192
187 base::FilePath DefaultComponentInstaller::GetInstallDirectory() { 193 base::FilePath DefaultComponentInstaller::GetInstallDirectory() {
188 return installer_traits_->GetBaseDirectory() 194 return installer_traits_->GetBaseDirectory()
189 .AppendASCII(current_version_.GetString()); 195 .AppendASCII(current_version_.GetString());
190 } 196 }
191 197
192 void DefaultComponentInstaller::FinishRegistration( 198 void DefaultComponentInstaller::FinishRegistration(
193 ComponentUpdateService* cus) { 199 ComponentUpdateService* cus) {
194 DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI)); 200 DCHECK(thread_checker_.CalledOnValidThread());
195 if (installer_traits_->CanAutoUpdate()) { 201 if (installer_traits_->CanAutoUpdate()) {
196 CrxComponent crx; 202 CrxComponent crx;
197 crx.name = installer_traits_->GetName(); 203 crx.name = installer_traits_->GetName();
198 crx.installer = this; 204 crx.installer = this;
199 crx.version = current_version_; 205 crx.version = current_version_;
200 crx.fingerprint = current_fingerprint_; 206 crx.fingerprint = current_fingerprint_;
201 installer_traits_->GetHash(&crx.pk_hash); 207 installer_traits_->GetHash(&crx.pk_hash);
202 ComponentUpdateService::Status status = cus->RegisterComponent(crx); 208 ComponentUpdateService::Status status = cus->RegisterComponent(crx);
203 if (status != ComponentUpdateService::kOk && 209 if (status != ComponentUpdateService::kOk &&
204 status != ComponentUpdateService::kReplaced) { 210 status != ComponentUpdateService::kReplaced) {
205 NOTREACHED() << "Component registration failed for " 211 NOTREACHED() << "Component registration failed for "
206 << installer_traits_->GetName(); 212 << installer_traits_->GetName();
207 return; 213 return;
208 } 214 }
209 } 215 }
210 216
211 if (current_version_.CompareTo(base::Version(kNullVersion)) > 0) { 217 if (current_version_.CompareTo(base::Version(kNullVersion)) > 0) {
212 scoped_ptr<base::DictionaryValue> manifest_copy( 218 scoped_ptr<base::DictionaryValue> manifest_copy(
213 current_manifest_->DeepCopy()); 219 current_manifest_->DeepCopy());
214 installer_traits_->ComponentReady( 220 installer_traits_->ComponentReady(
215 current_version_, GetInstallDirectory(), manifest_copy.Pass()); 221 current_version_, GetInstallDirectory(), manifest_copy.Pass());
216 } 222 }
217 } 223 }
218 224
219 } // namespace component_updater 225 } // namespace component_updater
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698