Place source URL of DownloadFile in extended attributes

content/browser/file_metadata_unittest_linux.cc

+// 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 <errno.h>
+#include <sys/types.h>
+#include <sys/xattr.h>
+
+#include <algorithm>
+#include <sstream>
+#include <string>
+
+#include "base/file_path.h"
+#include "base/file_util.h"
+#include "base/logging.h"
+#include "base/scoped_temp_dir.h"
+#include "base/string_split.h"
+#include "content/browser/file_metadata_linux.h"
+#include "googleurl/src/gurl.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+
+using std::istringstream;
+using std::string;
+using std::vector;
+
+class FileMetadataLinuxTest : public testing::Test {
+ public:
+  FileMetadataLinuxTest()
+      : source_url_("http://www.source.com"),
+        referrer_url_("http://www.referrer.com") {}
+
+ protected:
+  virtual void SetUp() OVERRIDE {
+    ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
+    ASSERT_TRUE(file_util::CreateTemporaryFileInDir(temp_dir_.path(),
+                                                    &test_file_));
+    int result = setxattr(test_file_.value().c_str(),
+                          "user.test", "test", 4, 0);
+    is_xattr_supported_ = (!result) || (errno != ENOTSUP);
+  }
+
+  void CheckExtendedAttributeValue(const string attr_name,
+                                   const string expected_value) const {
+    ssize_t len = getxattr(test_file_.value().c_str(), attr_name.c_str(),
+                           NULL, 0);
+    if (len <= static_cast<ssize_t>(0)) {
+      FAIL() << "Attribute '" << attr_name << "' does not exist";
+    }
+    char* buffer = new char[len];
+    len = getxattr(test_file_.value().c_str(), attr_name.c_str(), buffer, len);
+    EXPECT_EQ(expected_value.size(), static_cast<size_t>(len));
+    string real_value(buffer, len);
+    delete[] buffer;
+    EXPECT_EQ(expected_value, real_value);
+  }
+
+  void GetExtendedAttributeNames(vector<string>* attr_names) const {
+    ssize_t len = listxattr(test_file_.value().c_str(), NULL, 0);
+    if (len <= static_cast<ssize_t>(0)) return;
+    char* buffer = new char[len];
+    len = listxattr(test_file_.value().c_str(), buffer, len);
+    attr_names->clear();
+    base::SplitString(string(buffer, len), '\0', attr_names);
+    delete[] buffer;
+  }
+
+  void VerifyAttributesAreSetCorrectly() const {
+    vector<string> attr_names;
+    GetExtendedAttributeNames(&attr_names);
+
+    // Check if the attributes are set on the file
+    vector<string>::const_iterator pos = find(attr_names.begin(),
+        attr_names.end(), file_metadata::kSourceURLAttrName);
+    EXPECT_NE(pos, attr_names.end());
+    pos = find(attr_names.begin(), attr_names.end(),
+        file_metadata::kReferrerURLAttrName);
+    EXPECT_NE(pos, attr_names.end());
+
+    // Check if the attribute values are set correctly
+    CheckExtendedAttributeValue(file_metadata::kSourceURLAttrName,
+                                source_url_.spec());
+    CheckExtendedAttributeValue(file_metadata::kReferrerURLAttrName,
+                                referrer_url_.spec());
+  }
+
+  ScopedTempDir temp_dir_;

[benjhayden, 2012/07/20 17:20:04]

Please keep member variables private and expose them with accessors as
necessary.

+  FilePath test_file_;
+  GURL source_url_;
+  GURL referrer_url_;
+  bool is_xattr_supported_;
+};
+
+TEST_F(FileMetadataLinuxTest, CheckMetadataSetCorrectly) {
+  if (!is_xattr_supported_) {
+    LOG(INFO) << "Test is skipped because extended attributes are not "

[benjhayden, 2012/07/20 17:20:04]

Please put this message just once in SetUp instead of in each TEST_F.

+              << "supported on this OS/file system.";
+    return;
+  }
+  file_metadata::AddOriginMetadataToFile(test_file_,
+      source_url_, referrer_url_);
+  VerifyAttributesAreSetCorrectly();
+}
+
+TEST_F(FileMetadataLinuxTest, SetMetadataMultipleTimes) {
+  if (!is_xattr_supported_) {
+    LOG(INFO) << "Test is skipped because extended attributes are not "
+              << "supported on this OS/file system.";
+    return;
+  }
+  file_metadata::AddOriginMetadataToFile(test_file_,
+      GURL("http://www.dummy.com"), GURL("http://www.dummy.com"));
+  file_metadata::AddOriginMetadataToFile(test_file_,
+      source_url_, referrer_url_);
+  VerifyAttributesAreSetCorrectly();
+}
+
+TEST_F(FileMetadataLinuxTest, InvalidSourceURLTest) {
+  if (!is_xattr_supported_) {
+    LOG(INFO) << "Test is skipped because extended attributes are not "
+              << "supported on this OS/file system.";
+    return;
+  }
+  GURL invalid_url;
+  vector<string> attr_names;
+  file_metadata::AddOriginMetadataToFile(test_file_,
+      invalid_url, referrer_url_);
+  GetExtendedAttributeNames(&attr_names);
+  EXPECT_EQ(attr_names.end(), find(attr_names.begin(), attr_names.end(),
+      file_metadata::kSourceURLAttrName));
+  CheckExtendedAttributeValue(file_metadata::kReferrerURLAttrName,
+                              referrer_url_.spec());
+}
+
+TEST_F(FileMetadataLinuxTest, InvalidReferrerURLTest) {
+  if (!is_xattr_supported_) {
+    LOG(INFO) << "Test is skipped because extended attributes are not "
+              << "supported on this OS/file system.";
+    return;
+  }
+  GURL invalid_url;
+  vector<string> attr_names;
+  file_metadata::AddOriginMetadataToFile(test_file_, source_url_, invalid_url);
+  GetExtendedAttributeNames(&attr_names);
+  EXPECT_EQ(attr_names.end(), find(attr_names.begin(), attr_names.end(),
+      file_metadata::kReferrerURLAttrName));
+  CheckExtendedAttributeValue(file_metadata::kSourceURLAttrName,
+                              source_url_.spec());
+}
+
+TEST_F(FileMetadataLinuxTest, InvalidURLsTest) {
+  if (!is_xattr_supported_) {
+    LOG(INFO) << "Test is skipped because extended attributes are not "
+              << "supported on this OS/file system.";
+    return;
+  }
+  GURL invalid_url;
+  vector<string> attr_names;
+  file_metadata::AddOriginMetadataToFile(test_file_, invalid_url, invalid_url);
+  GetExtendedAttributeNames(&attr_names);
+  EXPECT_EQ(attr_names.end(), find(attr_names.begin(), attr_names.end(),
+      file_metadata::kSourceURLAttrName));
+  EXPECT_EQ(attr_names.end(), find(attr_names.begin(), attr_names.end(),
+      file_metadata::kReferrerURLAttrName));
+}
+
+}  // namespace