Enforce SafetyMode for YouTube if prefs::kForceSafeSearch is on.

net/cookies/cookie_util.h

 // 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.
 
 #ifndef NET_COOKIES_COOKIE_UTIL_H_
 #define NET_COOKIES_COOKIE_UTIL_H_
 
 #include <string>
+#include <utility>
+#include <vector>
 
+#include "base/strings/string_piece.h"
 #include "base/time/time.h"
 #include "net/base/net_export.h"
 
 class GURL;
 
 namespace net {
 namespace cookie_util {
 
 // Returns the effective TLD+1 for a given host. This only makes sense for http
 // and https schemes. For other schemes, the host will be returned unchanged
@@ skipping 13 matching lines @@
 // Returns true if a domain string represents a host-only cookie,
 // i.e. it doesn't begin with a leading '.' character.
 NET_EXPORT bool DomainIsHostOnly(const std::string& domain_string);
 
 // Parses the string with the cookie time (very forgivingly).
 NET_EXPORT base::Time ParseCookieTime(const std::string& time_string);
 
 // Convenience for converting a cookie origin (domain and https pair) to a URL.
 NET_EXPORT GURL CookieOriginToURL(const std::string& domain, bool is_https);
 
-}  // namspace cookie_util
+// A ParsedRequestCookie consists of the key and value of the cookie.
+typedef std::pair<base::StringPiece, base::StringPiece> ParsedRequestCookie;
+typedef std::vector<ParsedRequestCookie> ParsedRequestCookies;
+
+// Assumes that |header_value| is the cookie header value of a HTTP Request
+// following the cookie-string schema of RFC 6265, section 4.2.1, and returns
+// cookie name/value pairs. If cookie values are presented in double quotes,
+// these will appear in |parsed_cookies| as well. Assumes that the cookie
+// header is written by Chromium and therefore well-formed.
+NET_EXPORT void ParseRequestCookieLine(const std::string& header_value,

[erikwright (departed), 2014/07/09 15:36:23]

Please add tests for these methods.

I wish that "BuildCookieLine" in cookie_monster.cc were public so you could
easily test against the cookies built by Chromium, but at a minimum you can at
least have some hardcoded strings to test against.

[Marc Treib, 2014/07/10 09:03:36]

I've added some tests (which actually uncovered a bug in
SerializeRequestCookieLine), PTAL!

+                                       ParsedRequestCookies* parsed_cookies);
+
+// Writes all cookies of |parsed_cookies| into a HTTP Request header value
+// that belongs to the "Cookie" header.
+NET_EXPORT std::string SerializeRequestCookieLine(

[erikwright (departed), 2014/07/09 15:36:23]

This method requires that |parsed_cookies| be appropriately escaped. This should
be documented.

[Marc Treib, 2014/07/10 09:03:36]

Done.

+    const ParsedRequestCookies& parsed_cookies);
+
+}  // namespace cookie_util
 }  // namespace net
 
 #endif  // NET_COOKIES_COOKIE_UTIL_H_