Clang format slam.

net/proxy/proxy_script_decider_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 <vector>
 
 #include "base/bind.h"
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
@@ skipping 21 matching lines @@
   kFailedDownloading = -100,
   kFailedParsing = ERR_PAC_SCRIPT_FAILED,
 };
 
 class Rules {
  public:
   struct Rule {
     Rule(const GURL& url, int fetch_error, bool is_valid_script)
         : url(url),
           fetch_error(fetch_error),
-          is_valid_script(is_valid_script) {
-    }
+          is_valid_script(is_valid_script) {}
 
     base::string16 text() const {
       if (is_valid_script)
         return base::UTF8ToUTF16(url.spec() + "!FindProxyForURL");
       if (fetch_error == OK)
         return base::UTF8ToUTF16(url.spec() + "!invalid-script");
       return base::string16();
     }
 
     GURL url;
     int fetch_error;
     bool is_valid_script;
   };
 
   Rule AddSuccessRule(const char* url) {
     Rule rule(GURL(url), OK /*fetch_error*/, true);
     rules_.push_back(rule);
     return rule;
   }
 
   void AddFailDownloadRule(const char* url) {
-    rules_.push_back(Rule(GURL(url), kFailedDownloading /*fetch_error*/,
-        false));
+    rules_.push_back(
+        Rule(GURL(url), kFailedDownloading /*fetch_error*/, false));
   }
 
   void AddFailParsingRule(const char* url) {
     rules_.push_back(Rule(GURL(url), OK /*fetch_error*/, false));
   }
 
   const Rule& GetRuleByUrl(const GURL& url) const {
     for (RuleList::const_iterator it = rules_.begin(); it != rules_.end();
          ++it) {
       if (it->url == url)
@@ skipping 65 matching lines @@
 
   virtual void CompleteRequests(int result, const base::string16& script);
 
  private:
   CompletionCallback callback_;
   base::string16* utf16_text_;
   GURL gurl_;
   DISALLOW_COPY_AND_ASSIGN(MockDhcpProxyScriptFetcher);
 };
 
-MockDhcpProxyScriptFetcher::MockDhcpProxyScriptFetcher() { }
+MockDhcpProxyScriptFetcher::MockDhcpProxyScriptFetcher() {
+}
 
-MockDhcpProxyScriptFetcher::~MockDhcpProxyScriptFetcher() { }
+MockDhcpProxyScriptFetcher::~MockDhcpProxyScriptFetcher() {
+}
 
 int MockDhcpProxyScriptFetcher::Fetch(base::string16* utf16_text,
                                       const CompletionCallback& callback) {
   utf16_text_ = utf16_text;
   callback_ = callback;
   return ERR_IO_PENDING;
 }
 
-void MockDhcpProxyScriptFetcher::Cancel() { }
+void MockDhcpProxyScriptFetcher::Cancel() {
+}
 
 const GURL& MockDhcpProxyScriptFetcher::GetPacURL() const {
   return gurl_;
 }
 
 void MockDhcpProxyScriptFetcher::SetPacURL(const GURL& url) {
   gurl_ = url;
 }
 
 void MockDhcpProxyScriptFetcher::CompleteRequests(
-    int result, const base::string16& script) {
+    int result,
+    const base::string16& script) {
   *utf16_text_ = script;
   callback_.Run(result);
 }
 
 // Succeed using custom PAC script.
 TEST(ProxyScriptDeciderTest, CustomPacSucceeds) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   Rules::Rule rule = rules.AddSuccessRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   CapturingNetLog log;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, &log);
-  EXPECT_EQ(OK, decider.Start(
-      config, base::TimeDelta(), true, callback.callback()));
+  EXPECT_EQ(
+      OK, decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(rule.text(), decider.script_data()->utf16());
 
   // Check the NetLog was filled correctly.
   CapturingNetLog::CapturedEntryList entries;
   log.GetEntries(&entries);
 
   EXPECT_EQ(4u, entries.size());
-  EXPECT_TRUE(LogContainsBeginEvent(
-      entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsBeginEvent(entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
   EXPECT_TRUE(LogContainsBeginEvent(
       entries, 1, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEndEvent(
       entries, 2, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
-  EXPECT_TRUE(LogContainsEndEvent(
-      entries, 3, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsEndEvent(entries, 3, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
 
   EXPECT_TRUE(decider.effective_config().has_pac_url());
   EXPECT_EQ(config.pac_url(), decider.effective_config().pac_url());
 }
 
 // Fail downloading the custom PAC script.
 TEST(ProxyScriptDeciderTest, CustomPacFails1) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   rules.AddFailDownloadRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   CapturingNetLog log;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, &log);
-  EXPECT_EQ(kFailedDownloading,
-            decider.Start(config, base::TimeDelta(), true,
-                          callback.callback()));
+  EXPECT_EQ(
+      kFailedDownloading,
+      decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(NULL, decider.script_data());
 
   // Check the NetLog was filled correctly.
   CapturingNetLog::CapturedEntryList entries;
   log.GetEntries(&entries);
 
   EXPECT_EQ(4u, entries.size());
-  EXPECT_TRUE(LogContainsBeginEvent(
-      entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsBeginEvent(entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
   EXPECT_TRUE(LogContainsBeginEvent(
       entries, 1, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEndEvent(
       entries, 2, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
-  EXPECT_TRUE(LogContainsEndEvent(
-      entries, 3, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsEndEvent(entries, 3, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
 
   EXPECT_FALSE(decider.effective_config().has_pac_url());
 }
 
 // Fail parsing the custom PAC script.
 TEST(ProxyScriptDeciderTest, CustomPacFails2) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   rules.AddFailParsingRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL);
-  EXPECT_EQ(kFailedParsing,
-            decider.Start(config, base::TimeDelta(), true,
-                          callback.callback()));
+  EXPECT_EQ(
+      kFailedParsing,
+      decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(NULL, decider.script_data());
 }
 
 // Fail downloading the custom PAC script, because the fetcher was NULL.
 TEST(ProxyScriptDeciderTest, HasNullProxyScriptFetcher) {
   Rules rules;
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   TestCompletionCallback callback;
   ProxyScriptDecider decider(NULL, &dhcp_fetcher, NULL);
-  EXPECT_EQ(ERR_UNEXPECTED,
-            decider.Start(config, base::TimeDelta(), true,
-                          callback.callback()));
+  EXPECT_EQ(
+      ERR_UNEXPECTED,
+      decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(NULL, decider.script_data());
 }
 
 // Succeeds in choosing autodetect (WPAD DNS).
 TEST(ProxyScriptDeciderTest, AutodetectSuccess) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_auto_detect(true);
 
   Rules::Rule rule = rules.AddSuccessRule("http://wpad/wpad.dat");
 
   TestCompletionCallback callback;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL);
-  EXPECT_EQ(OK, decider.Start(
-      config, base::TimeDelta(), true, callback.callback()));
+  EXPECT_EQ(
+      OK, decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(rule.text(), decider.script_data()->utf16());
 
   EXPECT_TRUE(decider.effective_config().has_pac_url());
   EXPECT_EQ(rule.url, decider.effective_config().pac_url());
 }
 
 class ProxyScriptDeciderQuickCheckTest : public ::testing::Test {
  public:
   ProxyScriptDeciderQuickCheckTest()
       : rule_(rules_.AddSuccessRule("http://wpad/wpad.dat")),
-        fetcher_(&rules_) { }
+        fetcher_(&rules_) {}
 
   virtual void SetUp() OVERRIDE {
     request_context_.set_host_resolver(&resolver_);
     fetcher_.SetRequestContext(&request_context_);
     config_.set_auto_detect(true);
     decider_.reset(new ProxyScriptDecider(&fetcher_, &dhcp_fetcher_, NULL));
   }
 
   int StartDecider() {
-    return decider_->Start(config_, base::TimeDelta(), true,
-                            callback_.callback());
+    return decider_->Start(
+        config_, base::TimeDelta(), true, callback_.callback());
   }
 
  protected:
   scoped_ptr<ProxyScriptDecider> decider_;
   MockHostResolver resolver_;
   Rules rules_;
   Rules::Rule rule_;
   TestCompletionCallback callback_;
   RuleBasedProxyScriptFetcher fetcher_;
   ProxyConfig config_;
@@ skipping 50 matching lines @@
   EXPECT_EQ(ERR_IO_PENDING, StartDecider());
   ASSERT_TRUE(resolver_.has_pending_requests());
   callback_.WaitForResult();
   EXPECT_FALSE(resolver_.has_pending_requests());
   EXPECT_FALSE(decider_->effective_config().has_pac_url());
 }
 
 // Fails if DHCP check doesn't take place before QuickCheck.
 TEST_F(ProxyScriptDeciderQuickCheckTest, QuickCheckInhibitsDhcp) {
   MockDhcpProxyScriptFetcher dhcp_fetcher;
-  const char *kPac = "function FindProxyForURL(u,h) { return \"DIRECT\"; }";
+  const char* kPac = "function FindProxyForURL(u,h) { return \"DIRECT\"; }";
   base::string16 pac_contents = base::UTF8ToUTF16(kPac);
   GURL url("http://foobar/baz");
   dhcp_fetcher.SetPacURL(url);
   decider_.reset(new ProxyScriptDecider(&fetcher_, &dhcp_fetcher, NULL));
   EXPECT_EQ(ERR_IO_PENDING, StartDecider());
   dhcp_fetcher.CompleteRequests(OK, pac_contents);
   EXPECT_TRUE(decider_->effective_config().has_pac_url());
   EXPECT_EQ(decider_->effective_config().pac_url(), url);
 }
 
 // Fails if QuickCheck still happens when disabled. To ensure QuickCheck is not
 // happening, we add a synchronous failing resolver, which would ordinarily
 // mean a QuickCheck failure, then ensure that our ProxyScriptFetcher is still
 // asked to fetch.
 TEST_F(ProxyScriptDeciderQuickCheckTest, QuickCheckDisabled) {
-  const char *kPac = "function FindProxyForURL(u,h) { return \"DIRECT\"; }";
+  const char* kPac = "function FindProxyForURL(u,h) { return \"DIRECT\"; }";
   resolver_.set_synchronous_mode(true);
   resolver_.rules()->AddSimulatedFailure("wpad");
   MockProxyScriptFetcher fetcher;
   decider_.reset(new ProxyScriptDecider(&fetcher, &dhcp_fetcher_, NULL));
   EXPECT_EQ(ERR_IO_PENDING, StartDecider());
   EXPECT_TRUE(fetcher.has_pending_request());
   fetcher.NotifyFetchCompletion(OK, kPac);
 }
 
 TEST_F(ProxyScriptDeciderQuickCheckTest, ExplicitPacUrl) {
-  const char *kCustomUrl = "http://custom/proxy.pac";
+  const char* kCustomUrl = "http://custom/proxy.pac";
   config_.set_pac_url(GURL(kCustomUrl));
   Rules::Rule rule = rules_.AddSuccessRule(kCustomUrl);
   resolver_.rules()->AddSimulatedFailure("wpad");
   resolver_.rules()->AddRule("custom", "1.2.3.4");
   EXPECT_EQ(ERR_IO_PENDING, StartDecider());
   callback_.WaitForResult();
   EXPECT_TRUE(decider_->effective_config().has_pac_url());
   EXPECT_EQ(rule.url, decider_->effective_config().pac_url());
 }
 
 // Fails at WPAD (downloading), but succeeds in choosing the custom PAC.
 TEST(ProxyScriptDeciderTest, AutodetectFailCustomSuccess1) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_auto_detect(true);
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   rules.AddFailDownloadRule("http://wpad/wpad.dat");
   Rules::Rule rule = rules.AddSuccessRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL);
-  EXPECT_EQ(OK, decider.Start(
-      config, base::TimeDelta(), true, callback.callback()));
+  EXPECT_EQ(
+      OK, decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(rule.text(), decider.script_data()->utf16());
 
   EXPECT_TRUE(decider.effective_config().has_pac_url());
   EXPECT_EQ(rule.url, decider.effective_config().pac_url());
 }
 
 // Fails at WPAD (no DHCP config, DNS PAC fails parsing), but succeeds in
 // choosing the custom PAC.
 TEST(ProxyScriptDeciderTest, AutodetectFailCustomSuccess2) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_auto_detect(true);
   config.set_pac_url(GURL("http://custom/proxy.pac"));
   config.proxy_rules().ParseFromString("unused-manual-proxy:99");
 
   rules.AddFailParsingRule("http://wpad/wpad.dat");
   Rules::Rule rule = rules.AddSuccessRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   CapturingNetLog log;
 
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, &log);
-  EXPECT_EQ(OK, decider.Start(config, base::TimeDelta(),
-                          true, callback.callback()));
+  EXPECT_EQ(
+      OK, decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(rule.text(), decider.script_data()->utf16());
 
   // Verify that the effective configuration no longer contains auto detect or
   // any of the manual settings.
   EXPECT_TRUE(decider.effective_config().Equals(
       ProxyConfig::CreateFromCustomPacURL(GURL("http://custom/proxy.pac"))));
 
   // Check the NetLog was filled correctly.
   // (Note that various states are repeated since both WPAD and custom
   // PAC scripts are tried).
   CapturingNetLog::CapturedEntryList entries;
   log.GetEntries(&entries);
 
   EXPECT_EQ(10u, entries.size());
-  EXPECT_TRUE(LogContainsBeginEvent(
-      entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsBeginEvent(entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
   // This is the DHCP phase, which fails fetching rather than parsing, so
   // there is no pair of SET_PAC_SCRIPT events.
   EXPECT_TRUE(LogContainsBeginEvent(
       entries, 1, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEndEvent(
       entries, 2, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEvent(
-      entries, 3,
+      entries,
+      3,
       NetLog::TYPE_PROXY_SCRIPT_DECIDER_FALLING_BACK_TO_NEXT_PAC_SOURCE,
       NetLog::PHASE_NONE));
   // This is the DNS phase, which attempts a fetch but fails.
   EXPECT_TRUE(LogContainsBeginEvent(
       entries, 4, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEndEvent(
       entries, 5, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEvent(
-      entries, 6,
+      entries,
+      6,
       NetLog::TYPE_PROXY_SCRIPT_DECIDER_FALLING_BACK_TO_NEXT_PAC_SOURCE,
       NetLog::PHASE_NONE));
   // Finally, the custom PAC URL phase.
   EXPECT_TRUE(LogContainsBeginEvent(
       entries, 7, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEndEvent(
       entries, 8, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
-  EXPECT_TRUE(LogContainsEndEvent(
-      entries, 9, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsEndEvent(entries, 9, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
 }
 
 // Fails at WPAD (downloading), and fails at custom PAC (downloading).
 TEST(ProxyScriptDeciderTest, AutodetectFailCustomFails1) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_auto_detect(true);
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   rules.AddFailDownloadRule("http://wpad/wpad.dat");
   rules.AddFailDownloadRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL);
-  EXPECT_EQ(kFailedDownloading,
-            decider.Start(config, base::TimeDelta(), true,
-                          callback.callback()));
+  EXPECT_EQ(
+      kFailedDownloading,
+      decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(NULL, decider.script_data());
 }
 
 // Fails at WPAD (downloading), and fails at custom PAC (parsing).
 TEST(ProxyScriptDeciderTest, AutodetectFailCustomFails2) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_auto_detect(true);
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   rules.AddFailDownloadRule("http://wpad/wpad.dat");
   rules.AddFailParsingRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL);
-  EXPECT_EQ(kFailedParsing,
-            decider.Start(config, base::TimeDelta(), true,
-                          callback.callback()));
+  EXPECT_EQ(
+      kFailedParsing,
+      decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(NULL, decider.script_data());
 }
 
 // This is a copy-paste of CustomPacFails1, with the exception that we give it
 // a 1 millisecond delay. This means it will now complete asynchronously.
 // Moreover, we test the NetLog to make sure it logged the pause.
 TEST(ProxyScriptDeciderTest, CustomPacFails1_WithPositiveDelay) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   rules.AddFailDownloadRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   CapturingNetLog log;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, &log);
   EXPECT_EQ(ERR_IO_PENDING,
-            decider.Start(config, base::TimeDelta::FromMilliseconds(1),
-                      true, callback.callback()));
+            decider.Start(config,
+                          base::TimeDelta::FromMilliseconds(1),
+                          true,
+                          callback.callback()));
 
   EXPECT_EQ(kFailedDownloading, callback.WaitForResult());
   EXPECT_EQ(NULL, decider.script_data());
 
   // Check the NetLog was filled correctly.
   CapturingNetLog::CapturedEntryList entries;
   log.GetEntries(&entries);
 
   EXPECT_EQ(6u, entries.size());
-  EXPECT_TRUE(LogContainsBeginEvent(
-      entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsBeginEvent(entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
   EXPECT_TRUE(LogContainsBeginEvent(
       entries, 1, NetLog::TYPE_PROXY_SCRIPT_DECIDER_WAIT));
-  EXPECT_TRUE(LogContainsEndEvent(
-      entries, 2, NetLog::TYPE_PROXY_SCRIPT_DECIDER_WAIT));
+  EXPECT_TRUE(
+      LogContainsEndEvent(entries, 2, NetLog::TYPE_PROXY_SCRIPT_DECIDER_WAIT));
   EXPECT_TRUE(LogContainsBeginEvent(
       entries, 3, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEndEvent(
       entries, 4, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
-  EXPECT_TRUE(LogContainsEndEvent(
-      entries, 5, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsEndEvent(entries, 5, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
 }
 
 // This is a copy-paste of CustomPacFails1, with the exception that we give it
 // a -5 second delay instead of a 0 ms delay. This change should have no effect
 // so the rest of the test is unchanged.
 TEST(ProxyScriptDeciderTest, CustomPacFails1_WithNegativeDelay) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   DoNothingDhcpProxyScriptFetcher dhcp_fetcher;
 
   ProxyConfig config;
   config.set_pac_url(GURL("http://custom/proxy.pac"));
 
   rules.AddFailDownloadRule("http://custom/proxy.pac");
 
   TestCompletionCallback callback;
   CapturingNetLog log;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, &log);
-  EXPECT_EQ(kFailedDownloading,
-            decider.Start(config, base::TimeDelta::FromSeconds(-5),
-                          true, callback.callback()));
+  EXPECT_EQ(
+      kFailedDownloading,
+      decider.Start(
+          config, base::TimeDelta::FromSeconds(-5), true, callback.callback()));
   EXPECT_EQ(NULL, decider.script_data());
 
   // Check the NetLog was filled correctly.
   CapturingNetLog::CapturedEntryList entries;
   log.GetEntries(&entries);
 
   EXPECT_EQ(4u, entries.size());
-  EXPECT_TRUE(LogContainsBeginEvent(
-      entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsBeginEvent(entries, 0, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
   EXPECT_TRUE(LogContainsBeginEvent(
       entries, 1, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
   EXPECT_TRUE(LogContainsEndEvent(
       entries, 2, NetLog::TYPE_PROXY_SCRIPT_DECIDER_FETCH_PAC_SCRIPT));
-  EXPECT_TRUE(LogContainsEndEvent(
-      entries, 3, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
+  EXPECT_TRUE(
+      LogContainsEndEvent(entries, 3, NetLog::TYPE_PROXY_SCRIPT_DECIDER));
 }
 
 class SynchronousSuccessDhcpFetcher : public DhcpProxyScriptFetcher {
  public:
   explicit SynchronousSuccessDhcpFetcher(const base::string16& expected_text)
-      : gurl_("http://dhcppac/"), expected_text_(expected_text) {
-  }
+      : gurl_("http://dhcppac/"), expected_text_(expected_text) {}
 
   virtual int Fetch(base::string16* utf16_text,
                     const CompletionCallback& callback) OVERRIDE {
     *utf16_text = expected_text_;
     return OK;
   }
 
-  virtual void Cancel() OVERRIDE {
-  }
+  virtual void Cancel() OVERRIDE {}
 
-  virtual const GURL& GetPacURL() const OVERRIDE {
-    return gurl_;
-  }
+  virtual const GURL& GetPacURL() const OVERRIDE { return gurl_; }
 
-  const base::string16& expected_text() const {
-    return expected_text_;
-  }
+  const base::string16& expected_text() const { return expected_text_; }
 
  private:
   GURL gurl_;
   base::string16 expected_text_;
 
   DISALLOW_COPY_AND_ASSIGN(SynchronousSuccessDhcpFetcher);
 };
 
 // All of the tests above that use ProxyScriptDecider have tested
 // failure to fetch a PAC file via DHCP configuration, so we now test
 // success at downloading and parsing, and then success at downloading,
 // failure at parsing.
 
 TEST(ProxyScriptDeciderTest, AutodetectDhcpSuccess) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   SynchronousSuccessDhcpFetcher dhcp_fetcher(
       base::WideToUTF16(L"http://bingo/!FindProxyForURL"));
 
   ProxyConfig config;
   config.set_auto_detect(true);
 
   rules.AddSuccessRule("http://bingo/");
   rules.AddFailDownloadRule("http://wpad/wpad.dat");
 
   TestCompletionCallback callback;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL);
-  EXPECT_EQ(OK, decider.Start(
-      config, base::TimeDelta(), true, callback.callback()));
-  EXPECT_EQ(dhcp_fetcher.expected_text(),
-            decider.script_data()->utf16());
+  EXPECT_EQ(
+      OK, decider.Start(config, base::TimeDelta(), true, callback.callback()));
+  EXPECT_EQ(dhcp_fetcher.expected_text(), decider.script_data()->utf16());
 
   EXPECT_TRUE(decider.effective_config().has_pac_url());
   EXPECT_EQ(GURL("http://dhcppac/"), decider.effective_config().pac_url());
 }
 
 TEST(ProxyScriptDeciderTest, AutodetectDhcpFailParse) {
   Rules rules;
   RuleBasedProxyScriptFetcher fetcher(&rules);
   SynchronousSuccessDhcpFetcher dhcp_fetcher(
       base::WideToUTF16(L"http://bingo/!invalid-script"));
 
   ProxyConfig config;
   config.set_auto_detect(true);
 
   rules.AddFailParsingRule("http://bingo/");
   rules.AddFailDownloadRule("http://wpad/wpad.dat");
 
   TestCompletionCallback callback;
   ProxyScriptDecider decider(&fetcher, &dhcp_fetcher, NULL);
   // Since there is fallback to DNS-based WPAD, the final error will be that
   // it failed downloading, not that it failed parsing.
-  EXPECT_EQ(kFailedDownloading,
+  EXPECT_EQ(
+      kFailedDownloading,
       decider.Start(config, base::TimeDelta(), true, callback.callback()));
   EXPECT_EQ(NULL, decider.script_data());
 
   EXPECT_FALSE(decider.effective_config().has_pac_url());
 }
 
 class AsyncFailDhcpFetcher
     : public DhcpProxyScriptFetcher,
       public base::SupportsWeakPtr<AsyncFailDhcpFetcher> {
  public:
   AsyncFailDhcpFetcher() {}
   virtual ~AsyncFailDhcpFetcher() {}
 
   virtual int Fetch(base::string16* utf16_text,
                     const CompletionCallback& callback) OVERRIDE {
     callback_ = callback;
     base::MessageLoop::current()->PostTask(
         FROM_HERE,
         base::Bind(&AsyncFailDhcpFetcher::CallbackWithFailure, AsWeakPtr()));
     return ERR_IO_PENDING;
   }
 
-  virtual void Cancel() OVERRIDE {
-    callback_.Reset();
-  }
+  virtual void Cancel() OVERRIDE { callback_.Reset(); }
 
-  virtual const GURL& GetPacURL() const OVERRIDE {
-    return dummy_gurl_;
-  }
+  virtual const GURL& GetPacURL() const OVERRIDE { return dummy_gurl_; }
 
   void CallbackWithFailure() {
     if (!callback_.is_null())
       callback_.Run(ERR_PAC_NOT_IN_DHCP);
   }
 
  private:
   GURL dummy_gurl_;
   CompletionCallback callback_;
 };
@@ skipping 22 matching lines @@
 
   // Run the message loop to let the DHCP fetch complete and post the results
   // back. Before the fix linked to above, this would try to invoke on
   // the callback object provided by ProxyScriptDecider after it was
   // no longer valid.
   base::MessageLoop::current()->RunUntilIdle();
 }
 
 }  // namespace
 }  // namespace net