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

Unified Diff: content/browser/download/file_metadata_unittest_linux.cc

Issue 10821069: Moving file_metadata files under content/browser/download (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: Git-based patch to overcome the 'svn mv' problem Created 8 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 | « content/browser/download/file_metadata_mac.mm ('k') | content/browser/file_metadata_mac.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/download/file_metadata_unittest_linux.cc
diff --git a/content/browser/download/file_metadata_unittest_linux.cc b/content/browser/download/file_metadata_unittest_linux.cc
index 40ed3e5160eba56d39db7ac3399a2fc3ac08e0f1..ec2041a6c11dcf56c072eada241deaf2b558111a 100644
--- a/content/browser/download/file_metadata_unittest_linux.cc
+++ b/content/browser/download/file_metadata_unittest_linux.cc
@@ -19,6 +19,7 @@
#include "googleurl/src/gurl.h"
#include "testing/gtest/include/gtest/gtest.h"
+namespace content {
namespace {
using std::istringstream;
@@ -92,17 +93,14 @@ class FileMetadataLinuxTest : public testing::Test {
// Check if the attributes are set on the file
vector<string>::const_iterator pos = find(attr_names.begin(),
- attr_names.end(), content::kSourceURLAttrName);
+ attr_names.end(), kSourceURLAttrName);
EXPECT_NE(pos, attr_names.end());
- pos = find(attr_names.begin(), attr_names.end(),
- content::kReferrerURLAttrName);
+ pos = find(attr_names.begin(), attr_names.end(), kReferrerURLAttrName);
EXPECT_NE(pos, attr_names.end());
// Check if the attribute values are set correctly
- CheckExtendedAttributeValue(content::kSourceURLAttrName,
- source_url().spec());
- CheckExtendedAttributeValue(content::kReferrerURLAttrName,
- referrer_url().spec());
+ CheckExtendedAttributeValue(kSourceURLAttrName, source_url().spec());
+ CheckExtendedAttributeValue(kReferrerURLAttrName, referrer_url().spec());
}
private:
@@ -115,15 +113,15 @@ class FileMetadataLinuxTest : public testing::Test {
TEST_F(FileMetadataLinuxTest, CheckMetadataSetCorrectly) {
if (!is_xattr_supported()) return;
- content::AddOriginMetadataToFile(test_file(), source_url(), referrer_url());
+ AddOriginMetadataToFile(test_file(), source_url(), referrer_url());
VerifyAttributesAreSetCorrectly();
}
TEST_F(FileMetadataLinuxTest, SetMetadataMultipleTimes) {
if (!is_xattr_supported()) return;
- content::AddOriginMetadataToFile(test_file(),
- GURL("http://www.dummy.com"), GURL("http://www.dummy.com"));
- content::AddOriginMetadataToFile(test_file(), source_url(), referrer_url());
+ GURL dummy_url("http://www.dummy.com");
+ AddOriginMetadataToFile(test_file(), dummy_url, dummy_url);
+ AddOriginMetadataToFile(test_file(), source_url(), referrer_url());
VerifyAttributesAreSetCorrectly();
}
@@ -131,35 +129,35 @@ TEST_F(FileMetadataLinuxTest, InvalidSourceURLTest) {
if (!is_xattr_supported()) return;
GURL invalid_url;
vector<string> attr_names;
- content::AddOriginMetadataToFile(test_file(), invalid_url, referrer_url());
+ AddOriginMetadataToFile(test_file(), invalid_url, referrer_url());
GetExtendedAttributeNames(&attr_names);
EXPECT_EQ(attr_names.end(), find(attr_names.begin(), attr_names.end(),
- content::kSourceURLAttrName));
- CheckExtendedAttributeValue(content::kReferrerURLAttrName,
- referrer_url().spec());
+ kSourceURLAttrName));
+ CheckExtendedAttributeValue(kReferrerURLAttrName, referrer_url().spec());
}
TEST_F(FileMetadataLinuxTest, InvalidReferrerURLTest) {
if (!is_xattr_supported()) return;
GURL invalid_url;
vector<string> attr_names;
- content::AddOriginMetadataToFile(test_file(), source_url(), invalid_url);
+ AddOriginMetadataToFile(test_file(), source_url(), invalid_url);
GetExtendedAttributeNames(&attr_names);
EXPECT_EQ(attr_names.end(), find(attr_names.begin(), attr_names.end(),
- content::kReferrerURLAttrName));
- CheckExtendedAttributeValue(content::kSourceURLAttrName, source_url().spec());
+ kReferrerURLAttrName));
+ CheckExtendedAttributeValue(kSourceURLAttrName, source_url().spec());
}
TEST_F(FileMetadataLinuxTest, InvalidURLsTest) {
if (!is_xattr_supported()) return;
GURL invalid_url;
vector<string> attr_names;
- content::AddOriginMetadataToFile(test_file(), invalid_url, invalid_url);
+ AddOriginMetadataToFile(test_file(), invalid_url, invalid_url);
GetExtendedAttributeNames(&attr_names);
EXPECT_EQ(attr_names.end(), find(attr_names.begin(), attr_names.end(),
- content::kSourceURLAttrName));
+ kSourceURLAttrName));
EXPECT_EQ(attr_names.end(), find(attr_names.begin(), attr_names.end(),
- content::kReferrerURLAttrName));
+ kReferrerURLAttrName));
}
} // namespace
+} // namespace content
« no previous file with comments | « content/browser/download/file_metadata_mac.mm ('k') | content/browser/file_metadata_mac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698