Implement offline mode behind a flag

net/http/http_network_layer_unittest.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 "net/http/http_network_layer.h"
 
 #include "net/base/mock_cert_verifier.h"
 #include "net/base/net_log.h"
 #include "net/dns/mock_host_resolver.h"
 #include "net/http/http_network_session.h"
@@ skipping 272 matching lines @@
   // We should have read the original page data.
   std::string contents;
   rv = ReadTransaction(trans.get(), &contents);
   EXPECT_EQ(OK, rv);
   EXPECT_EQ("Bypass message", contents);
 
   // We should have no entries in our bad proxy list.
   ASSERT_EQ(0u, proxy_service_->proxy_retry_info().size());
 }
 
+TEST_F(HttpNetworkLayerTest, ConfirmNetworkVerified) {

[rvargas (doing something else), 2013/04/08 19:11:52]

nit: remove "Confirm" from the name of both tests?

[Randy Smith (Not in Mondays), 2013/04/08 21:03:11]

Done.

+  MockRead data_reads[] = {
+    MockRead("HTTP/1.0 200 OK\r\n\r\n"),
+    MockRead("hello world"),
+    MockRead(SYNCHRONOUS, OK),
+  };
+  MockWrite data_writes[] = {
+    MockWrite("GET / HTTP/1.1\r\n"
+                   "Host: www.google.com\r\n"

[rvargas (doing something else), 2013/04/08 19:11:52]

nit: we usually don't extra indent when splitting strings

[Randy Smith (Not in Mondays), 2013/04/08 21:03:11]

Ok.  I'll fix the GET test as well (which is where this was copied from).

+                   "Connection: keep-alive\r\n"
+                   "User-Agent: Foo/1.0\r\n\r\n"),
+  };
+  StaticSocketDataProvider data(data_reads, arraysize(data_reads),
+                                     data_writes, arraysize(data_writes));

[rvargas (doing something else), 2013/04/08 19:11:52]

nit: indent four less spaces

[Randy Smith (Not in Mondays), 2013/04/08 21:03:11]

Done (in GET test as well).

+  mock_socket_factory_.AddSocketDataProvider(&data);
+
+  TestCompletionCallback callback;
+
+  HttpRequestInfo request_info;
+  request_info.url = GURL("http://www.google.com/");
+  request_info.method = "GET";
+  request_info.extra_headers.SetHeader(HttpRequestHeaders::kUserAgent,
+                                       "Foo/1.0");
+  request_info.load_flags = LOAD_NORMAL;
+
+  scoped_ptr<HttpTransaction> trans;
+  int rv = factory_->CreateTransaction(DEFAULT_PRIORITY, &trans, NULL);
+  EXPECT_EQ(OK, rv);
+
+  rv = trans->Start(&request_info, callback.callback(), BoundNetLog());
+  if (rv == ERR_IO_PENDING)
+    rv = callback.WaitForResult();

[rvargas (doing something else), 2013/04/08 19:11:52]

nit: callback.GetResult(rv)

[Randy Smith (Not in Mondays), 2013/04/08 21:03:11]

Done (GET as well).

+  ASSERT_EQ(OK, rv);
+
+  EXPECT_TRUE(trans->GetResponseInfo()->network_accessed);
+}
+
+TEST_F(HttpNetworkLayerTest, ConfirmNetworkUnVerified) {
+  MockRead data_reads[] = {
+    MockRead(ASYNC, ERR_CONNECTION_RESET),
+  };
+  MockWrite data_writes[] = {
+    MockWrite("GET / HTTP/1.1\r\n"
+                   "Host: www.google.com\r\n"
+                   "Connection: keep-alive\r\n"
+                   "User-Agent: Foo/1.0\r\n\r\n"),
+  };
+  StaticSocketDataProvider data(data_reads, arraysize(data_reads),
+                                     data_writes, arraysize(data_writes));
+  mock_socket_factory_.AddSocketDataProvider(&data);
+
+  TestCompletionCallback callback;
+
+  HttpRequestInfo request_info;
+  request_info.url = GURL("http://www.google.com/");
+  request_info.method = "GET";
+  request_info.extra_headers.SetHeader(HttpRequestHeaders::kUserAgent,
+                                       "Foo/1.0");
+  request_info.load_flags = LOAD_NORMAL;
+
+  scoped_ptr<HttpTransaction> trans;
+  int rv = factory_->CreateTransaction(DEFAULT_PRIORITY, &trans, NULL);
+  EXPECT_EQ(OK, rv);
+
+  rv = trans->Start(&request_info, callback.callback(), BoundNetLog());
+  if (rv == ERR_IO_PENDING)
+    rv = callback.WaitForResult();
+  ASSERT_EQ(ERR_CONNECTION_RESET, rv);
+
+  // If the response info is null, that means that any consumer won't
+  // see the network accessed bit set.
+  EXPECT_EQ(NULL, trans->GetResponseInfo());
+}
+
 }  // namespace
 
 }  // namespace net