Skip ThrottlesAddedExactlyOnceTo*Downloads tests with the network service.

These tests don't apply to the network service code path because
1) ResourceThrottles aren't used
2) Downloads go through a different path (interception of the URLLoader in NavigationRequest) that make the bug for these tests not possible

Bug: 787614
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ia3d620540801d26344fb8453228e911a2c629c10
Reviewed-on: https://chromium-review.googlesource.com/1028364Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553636}

chrome/browser/loader/chrome_resource_dispatcher_host_delegate_browsertest.cc

@@ -58,6 +58,7 @@
 #include "net/test/url_request/url_request_mock_http_job.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_filter.h"
+#include "services/network/public/cpp/features.h"
 #include "testing/gmock/include/gmock/gmock-matchers.h"
 
 using content::ResourceType;
@@ -590,6 +591,10 @@ IN_PROC_BROWSER_TEST_F(ChromeResourceDispatcherHostDelegateBrowserTest,
 // See https://crbug.com/640545
 IN_PROC_BROWSER_TEST_F(ChromeResourceDispatcherHostDelegateBrowserTest,
                        ThrottlesAddedExactlyOnceToTinySniffedDownloads) {
+  // This code path isn't used when the network service is enabled.
+  if (base::FeatureList::IsEnabled(network::features::kNetworkService))
+    return;
+
   GURL url = embedded_test_server()->GetURL("/downloads/tiny_binary.bin");
   DownloadTestObserverNotInProgress download_observer(
       content::BrowserContext::GetDownloadManager(browser()->profile()), 1);
@@ -603,6 +608,10 @@ IN_PROC_BROWSER_TEST_F(ChromeResourceDispatcherHostDelegateBrowserTest,
 // have their mime type determined before the end of the response is reported.
 IN_PROC_BROWSER_TEST_F(ChromeResourceDispatcherHostDelegateBrowserTest,
                        ThrottlesAddedExactlyOnceToLargeSniffedDownloads) {
+  // This code path isn't used when the network service is enabled.
+  if (base::FeatureList::IsEnabled(network::features::kNetworkService))
+    return;
+
   GURL url = embedded_test_server()->GetURL("/downloads/thisdayinhistory.xls");
   DownloadTestObserverNotInProgress download_observer(
       content::BrowserContext::GetDownloadManager(browser()->profile()), 1);
@@ -616,6 +625,10 @@ IN_PROC_BROWSER_TEST_F(ChromeResourceDispatcherHostDelegateBrowserTest,
 // <a download> click.
 IN_PROC_BROWSER_TEST_F(ChromeResourceDispatcherHostDelegateBrowserTest,
                        ThrottlesAddedExactlyOnceToADownloads) {
+  // This code path isn't used when the network service is enabled.
+  if (base::FeatureList::IsEnabled(network::features::kNetworkService))
+    return;
+
   DownloadTestObserverNotInProgress download_observer(
       content::BrowserContext::GetDownloadManager(browser()->profile()), 1);
   download_observer.StartObserving();

testing/buildbot/filters/mojo.fyi.network_browser_tests.filter

@@ -246,11 +246,6 @@
 -ChromeResourceDispatcherHostDelegateBrowserTest.PolicyHeader
 -ChromeResourceDispatcherHostDelegateBrowserTest.PolicyHeaderForRedirect
 
-# http://crbug.com/787614
--ChromeResourceDispatcherHostDelegateBrowserTest.ThrottlesAddedExactlyOnceToADownloads
--ChromeResourceDispatcherHostDelegateBrowserTest.ThrottlesAddedExactlyOnceToLargeSniffedDownloads
--ChromeResourceDispatcherHostDelegateBrowserTest.ThrottlesAddedExactlyOnceToTinySniffedDownloads
-
 # Add support of download_to_file for URLLoader.
 # http://crbug.com/791702
 -OutOfProcessPPAPITest.FileRef1