summaryrefslogtreecommitdiffstats
path: root/chrome/browser/extensions/extensions_service.cc
diff options
context:
space:
mode:
authoraa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-07 10:26:31 +0000
committeraa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-07 10:26:31 +0000
commitb9a622b9fe09868974a5af845bc474752b239566 (patch)
tree939d1e3947e35bbd960a925992bc7f049fe5775f /chrome/browser/extensions/extensions_service.cc
parent72cbd32707a2ede460bcc1b3cb199e653282a8ed (diff)
downloadchromium_src-b9a622b9fe09868974a5af845bc474752b239566.zip
chromium_src-b9a622b9fe09868974a5af845bc474752b239566.tar.gz
chromium_src-b9a622b9fe09868974a5af845bc474752b239566.tar.bz2
Revert "Implement chromium.self in content scripts..."
This reverts commit 61ab30f52667e739602ab2af4fd8f2d8a0a2a2f0. Still seeing memory errors. Review URL: http://codereview.chromium.org/63056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13243 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/extensions_service.cc')
-rw-r--r--chrome/browser/extensions/extensions_service.cc24
1 files changed, 15 insertions, 9 deletions
diff --git a/chrome/browser/extensions/extensions_service.cc b/chrome/browser/extensions/extensions_service.cc
index 942d23c..5c836d0 100644
--- a/chrome/browser/extensions/extensions_service.cc
+++ b/chrome/browser/extensions/extensions_service.cc
@@ -258,9 +258,18 @@ void ExtensionsServiceBackend::LoadSingleExtension(
LOG(INFO) << "Loading single extension from " <<
WideToASCII(extension_path.BaseName().ToWStringHack());
- Extension* extension = LoadExtension(extension_path,
- false); // don't require ID
+ Extension* extension = LoadExtension(extension_path);
if (extension) {
+ if (extension->id().empty()) {
+ // Generate an ID
+ static int counter = 0;
+ std::string id = StringPrintf("%x", counter);
+ ++counter;
+
+ // pad the string out to 40 chars with zeroes.
+ id.insert(0, 40 - id.length(), '0');
+ extension->set_id(id);
+ }
ExtensionList* extensions = new ExtensionList;
extensions->push_back(extension);
ReportExtensionsLoaded(extensions);
@@ -281,12 +290,11 @@ Extension* ExtensionsServiceBackend::LoadExtensionCurrentVersion(
WideToASCII(extension_path.BaseName().ToWStringHack()) <<
" version: " << version_str;
- return LoadExtension(extension_path.AppendASCII(version_str),
- true); // require id
+ return LoadExtension(extension_path.AppendASCII(version_str));
}
Extension* ExtensionsServiceBackend::LoadExtension(
- const FilePath& extension_path, bool require_id) {
+ const FilePath& extension_path) {
FilePath manifest_path =
extension_path.AppendASCII(Extension::kManifestFilename);
if (!file_util::PathExists(manifest_path)) {
@@ -309,7 +317,7 @@ Extension* ExtensionsServiceBackend::LoadExtension(
scoped_ptr<Extension> extension(new Extension(extension_path));
if (!extension->InitFromValue(*static_cast<DictionaryValue*>(root.get()),
- require_id, &error)) {
+ &error)) {
ReportExtensionLoadError(extension_path, error);
return NULL;
}
@@ -630,9 +638,7 @@ bool ExtensionsServiceBackend::InstallOrUpdateExtension(
DictionaryValue* dict = manifest.get();
Extension extension;
std::string error;
- if (!extension.InitFromValue(*dict,
- true, // require ID
- &error)) {
+ if (!extension.InitFromValue(*dict, &error)) {
ReportExtensionInstallError(source_file,
"Invalid extension manifest.");
return false;