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

Unified Diff: chrome/common/extensions/api/extension_api.cc

Issue 11571014: Lazy load chrome.* APIs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more progress Created 7 years, 11 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/common/extensions/api/extension_api.cc
diff --git a/chrome/common/extensions/api/extension_api.cc b/chrome/common/extensions/api/extension_api.cc
index 85207e7325c89a18c708f333ad1dabe98e395cc7..db6e9b46b6204b6721085fbd9ff6ee410d71a524 100644
--- a/chrome/common/extensions/api/extension_api.cc
+++ b/chrome/common/extensions/api/extension_api.cc
@@ -583,7 +583,7 @@ void RemoveDisallowedAPIs(const Extension& extension,
} // namespace
-scoped_ptr<std::set<std::string> > ExtensionAPI::GetAPIsForContext(
+std::set<std::string> ExtensionAPI::GetAPIsForContext(
Feature::Context context, const Extension* extension, const GURL& url) {
// We're forced to load all schemas now because we need to know the metadata
// about every API -- and the metadata is stored in the schemas themselves.
@@ -641,17 +641,28 @@ scoped_ptr<std::set<std::string> > ExtensionAPI::GetAPIsForContext(
// Filter out all non-API features and remove the feature type part of the
// name.
- scoped_ptr<std::set<std::string> > result(new std::set<std::string>());
+ std::set<std::string> result;
for (std::set<std::string>::iterator iter = temp_result.begin();
iter != temp_result.end(); ++iter) {
std::string feature_type;
std::string feature_name;
SplitDependencyName(*iter, &feature_type, &feature_name);
if (feature_type == "api")
- result->insert(feature_name);
+ result.insert(feature_name);
}
- return result.Pass();
+ return result;
+}
+
+std::set<std::string> ExtensionAPI::GetAllAPINames() {
+ std::set<std::string> result;
+ for (SchemaMap::iterator i = schemas_.begin(); i != schemas_.end(); ++i)
+ result.insert(i->first);
+ for (UnloadedSchemaMap::iterator i = unloaded_schemas_.begin();
+ i != unloaded_schemas_.end(); ++i) {
+ result.insert(i->first);
+ }
+ return result;
}
Feature* ExtensionAPI::GetFeature(const std::string& full_name) {

Powered by Google App Engine
This is Rietveld 408576698