Add content_browsertests for testing cache control flags

content/browser/service_worker/service_worker_browsertest.cc

 // Copyright 2014 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 <stddef.h>
 #include <stdint.h>
 
 #include <map>
 #include <memory>
 #include <utility>
@@ skipping 1150 matching lines @@
 // Tests that when data saver is enabled and a cross-origin fetch by a webpage
 // is intercepted by a serviceworker, and the serviceworker does a fetch, the
 // preflight request does not have save-data in Access-Control-Request-Headers.
 IN_PROC_BROWSER_TEST_F(ServiceWorkerBrowserTest, CrossOriginFetchWithSaveData) {
   const char kPageUrl[] = "/service_worker/fetch_cross_origin.html";
   const char kWorkerUrl[] = "/service_worker/fetch_event_pass_through.js";
   net::EmbeddedTestServer cross_origin_server;
   cross_origin_server.ServeFilesFromSourceDirectory("content/test/data");
   cross_origin_server.RegisterRequestHandler(
       base::Bind(&VerifySaveDataNotInAccessControlRequestHeader));
-  cross_origin_server.Start();
+  ASSERT_TRUE(cross_origin_server.Start());

[kinuko, 2016/05/13 14:10:18]

nit: this change looks fine but it's unrelated to this patch?

[kinuko, 2016/05/13 14:11:57]

I see, you needed this because you added the warning.  It'd be probably a bit
nicer to split the patch for the part then

 
   MockContentBrowserClient content_browser_client;
   content_browser_client.set_data_saver_enabled(true);
   ContentBrowserClient* old_client =
       SetBrowserClientForTesting(&content_browser_client);
   shell()->web_contents()->GetRenderViewHost()->OnWebkitPreferencesChanged();
   scoped_refptr<WorkerActivatedObserver> observer =
       new WorkerActivatedObserver(wrapper());
   observer->Init();
   public_context()->RegisterServiceWorker(
@@ skipping 652 matching lines @@
   NavigateToTestPage();
   // The V8 code cache must be stored to the CacheStorage which must be bigger
   // than 12 byte.
   EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize);
 
   NavigateToTestPage();
   EXPECT_GT(GetSideDataSize(), kV8CacheTimeStampDataSize);
 }
 
 }  // namespace content