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

Unified Diff: chrome/browser/extensions/api/api_resource.cc

Issue 10919201: Make sure that a given app/extension requests only its own resources. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Improve tests; fix merge conflicts. Created 8 years, 3 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/browser/extensions/api/api_resource.cc
diff --git a/chrome/browser/extensions/api/api_resource.cc b/chrome/browser/extensions/api/api_resource.cc
index 400acded39f77ce83bd8bc6b5c4cdcd9b3d05f77..19b7b9137b0c8eab80f631499378e82a4df4420e 100644
--- a/chrome/browser/extensions/api/api_resource.cc
+++ b/chrome/browser/extensions/api/api_resource.cc
@@ -7,8 +7,15 @@
namespace extensions {
-ApiResource::ApiResource(ApiResourceEventNotifier* event_notifier)
- : event_notifier_(event_notifier) {
+ApiResource::ApiResource(const std::string& owner_extension_id,
+ ApiResourceEventNotifier* event_notifier)
+ : owner_extension_id_(owner_extension_id),
+ event_notifier_(event_notifier) {
+
+ CHECK(!owner_extension_id_.empty());
+ if (event_notifier)
+ CHECK(event_notifier->src_extension_id() == owner_extension_id_);
+
// scoped_refptr<> constructor does the initial AddRef() for us on
// event_notifier_.
}
« no previous file with comments | « chrome/browser/extensions/api/api_resource.h ('k') | chrome/browser/extensions/api/api_resource_event_notifier.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698