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

Unified Diff: chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_attribute.h

Issue 10012004: Implemented proper support for checking schemes and requested resource types. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Renamed 'scheme' to 'schemes' Created 8 years, 8 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/declarative_webrequest/webrequest_condition_attribute.h
diff --git a/chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_attribute.h b/chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_attribute.h
index d0684bf34f136cf2ff2fa14ab80e99f0959f959a..26fdf871ed2e3864732a1e75a0af1821e6756d0c 100644
--- a/chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_attribute.h
+++ b/chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_attribute.h
@@ -14,6 +14,7 @@
#include "base/memory/scoped_ptr.h"
#include "chrome/browser/extensions/api/declarative_webrequest/request_stages.h"
#include "chrome/common/extensions/api/experimental.declarative.h"
+#include "webkit/glue/resource_type.h"
namespace base {
class Value;
@@ -30,7 +31,7 @@ namespace extensions {
class WebRequestConditionAttribute {
public:
enum Type {
- CONDITION_HAS_SCHEME
+ CONDITION_RESOURCE_TYPE
};
WebRequestConditionAttribute();
@@ -71,13 +72,11 @@ typedef std::vector<linked_ptr<WebRequestConditionAttribute> >
// The following are concrete condition attributes.
//
-// Condition that checks whether a URL has a specific scheme.
-// TODO(battre): Generalize this to allow checking for multiple schemes.
-// TODO(battre): Alternatively, move the scheme check into the URLMatcher.
-class WebRequestConditionAttributeHasScheme
+// Condition that checks whether a request is for a specific resource type.
+class WebRequestConditionAttributeResourceType
: public WebRequestConditionAttribute {
public:
- virtual ~WebRequestConditionAttributeHasScheme();
+ virtual ~WebRequestConditionAttributeResourceType();
static bool IsMatchingType(const std::string& instance_type);
@@ -94,11 +93,12 @@ class WebRequestConditionAttributeHasScheme
virtual Type GetType() const OVERRIDE;
private:
- explicit WebRequestConditionAttributeHasScheme(const std::string& pattern);
+ explicit WebRequestConditionAttributeResourceType(
+ const std::vector<ResourceType::Type>& types);
- std::string pattern_;
+ std::vector<ResourceType::Type> types_;
- DISALLOW_COPY_AND_ASSIGN(WebRequestConditionAttributeHasScheme);
+ DISALLOW_COPY_AND_ASSIGN(WebRequestConditionAttributeResourceType);
};
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698