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

Unified Diff: chrome/renderer/extensions/v8_schema_registry.cc

Issue 15091002: Lazily load API schemas from resource files and convert all APIs to features (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix chromeos tests Created 7 years, 6 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/renderer/extensions/v8_schema_registry.cc
diff --git a/chrome/renderer/extensions/v8_schema_registry.cc b/chrome/renderer/extensions/v8_schema_registry.cc
index 1a92bb1d325795e78adf150b509c925eb9571b85..2beed3091e95cebbc4e77e18a00c721a2bbc26c5 100644
--- a/chrome/renderer/extensions/v8_schema_registry.cc
+++ b/chrome/renderer/extensions/v8_schema_registry.cc
@@ -60,15 +60,15 @@ scoped_ptr<NativeHandler> V8SchemaRegistry::AsNativeHandler() {
}
v8::Handle<v8::Array> V8SchemaRegistry::GetSchemas(
- const std::set<std::string>& apis) {
+ const std::vector<std::string>& apis) {
v8::Isolate* isolate = v8::Isolate::GetCurrent();
v8::HandleScope handle_scope(isolate);
v8::Context::Scope context_scope(GetOrCreateContext(isolate));
v8::Handle<v8::Array> v8_apis(v8::Array::New(apis.size()));
size_t api_index = 0;
- for (std::set<std::string>::const_iterator i = apis.begin(); i != apis.end();
- ++i) {
+ for (std::vector<std::string>::const_iterator i = apis.begin();
+ i != apis.end(); ++i) {
v8_apis->Set(api_index++, GetSchema(*i));
}
return handle_scope.Close(v8_apis);
« no previous file with comments | « chrome/renderer/extensions/v8_schema_registry.h ('k') | chrome/renderer/resources/extensions/test_custom_bindings.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698