Ensure loading PDFs doesn't generate 2 http requests

extensions/browser/guest_view/mime_handler_view/mime_handler_view_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 "base/base_paths.h"
 #include "base/files/file_util.h"
 #include "base/path_service.h"
+#include "base/run_loop.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "components/guest_view/browser/test_guest_view_manager.h"
+#include "content/public/browser/browser_thread.h"
 #include "content/public/test/browser_test_utils.h"
 #include "extensions/browser/api/extensions_api_client.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/guest_view/extensions_guest_view_manager_delegate.h"
 #include "extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.h"
 #include "extensions/browser/guest_view/mime_handler_view/test_mime_handler_view_guest.h"
 #include "extensions/test/result_catcher.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
+#include "net/url_request/url_request_filter.h"
+#include "net/url_request/url_request_interceptor.h"
 
 using extensions::ExtensionsAPIClient;
 using extensions::MimeHandlerViewGuest;
 using extensions::TestMimeHandlerViewGuest;
 using guest_view::GuestViewManager;
 using guest_view::GuestViewManagerDelegate;
 using guest_view::TestGuestViewManager;
 using guest_view::TestGuestViewManagerFactory;
 
 // The test extension id is set by the key value in the manifest.
 const char* kExtensionId = "oickdpebdnfbgkcaoklfcdhjniefkcji";
 
+// Counts the number of URL requests made for a given URL.
+class URLRequestCounter {
+ public:
+  URLRequestCounter(const GURL& url) : url_(url), count_(0) {
+    base::RunLoop run_loop;
+    content::BrowserThread::PostTaskAndReply(
+        content::BrowserThread::IO, FROM_HERE,
+        base::Bind(&URLRequestCounter::AddInterceptor, base::Unretained(this)),
+        run_loop.QuitClosure());
+    run_loop.Run();
+  }
+
+  ~URLRequestCounter() {
+    base::RunLoop run_loop;
+    content::BrowserThread::PostTaskAndReply(
+        content::BrowserThread::IO, FROM_HERE,
+        base::Bind(&URLRequestCounter::RemoveInterceptor,
+                   base::Unretained(this)),
+        run_loop.QuitClosure());
+    run_loop.Run();
+  }
+
+  int count() { return count_; }
+
+ private:
+  class SimpleRequestInterceptor : public net::URLRequestInterceptor {
+   public:
+    SimpleRequestInterceptor(int* count) : count_(count) {}
+
+    // URLRequestInterceptor implementation:
+    net::URLRequestJob* MaybeInterceptRequest(
+        net::URLRequest* request,
+        net::NetworkDelegate* network_delegate) const override {
+      (*count_)++;
+      return nullptr;
+    }
+
+    int* count_;

[Sam McNally, 2016/08/30 03:04:28]

private:

[raymes, 2016/08/30 07:32:54]

Done.

+  };
+
+  void AddInterceptor() {
+    net::URLRequestFilter::GetInstance()->AddUrlInterceptor(
+        url_, base::WrapUnique(new SimpleRequestInterceptor(&count_)));

[Sam McNally, 2016/08/30 03:04:28]

base::MakeUnique<SimpleRequestInterceptor>(&count_)

[raymes, 2016/08/30 07:32:54]

Done.

+  }
+
+  void RemoveInterceptor() {
+    net::URLRequestFilter::GetInstance()->RemoveUrlHandler(url_);
+  }
+
+  GURL url_;

[Sam McNally, 2016/08/30 03:04:28]

const

[raymes, 2016/08/30 07:32:54]

Done.

+  int count_;

[Sam McNally, 2016/08/30 03:04:28]

You're reading and writing |count_| on different threads. Are you sure there's
sufficient synchronisation?

[raymes, 2016/08/30 07:32:54]

Done - shifted the code around as discussed.

+};

[Sam McNally, 2016/08/30 03:04:28]

DISALLOW_COPY_AND_ASSIGN

[raymes, 2016/08/30 07:32:54]

Done.

+
 class MimeHandlerViewTest : public ExtensionApiTest {
  public:
   MimeHandlerViewTest() {
     GuestViewManager::set_factory_for_testing(&factory_);
   }
 
   ~MimeHandlerViewTest() override {}
 
+  void SetUpOnMainThread() override {
+    ExtensionApiTest::SetUpOnMainThread();
+
+    StartEmbeddedTestServer();

[Sam McNally, 2016/08/30 03:04:28]

No ASSERT_TRUE()?

[raymes, 2016/08/30 07:32:54]

Done.

+    embedded_test_server()->ServeFilesFromDirectory(
+        test_data_dir_.AppendASCII("mime_handler_view"));
+  }
+
   // TODO(paulmeyer): This function is implemented over and over by the
   // different GuestView test classes. It really needs to be refactored out to
   // some kind of GuestViewTest base class.
   TestGuestViewManager* GetGuestViewManager() {
     TestGuestViewManager* manager = static_cast<TestGuestViewManager*>(
         TestGuestViewManager::FromBrowserContext(browser()->profile()));
     // TestGuestViewManager::WaitForSingleGuestCreated can and will get called
     // before a guest is created. Since GuestViewManager is usually not created
     // until the first guest is created, this means that |manager| will be
     // nullptr if trying to use the manager to wait for the first guest. Because
@@ skipping 28 matching lines @@
     ASSERT_TRUE(extension);
 
     extensions::ResultCatcher catcher;
     ui_test_utils::NavigateToURL(browser(), url);
 
     if (!catcher.GetNextResult())
       FAIL() << catcher.message();
   }
 
   void RunTest(const std::string& path) {
-    ASSERT_TRUE(StartEmbeddedTestServer());
-    embedded_test_server()->ServeFilesFromDirectory(
-        test_data_dir_.AppendASCII("mime_handler_view"));
-
     RunTestWithUrl(embedded_test_server()->GetURL("/" + path));
   }
 
  private:
   TestGuestViewManagerFactory factory_;
 };
 
 IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, PostMessage) {
   RunTest("test_postmessage.html");
 }
@@ skipping 48 matching lines @@
       GetGuestViewManager()->WaitForSingleGuestCreated();
   TestMimeHandlerViewGuest* guest = static_cast<TestMimeHandlerViewGuest*>(
       MimeHandlerViewGuest::FromWebContents(guest_web_contents));
   guest->WaitForGuestAttached();
 
   // Ensure that the guest has the correct size after it has attached.
   auto guest_size = guest->size();
   CHECK_EQ(guest_size.width(), 500);
   CHECK_EQ(guest_size.height(), 400);
 }
+
+// Regression test for crbug.com/587709.
+IN_PROC_BROWSER_TEST_F(MimeHandlerViewTest, SingleRequest) {
+  GURL url(embedded_test_server()->GetURL("/testBasic.csv"));
+  URLRequestCounter request_counter(url);
+  RunTest("testBasic.csv");
+  EXPECT_EQ(1, request_counter.count());
+}