Net: Standardize HttpByteRange printing

net/http/http_util.h

Index: net/http/http_util.h
diff --git a/net/http/http_util.h b/net/http/http_util.h
index ae65146e6e4a42a21f673787d037f06330c9eb0b..b5b833f30bc43c61780cc8c9c698d04090bbc930 100644
--- a/net/http/http_util.h
+++ b/net/http/http_util.h
@@ -66,6 +66,13 @@ class NET_EXPORT HttpUtil {
   static bool ParseRangeHeader(const std::string& range_specifier,
                                std::vector<HttpByteRange>* ranges);
 
+  // Prints the ranges in the standard HTTP form, e.g. "bytes=0-10,300-400,-40".
+  // |ranges| must contain one or more elements. All elements must be valid.
+  static std::string PrintRanges(const std::vector<HttpByteRange>& ranges);
+
+  // Prints the range header a single range. Assumes a valid |range|.
+  static std::string PrintRange(const HttpByteRange& range);

[cbentzel, 2013/11/22 16:24:35]

Remove PrintRanges and place it in the only test file which uses it.

Move PrintRange over to HttpByteRange and remove this usage. No callers used
PrintInterval that I saw, and the byte units are really part of the header.

+
   // Scans the '\r\n'-delimited headers for the given header name.  Returns
   // true if a match is found.  Input is assumed to be well-formed.
   // TODO(darin): kill this