| Index: chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_unittest.cc
|
| diff --git a/chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_unittest.cc b/chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_unittest.cc
|
| index 70eb2e2f12509f0a20a7fb74721d9be81756032f..0db6afbd0f44bebd7e79894c6dae51d40c89069d 100644
|
| --- a/chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_unittest.cc
|
| +++ b/chrome/browser/extensions/api/declarative_webrequest/webrequest_condition_unittest.cc
|
| @@ -10,7 +10,6 @@
|
| #include "base/test/values_test_util.h"
|
| #include "base/values.h"
|
| #include "chrome/browser/extensions/api/declarative_webrequest/webrequest_constants.h"
|
| -#include "chrome/browser/extensions/api/declarative_webrequest/webrequest_rule.h"
|
| #include "chrome/common/extensions/matcher/url_matcher_constants.h"
|
| #include "content/public/browser/resource_request_info.h"
|
| #include "net/url_request/url_request_test_util.h"
|
| @@ -70,7 +69,7 @@ TEST(WebRequestConditionTest, CreateCondition) {
|
| ASSERT_TRUE(result.get());
|
|
|
| URLMatcherConditionSet::Vector url_matcher_condition_set;
|
| - url_matcher_condition_set.push_back(result->url_matcher_condition_set());
|
| + result->GetURLMatcherConditionSets(&url_matcher_condition_set);
|
| matcher.AddConditionSets(url_matcher_condition_set);
|
| std::set<URLMatcherConditionSet::ID> url_match_ids;
|
|
|
| @@ -83,7 +82,7 @@ TEST(WebRequestConditionTest, CreateCondition) {
|
| ResourceType::MAIN_FRAME, NULL, -1, -1);
|
| EXPECT_TRUE(result->IsFulfilled(
|
| url_match_ids,
|
| - WebRequestRule::RequestData(&match_request, ON_BEFORE_REQUEST)));
|
| + DeclarativeWebRequestData(&match_request, ON_BEFORE_REQUEST)));
|
|
|
| GURL https_url("https://www.example.com");
|
| net::TestURLRequest wrong_resource_type(https_url, NULL, &context);
|
| @@ -94,7 +93,7 @@ TEST(WebRequestConditionTest, CreateCondition) {
|
| ResourceType::SUB_FRAME, NULL, -1, -1);
|
| EXPECT_FALSE(result->IsFulfilled(
|
| url_match_ids,
|
| - WebRequestRule::RequestData(&wrong_resource_type, ON_BEFORE_REQUEST)));
|
| + DeclarativeWebRequestData(&wrong_resource_type, ON_BEFORE_REQUEST)));
|
| }
|
|
|
| // Conditions without UrlFilter attributes need to be independent of URL
|
| @@ -160,17 +159,17 @@ TEST(WebRequestConditionTest, NoUrlAttributes) {
|
| // attributes are fulfilled.
|
| EXPECT_FALSE(condition_no_url_false->IsFulfilled(
|
| dummy_match_set,
|
| - WebRequestRule::RequestData(&https_request, ON_BEFORE_REQUEST)));
|
| + DeclarativeWebRequestData(&https_request, ON_BEFORE_REQUEST)));
|
|
|
| EXPECT_TRUE(condition_no_url_true->IsFulfilled(
|
| dummy_match_set,
|
| - WebRequestRule::RequestData(&https_request, ON_BEFORE_REQUEST)));
|
| + DeclarativeWebRequestData(&https_request, ON_BEFORE_REQUEST)));
|
|
|
| // 2. An empty condition (in particular, without UrlFilter attributes) is
|
| // always fulfilled.
|
| EXPECT_TRUE(condition_empty->IsFulfilled(
|
| dummy_match_set,
|
| - WebRequestRule::RequestData(&https_request, ON_BEFORE_REQUEST)));
|
| + DeclarativeWebRequestData(&https_request, ON_BEFORE_REQUEST)));
|
| }
|
|
|
| TEST(WebRequestConditionTest, CreateConditionSet) {
|
| @@ -231,7 +230,7 @@ TEST(WebRequestConditionTest, CreateConditionSet) {
|
| EXPECT_TRUE(result->IsFulfilled(
|
| *(url_match_ids.begin()),
|
| url_match_ids,
|
| - WebRequestRule::RequestData(&http_request, ON_BEFORE_REQUEST)));
|
| + DeclarativeWebRequestData(&http_request, ON_BEFORE_REQUEST)));
|
|
|
| GURL https_url("https://www.example.com");
|
| url_match_ids = matcher.MatchURL(https_url);
|
| @@ -240,7 +239,7 @@ TEST(WebRequestConditionTest, CreateConditionSet) {
|
| EXPECT_TRUE(result->IsFulfilled(
|
| *(url_match_ids.begin()),
|
| url_match_ids,
|
| - WebRequestRule::RequestData(&https_request, ON_BEFORE_REQUEST)));
|
| + DeclarativeWebRequestData(&https_request, ON_BEFORE_REQUEST)));
|
|
|
| // Check that both, hostPrefix and hostSuffix are evaluated.
|
| GURL https_foo_url("https://foo.example.com");
|
| @@ -250,7 +249,7 @@ TEST(WebRequestConditionTest, CreateConditionSet) {
|
| EXPECT_FALSE(result->IsFulfilled(
|
| -1,
|
| url_match_ids,
|
| - WebRequestRule::RequestData(&https_foo_request, ON_BEFORE_REQUEST)));
|
| + DeclarativeWebRequestData(&https_foo_request, ON_BEFORE_REQUEST)));
|
| }
|
|
|
| TEST(WebRequestConditionTest, TestPortFilter) {
|
|
|