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

Unified Diff: extensions/browser/extension_protocols_unittest.cc

Issue 425653002: content: ResourceType cleanup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: REBASE Created 6 years, 5 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
« no previous file with comments | « extensions/browser/extension_protocols.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/extension_protocols_unittest.cc
diff --git a/extensions/browser/extension_protocols_unittest.cc b/extensions/browser/extension_protocols_unittest.cc
index 02d9fdf65f461dd2fecf5233df2800f81ec160c7..e2f0607f3f79ebeec0cec33b9fccc57fcc0c3268 100644
--- a/extensions/browser/extension_protocols_unittest.cc
+++ b/extensions/browser/extension_protocols_unittest.cc
@@ -120,7 +120,7 @@ class ExtensionProtocolTest : public testing::Test {
}
void StartRequest(net::URLRequest* request,
- ResourceType::Type resource_type) {
+ ResourceType resource_type) {
content::ResourceRequestInfo::AllocateForTesting(request,
resource_type,
&resource_context_,
@@ -181,7 +181,7 @@ TEST_F(ExtensionProtocolTest, IncognitoRequest) {
net::DEFAULT_PRIORITY,
&test_delegate_,
resource_context_.GetRequestContext());
- StartRequest(&request, ResourceType::MAIN_FRAME);
+ StartRequest(&request, content::RESOURCE_TYPE_MAIN_FRAME);
EXPECT_EQ(net::URLRequestStatus::FAILED, request.status().status());
if (cases[i].should_allow_main_frame_load) {
@@ -199,7 +199,7 @@ TEST_F(ExtensionProtocolTest, IncognitoRequest) {
net::DEFAULT_PRIORITY,
&test_delegate_,
resource_context_.GetRequestContext());
- StartRequest(&request, ResourceType::SUB_FRAME);
+ StartRequest(&request, content::RESOURCE_TYPE_SUB_FRAME);
EXPECT_EQ(net::URLRequestStatus::FAILED, request.status().status());
if (cases[i].should_allow_sub_frame_load) {
@@ -241,7 +241,7 @@ TEST_F(ExtensionProtocolTest, ComponentResourceRequest) {
net::DEFAULT_PRIORITY,
&test_delegate_,
resource_context_.GetRequestContext());
- StartRequest(&request, ResourceType::MEDIA);
+ StartRequest(&request, content::RESOURCE_TYPE_MEDIA);
EXPECT_EQ(net::URLRequestStatus::SUCCESS, request.status().status());
CheckForContentLengthHeader(&request);
}
@@ -254,7 +254,7 @@ TEST_F(ExtensionProtocolTest, ComponentResourceRequest) {
net::DEFAULT_PRIORITY,
&test_delegate_,
resource_context_.GetRequestContext());
- StartRequest(&request, ResourceType::MEDIA);
+ StartRequest(&request, content::RESOURCE_TYPE_MEDIA);
EXPECT_EQ(net::URLRequestStatus::SUCCESS, request.status().status());
CheckForContentLengthHeader(&request);
}
@@ -277,7 +277,7 @@ TEST_F(ExtensionProtocolTest, ResourceRequestResponseHeaders) {
net::DEFAULT_PRIORITY,
&test_delegate_,
resource_context_.GetRequestContext());
- StartRequest(&request, ResourceType::MEDIA);
+ StartRequest(&request, content::RESOURCE_TYPE_MEDIA);
EXPECT_EQ(net::URLRequestStatus::SUCCESS, request.status().status());
// Check that cache-related headers are set.
@@ -316,7 +316,7 @@ TEST_F(ExtensionProtocolTest, AllowFrameRequests) {
net::DEFAULT_PRIORITY,
&test_delegate_,
resource_context_.GetRequestContext());
- StartRequest(&request, ResourceType::MAIN_FRAME);
+ StartRequest(&request, content::RESOURCE_TYPE_MAIN_FRAME);
EXPECT_EQ(net::URLRequestStatus::SUCCESS, request.status().status());
}
{
@@ -324,7 +324,7 @@ TEST_F(ExtensionProtocolTest, AllowFrameRequests) {
net::DEFAULT_PRIORITY,
&test_delegate_,
resource_context_.GetRequestContext());
- StartRequest(&request, ResourceType::SUB_FRAME);
+ StartRequest(&request, content::RESOURCE_TYPE_SUB_FRAME);
EXPECT_EQ(net::URLRequestStatus::SUCCESS, request.status().status());
}
@@ -334,7 +334,7 @@ TEST_F(ExtensionProtocolTest, AllowFrameRequests) {
net::DEFAULT_PRIORITY,
&test_delegate_,
resource_context_.GetRequestContext());
- StartRequest(&request, ResourceType::MEDIA);
+ StartRequest(&request, content::RESOURCE_TYPE_MEDIA);
EXPECT_EQ(net::URLRequestStatus::FAILED, request.status().status());
}
}
« no previous file with comments | « extensions/browser/extension_protocols.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698