Clang format slam.

net/proxy/proxy_config_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/proxy/proxy_config.h"
 #include "net/proxy/proxy_config_service_common_unittest.h"
 #include "net/proxy/proxy_info.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace net {
@@ skipping 89 matching lines @@
     const char* proxy_rules;
 
     ProxyConfig::ProxyRules::Type type;
     // These will be PAC-stle strings, eg 'PROXY foo.com'
     const char* single_proxy;
     const char* proxy_for_http;
     const char* proxy_for_https;
     const char* proxy_for_ftp;
     const char* fallback_proxy;
   } tests[] = {
-    // One HTTP proxy for all schemes.
-    {
-      "myproxy:80",
+        // One HTTP proxy for all schemes.
+        {
+         "myproxy:80", ProxyConfig::ProxyRules::TYPE_SINGLE_PROXY,
+         "PROXY myproxy:80", NULL, NULL, NULL, NULL,
+        },
 
-      ProxyConfig::ProxyRules::TYPE_SINGLE_PROXY,
-      "PROXY myproxy:80",
-      NULL,
-      NULL,
-      NULL,
-      NULL,
-    },
+        // Multiple HTTP proxies for all schemes.
+        {
+         "myproxy:80,https://myotherproxy",
+         ProxyConfig::ProxyRules::TYPE_SINGLE_PROXY,
+         "PROXY myproxy:80;HTTPS myotherproxy:443", NULL, NULL, NULL, NULL,
+        },
 
-    // Multiple HTTP proxies for all schemes.
-    {
-      "myproxy:80,https://myotherproxy",
+        // Only specify a proxy server for "http://" urls.
+        {
+         "http=myproxy:80", ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
+         NULL, "PROXY myproxy:80", NULL, NULL, NULL,
+        },
 
-      ProxyConfig::ProxyRules::TYPE_SINGLE_PROXY,
-      "PROXY myproxy:80;HTTPS myotherproxy:443",
-      NULL,
-      NULL,
-      NULL,
-      NULL,
-    },
+        // Specify an HTTP proxy for "ftp://" and a SOCKS proxy for "https://"
+        // urls.
+        {
+         "ftp=ftp-proxy ; https=socks4://foopy",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL, NULL,
+         "SOCKS foopy:1080", "PROXY ftp-proxy:80", NULL,
+        },
 
-    // Only specify a proxy server for "http://" urls.
-    {
-      "http=myproxy:80",
+        // Give a scheme-specific proxy as well as a non-scheme specific.
+        // The first entry "foopy" takes precedance marking this list as
+        // TYPE_SINGLE_PROXY.
+        {
+         "foopy ; ftp=ftp-proxy", ProxyConfig::ProxyRules::TYPE_SINGLE_PROXY,
+         "PROXY foopy:80", NULL, NULL, NULL, NULL,
+        },
 
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      "PROXY myproxy:80",
-      NULL,
-      NULL,
-      NULL,
-    },
+        // Give a scheme-specific proxy as well as a non-scheme specific.
+        // The first entry "ftp=ftp-proxy" takes precedance marking this list as
+        // TYPE_PROXY_PER_SCHEME.
+        {
+         "ftp=ftp-proxy ; foopy",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL, NULL, NULL,
+         "PROXY ftp-proxy:80", NULL,
+        },
 
-    // Specify an HTTP proxy for "ftp://" and a SOCKS proxy for "https://" urls.
-    {
-      "ftp=ftp-proxy ; https=socks4://foopy",
+        // Include a list of entries for a single scheme.
+        {
+         "ftp=ftp1,ftp2,ftp3", ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
+         NULL, NULL, NULL, "PROXY ftp1:80;PROXY ftp2:80;PROXY ftp3:80", NULL,
+        },
 
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      NULL,
-      "SOCKS foopy:1080",
-      "PROXY ftp-proxy:80",
-      NULL,
-    },
+        // Include multiple entries for the same scheme -- they accumulate.
+        {
+         "http=http1,http2; http=http3",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL,
+         "PROXY http1:80;PROXY http2:80;PROXY http3:80", NULL, NULL, NULL,
+        },
 
-    // Give a scheme-specific proxy as well as a non-scheme specific.
-    // The first entry "foopy" takes precedance marking this list as
-    // TYPE_SINGLE_PROXY.
-    {
-      "foopy ; ftp=ftp-proxy",
+        // Include lists of entries for multiple schemes.
+        {
+         "ftp=ftp1,ftp2,ftp3 ; http=http1,http2; ",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL,
+         "PROXY http1:80;PROXY http2:80", NULL,
+         "PROXY ftp1:80;PROXY ftp2:80;PROXY ftp3:80", NULL,
+        },
 
-      ProxyConfig::ProxyRules::TYPE_SINGLE_PROXY,
-      "PROXY foopy:80",
-      NULL,
-      NULL,
-      NULL,
-      NULL,
-    },
+        // Include non-default proxy schemes.
+        {
+         "http=https://secure_proxy; ftp=socks4://socks_proxy; "
+         "https=socks://foo",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL,
+         "HTTPS secure_proxy:443", "SOCKS5 foo:1080", "SOCKS socks_proxy:1080",
+         NULL,
+        },
 
-    // Give a scheme-specific proxy as well as a non-scheme specific.
-    // The first entry "ftp=ftp-proxy" takes precedance marking this list as
-    // TYPE_PROXY_PER_SCHEME.
-    {
-      "ftp=ftp-proxy ; foopy",
+        // Only SOCKS proxy present, others being blank.
+        {
+         "socks=foopy", ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL,
+         NULL, NULL, NULL, "SOCKS foopy:1080",
+        },
 
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      NULL,
-      NULL,
-      "PROXY ftp-proxy:80",
-      NULL,
-    },
+        // SOCKS proxy present along with other proxies too
+        {
+         "http=httpproxy ; https=httpsproxy ; ftp=ftpproxy ; socks=foopy ",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL,
+         "PROXY httpproxy:80", "PROXY httpsproxy:80", "PROXY ftpproxy:80",
+         "SOCKS foopy:1080",
+        },
 
-    // Include a list of entries for a single scheme.
-    {
-      "ftp=ftp1,ftp2,ftp3",
+        // SOCKS proxy (with modifier) present along with some proxies
+        // (FTP being blank)
+        {
+         "http=httpproxy ; https=httpsproxy ; socks=socks5://foopy ",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL,
+         "PROXY httpproxy:80", "PROXY httpsproxy:80", NULL, "SOCKS5 foopy:1080",
+        },
 
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      NULL,
-      NULL,
-      "PROXY ftp1:80;PROXY ftp2:80;PROXY ftp3:80",
-      NULL,
-    },
+        // Include unsupported schemes -- they are discarded.
+        {
+         "crazy=foopy ; foo=bar ; https=myhttpsproxy",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL, NULL,
+         "PROXY myhttpsproxy:80", NULL, NULL,
+        },
 
-    // Include multiple entries for the same scheme -- they accumulate.
-    {
-      "http=http1,http2; http=http3",
+        // direct:// as first option for a scheme.
+        {
+         "http=direct://,myhttpproxy; https=direct://",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL,
+         "DIRECT;PROXY myhttpproxy:80", "DIRECT", NULL, NULL,
+        },
 
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      "PROXY http1:80;PROXY http2:80;PROXY http3:80",
-      NULL,
-      NULL,
-      NULL,
-    },
-
-    // Include lists of entries for multiple schemes.
-    {
-      "ftp=ftp1,ftp2,ftp3 ; http=http1,http2; ",
-
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      "PROXY http1:80;PROXY http2:80",
-      NULL,
-      "PROXY ftp1:80;PROXY ftp2:80;PROXY ftp3:80",
-      NULL,
-    },
-
-    // Include non-default proxy schemes.
-    {
-      "http=https://secure_proxy; ftp=socks4://socks_proxy; https=socks://foo",
-
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      "HTTPS secure_proxy:443",
-      "SOCKS5 foo:1080",
-      "SOCKS socks_proxy:1080",
-      NULL,
-    },
-
-    // Only SOCKS proxy present, others being blank.
-    {
-      "socks=foopy",
-
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      NULL,
-      NULL,
-      NULL,
-      "SOCKS foopy:1080",
-      },
-
-    // SOCKS proxy present along with other proxies too
-    {
-      "http=httpproxy ; https=httpsproxy ; ftp=ftpproxy ; socks=foopy ",
-
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      "PROXY httpproxy:80",
-      "PROXY httpsproxy:80",
-      "PROXY ftpproxy:80",
-      "SOCKS foopy:1080",
-    },
-
-    // SOCKS proxy (with modifier) present along with some proxies
-    // (FTP being blank)
-    {
-      "http=httpproxy ; https=httpsproxy ; socks=socks5://foopy ",
-
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      "PROXY httpproxy:80",
-      "PROXY httpsproxy:80",
-      NULL,
-      "SOCKS5 foopy:1080",
-      },
-
-    // Include unsupported schemes -- they are discarded.
-    {
-      "crazy=foopy ; foo=bar ; https=myhttpsproxy",
-
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      NULL,
-      "PROXY myhttpsproxy:80",
-      NULL,
-      NULL,
-    },
-
-    // direct:// as first option for a scheme.
-    {
-      "http=direct://,myhttpproxy; https=direct://",
-
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      "DIRECT;PROXY myhttpproxy:80",
-      "DIRECT",
-      NULL,
-      NULL,
-    },
-
-    // direct:// as a second option for a scheme.
-    {
-      "http=myhttpproxy,direct://",
-
-      ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME,
-      NULL,
-      "PROXY myhttpproxy:80;DIRECT",
-      NULL,
-      NULL,
-      NULL,
-    },
-
-  };
+        // direct:// as a second option for a scheme.
+        {
+         "http=myhttpproxy,direct://",
+         ProxyConfig::ProxyRules::TYPE_PROXY_PER_SCHEME, NULL,
+         "PROXY myhttpproxy:80;DIRECT", NULL, NULL, NULL,
+        },
+    };
 
   ProxyConfig config;
 
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) {
     config.proxy_rules().ParseFromString(tests[i].proxy_rules);
 
     EXPECT_EQ(tests[i].type, config.proxy_rules().type);
     ExpectProxyServerEquals(tests[i].single_proxy,
                             config.proxy_rules().single_proxies);
     ExpectProxyServerEquals(tests[i].proxy_for_http,
                             config.proxy_rules().proxies_for_http);
     ExpectProxyServerEquals(tests[i].proxy_for_https,
                             config.proxy_rules().proxies_for_https);
     ExpectProxyServerEquals(tests[i].proxy_for_ftp,
                             config.proxy_rules().proxies_for_ftp);
     ExpectProxyServerEquals(tests[i].fallback_proxy,
                             config.proxy_rules().fallback_proxies);
   }
 }
 
 TEST(ProxyConfigTest, ProxyRulesSetBypassFlag) {
   // Test whether the did_bypass_proxy() flag is set in proxy info correctly.
   ProxyConfig::ProxyRules rules;
-  ProxyInfo  result;
+  ProxyInfo result;
 
   rules.ParseFromString("http=httpproxy:80");
   rules.bypass_rules.AddRuleFromString(".com");
 
   rules.Apply(GURL("http://example.com"), &result);
   EXPECT_TRUE(result.is_direct_only());
   EXPECT_TRUE(result.did_bypass_proxy());
 
   rules.Apply(GURL("http://example.org"), &result);
   EXPECT_FALSE(result.is_direct());
   EXPECT_FALSE(result.did_bypass_proxy());
 
   // Try with reversed bypass rules.
   rules.reverse_bypass = true;
 
   rules.Apply(GURL("http://example.org"), &result);
   EXPECT_TRUE(result.is_direct_only());
   EXPECT_TRUE(result.did_bypass_proxy());
 
   rules.Apply(GURL("http://example.com"), &result);
   EXPECT_FALSE(result.is_direct());
   EXPECT_FALSE(result.did_bypass_proxy());
 }
 
 }  // namespace
 }  // namespace net