Chromium Code Reviews| Index: chrome/renderer/extensions/app_bindings.cc |
| diff --git a/chrome/renderer/extensions/app_bindings.cc b/chrome/renderer/extensions/app_bindings.cc |
| index 87d63ec24f1c74288b5eed81ed42037296a16945..dc2213d52a59d9eb87fe9fec5165b4b7e8e47964 100644 |
| --- a/chrome/renderer/extensions/app_bindings.cc |
| +++ b/chrome/renderer/extensions/app_bindings.cc |
| @@ -55,7 +55,6 @@ const char* kInvalidCallbackIdError = "Invalid callbackId"; |
| } // namespace |
| - |
| AppBindings::AppBindings(ExtensionDispatcher* dispatcher, |
| ChromeV8Context* context) |
| : ChromeV8Extension(dispatcher), |
| @@ -72,7 +71,6 @@ AppBindings::AppBindings(ExtensionDispatcher* dispatcher, |
| base::Bind(&AppBindings::GetAppNotifyChannel, base::Unretained(this))); |
| } |
| - |
| v8::Handle<v8::Value> AppBindings::GetIsInstalled( |
| const v8::Arguments& args) { |
| // TODO(aa): Hm, maybe ExtensionBindingsContext should have GetExtension() |
| @@ -81,9 +79,8 @@ v8::Handle<v8::Value> AppBindings::GetIsInstalled( |
| extension_dispatcher_->extensions()->GetByID(context_->extension_id()); |
| // TODO(aa): Why only hosted app? |
| - // TODO(aa): GARRR - why is there IsExtensionActive and IsApplicationActive!? |
|
Aaron Boodman
2012/03/06 01:17:21
:) - Thanks I really appreciate these little clean
|
| bool result = extension && extension->is_hosted_app() && |
| - extension_dispatcher_->IsApplicationActive(extension->id()); |
| + extension_dispatcher_->IsExtensionActive(extension->id()); |
| return v8::Boolean::New(result); |
| } |