Added HttpUtils::HasValidators and HttpResponse::HasValidators

net/http/http_util_unittest.cc

Index: net/http/http_util_unittest.cc
diff --git a/net/http/http_util_unittest.cc b/net/http/http_util_unittest.cc
index a5c984db66b1f5e46798036e8c1ac3758fab9f81..8c38206a737482d4897fc95651b5f1650b319546 100644
--- a/net/http/http_util_unittest.cc
+++ b/net/http/http_util_unittest.cc
@@ -1178,4 +1178,20 @@ TEST(HttpUtilTest, NameValuePairsIteratorMissingEndQuote) {
       &parser, false, true, std::string(), std::string()));
 }
 
+TEST(HttpUtilTest, HasValidators) {

[bengr, 2015/12/03 17:42:21]

For completeness, I think you should test the following cases for each of
http/0.9, http/1.0, and http/1.1:

no etag, no last-modified
weak etag, no last-modified
strong etag, no last-modified
no etag, last-modified
weak etag, last-modified
strong etag, last-modified
no etag, last-modified with bad date
weak etag, last-modified with bad date

For http 0.9 and 1.0, I'm ok with not testing strong and weak etags separately.
strong etag, last-modified with bad date

[jamartin (wrong), 2015/12/08 16:58:18]

Done.

I've gone for the full N^3 approach on this class but not in the other
(http_response_header_unittest.cc). Let me know if you want me to replicate
these tests in there as well.

+  EXPECT_FALSE(HttpUtil::HasValidators(HttpVersion(0, 9), "anything",
+                                       "Tue, 15 Nov 1994 12:45:26 GMT"));
+
+  EXPECT_FALSE(HttpUtil::HasValidators(HttpVersion(1, 0), "", ""));
+  EXPECT_FALSE(HttpUtil::HasValidators(HttpVersion(1, 0), "stub etag", ""));

[bengr, 2015/12/03 17:42:21]

Maybe use the string "anything" here too, as well as below.

[jamartin (wrong), 2015/12/08 16:58:18]

Done.

+  EXPECT_FALSE(HttpUtil::HasValidators(HttpVersion(1, 0), "", "invalid date"));
+  EXPECT_TRUE(HttpUtil::HasValidators(HttpVersion(1, 0), "",

[bengr, 2015/12/03 17:42:21]

Also EXPECT_TRUE when both an etag and a last-modified are supplied.

[jamartin (wrong), 2015/12/08 16:58:18]

Done.

+                                      "Tue, 15 Nov 1994 12:45:26 GMT"));
+
+  EXPECT_FALSE(HttpUtil::HasValidators(HttpVersion(1, 1), "", ""));
+  EXPECT_TRUE(HttpUtil::HasValidators(HttpVersion(1, 1), "stub etag", ""));
+  EXPECT_TRUE(HttpUtil::HasValidators(HttpVersion(1, 1), "stub etag",
+                                      "Tue, 15 Nov 1994 12:45:26 GMT"));
+}
+
 }  // namespace net