Chromium Code Reviews| Index: chrome/renderer/extensions/api_definitions_natives.cc |
| diff --git a/chrome/renderer/extensions/api_definitions_natives.cc b/chrome/renderer/extensions/api_definitions_natives.cc |
| index 37621acda6a25193cf5156c401d6af66e482c4f8..024e05fb6088ab851460771fea4d543f854a7b46 100644 |
| --- a/chrome/renderer/extensions/api_definitions_natives.cc |
| +++ b/chrome/renderer/extensions/api_definitions_natives.cc |
| @@ -4,9 +4,6 @@ |
| #include "chrome/renderer/extensions/api_definitions_natives.h" |
| -#include "chrome/common/extensions/api/extension_api.h" |
| -#include "chrome/renderer/extensions/user_script_slave.h" |
| - |
| namespace extensions { |
| ApiDefinitionsNatives::ApiDefinitionsNatives( |
| @@ -22,28 +19,8 @@ v8::Handle<v8::Value> ApiDefinitionsNatives::GetExtensionAPIDefinition( |
| ChromeV8Context* v8_context = |
| extension_dispatcher()->v8_context_set().GetCurrent(); |
| CHECK(v8_context); |
| - |
| - // TODO(kalman): This is being calculated twice, first in |
| - // ExtensionDispatcher then again here. It might as well be a property of |
| - // ChromeV8Context, however, this would require making ChromeV8Context take |
| - // an Extension rather than an extension ID. In itself this is fine, |
| - // however it does not play correctly with the "IsTestExtensionId" checks. |
| - // We need to remove that first. |
|
not at google - send to devlin
2012/04/10 20:42:02
yay!
|
| - scoped_ptr<std::set<std::string> > apis; |
| - |
| - const std::string& extension_id = v8_context->extension_id(); |
| - if (extension_dispatcher()->IsTestExtensionId(extension_id)) { |
| - apis.reset(new std::set<std::string>()); |
| - // The minimal set of APIs that tests need. |
| - apis->insert("extension"); |
| - } else { |
| - apis = ExtensionAPI::GetSharedInstance()->GetAPIsForContext( |
| - v8_context->context_type(), |
| - extension_dispatcher()->extensions()->GetByID(extension_id), |
| - UserScriptSlave::GetDataSourceURLForFrame(v8_context->web_frame())); |
| - } |
| - |
| - return extension_dispatcher()->v8_schema_registry()->GetSchemas(*apis); |
| + return extension_dispatcher()->v8_schema_registry()->GetSchemas( |
| + *v8_context->GetAvailableExtensionAPIs()); |
| } |
| } // namespace extensions |