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

Unified Diff: chrome/browser/extensions/api/declarative_webrequest/webrequest_rule.cc

Issue 9820003: Implementation of beginning of Declarative Web Request API backend (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pacify clang Created 8 years, 9 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_rule.cc
diff --git a/chrome/browser/extensions/api/declarative_webrequest/webrequest_rule.cc b/chrome/browser/extensions/api/declarative_webrequest/webrequest_rule.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d08f76963a92a7a26f6eb89b5a6bb644d47ac594
--- /dev/null
+++ b/chrome/browser/extensions/api/declarative_webrequest/webrequest_rule.cc
@@ -0,0 +1,66 @@
+// 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.
+
+#include "chrome/browser/extensions/api/declarative_webrequest/webrequest_rule.h"
+
+#include "base/logging.h"
+#include "chrome/browser/extensions/api/declarative_webrequest/webrequest_action.h"
+#include "chrome/browser/extensions/api/declarative_webrequest/webrequest_condition.h"
+
+namespace extensions {
+
+WebRequestRule::WebRequestRule(
+ const GlobalRuleId& id,
+ scoped_ptr<WebRequestConditionSet> conditions,
+ scoped_ptr<WebRequestActionSet> actions)
+ : id_(id),
+ conditions_(conditions.release()),
+ actions_(actions.release()) {
+ CHECK(conditions_.get());
+ CHECK(actions_.get());
+}
+
+WebRequestRule::~WebRequestRule() {}
+
+// static
+bool WebRequestRule::CheckConsistency(
+ WebRequestConditionSet* conditions,
+ WebRequestActionSet* actions,
+ std::string* error) {
+ // TODO(battre): Implement this.
+ return true;
+}
+
+// static
+scoped_ptr<WebRequestRule> WebRequestRule::Create(
+ URLMatcherConditionFactory* url_matcher_condition_factory,
+ const std::string& extension_id,
+ linked_ptr<RulesRegistry::Rule> rule,
+ std::string* error) {
+ scoped_ptr<WebRequestRule> error_result;
+
+ scoped_ptr<WebRequestConditionSet> conditions =
+ WebRequestConditionSet::Create(
+ url_matcher_condition_factory, rule->conditions, error);
+ if (!error->empty())
+ return error_result.Pass();
+ CHECK(conditions.get());
+
+ scoped_ptr<WebRequestActionSet> actions =
+ WebRequestActionSet::Create(rule->actions, error);
+ if (!error->empty())
+ return error_result.Pass();
+ CHECK(actions.get());
+
+ if (!CheckConsistency(conditions.get(), actions.get(), error)) {
+ DCHECK(!error->empty());
+ return error_result.Pass();
+ }
+
+ GlobalRuleId rule_id = make_pair(extension_id, *(rule->id));
+ return scoped_ptr<WebRequestRule>(
+ new WebRequestRule(rule_id, conditions.Pass(), actions.Pass()));
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698