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

Unified Diff: chrome/common/extensions/event_matcher.h

Issue 10388135: Add common code for extensions event filtering. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: respond to comments Created 8 years, 7 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/event_matcher.h
diff --git a/chrome/common/extensions/event_matcher.h b/chrome/common/extensions/event_matcher.h
new file mode 100644
index 0000000000000000000000000000000000000000..dff6bb2a8abee238e553713bf5958391d54c1611
--- /dev/null
+++ b/chrome/common/extensions/event_matcher.h
@@ -0,0 +1,50 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_COMMON_EXTENSIONS_EVENT_MATCHER_H_
+#define CHROME_COMMON_EXTENSIONS_EVENT_MATCHER_H_
+#pragma once
+
+#include "base/memory/scoped_ptr.h"
+#include "base/values.h"
+
+namespace extensions {
+
+class EventFilteringInfo;
+
+// Matches EventFilteringInfos against a set of criteria. This is intended to
+// be used by EventRouter which performs efficient URL matching across
+// potentially many EventMatchers itself. This is why this class only exposes
+// MatchNonURLCriteria() - URL matching is handled by EventRouter.
+class EventMatcher {
Matt Perry 2012/05/16 00:37:32 Maybe FilteredEventMatcher (and likewise FilteredE
battre 2012/05/16 13:35:12 Or EventFilter?
koz (OOO until 15th September) 2012/05/17 05:08:42 EventFilter is a good one, I'll go with that. (I d
+ public:
+ EventMatcher();
+ ~EventMatcher();
+
+ // Returns true if |event_info| satisfies this matcher's criteria, not taking
+ // into consideration any URL criteria.
+ bool MatchNonURLCriteria(const EventFilteringInfo& event_info) const;
+
+ void set_url_filters(scoped_ptr<base::ListValue> url_filters) {
+ url_filters_ = url_filters.Pass();
+ }
+
+ // Returns NULL if no url_filters have been specified.
+ base::ListValue* url_filters() const {
+ return url_filters_.get();
+ }
+
+ bool has_url_filters() const {
+ return url_filters_.get() && !url_filters_->empty();
+ }
+
+ private:
+ scoped_ptr<base::ListValue> url_filters_;
+
+ DISALLOW_COPY_AND_ASSIGN(EventMatcher);
+};
+
+} // namespace extensions
+
+#endif // CHROME_COMMON_EXTENSIONS_EVENT_MATCHER_H_

Powered by Google App Engine
This is Rietveld 408576698