Add stream operators for URLPattern and URLPatternSet and unit test for URLPatternSet

extensions/common/url_pattern_set.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 "extensions/common/url_pattern_set.h"
 
 #include <iterator>
 
 #include "base/logging.h"
 #include "base/memory/linked_ptr.h"
@@ skipping 81 matching lines @@
 
 URLPatternSet& URLPatternSet::operator=(const URLPatternSet& rhs) {
   patterns_ = rhs.patterns_;
   return *this;
 }
 
 bool URLPatternSet::operator==(const URLPatternSet& other) const {
   return patterns_ == other.patterns_;
 }
 
+std::ostream& operator<<(std::ostream& out,
+                         const URLPatternSet& url_pattern_set) {
+  out << "{ ";
+  std::set<URLPattern>::const_iterator end = url_pattern_set.patterns().end();
+  if (!url_pattern_set.patterns().empty())
+    --end;
+
+  copy(url_pattern_set.patterns().begin(),

[Jeffrey Yasskin, 2014/05/28 21:06:08]

Heh. Nice use of the STL, but I think it'll wind up simpler just to write the
for loop.

[aboxhall, 2014/05/28 22:06:44]

Done (not much simpler actually!)

+       end,
+       std::ostream_iterator<URLPattern>(out, ", "));
+
+  if (!url_pattern_set.patterns().empty())
+    out << *end << " ";
+
+  out << "}";
+  return out;
+}
+
 bool URLPatternSet::is_empty() const {
   return patterns_.empty();
 }
 
 size_t URLPatternSet::size() const {
   return patterns_.size();
 }
 
 bool URLPatternSet::AddPattern(const URLPattern& pattern) {
   return patterns_.insert(pattern).second;
@@ skipping 103 matching lines @@
   for (size_t i = 0; i < value.GetSize(); ++i) {
     std::string item;
     if (!value.GetString(i, &item))
       return false;
     patterns.push_back(item);
   }
   return Populate(patterns, valid_schemes, allow_file_access, error);
 }
 
 }  // namespace extensions