clang-format //shell and add a PRESUBMIT.py to keep it clang-formatted.

shell/application_manager/application_manager_unittest.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/at_exit.h"
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/memory/scoped_vector.h"
 #include "base/message_loop/message_loop.h"
 #include "mojo/public/cpp/application/application_connection.h"
@@ skipping 372 matching lines @@
   }
 
   TesterContext* context_;
   scoped_ptr<ApplicationImpl> app_;
   std::string requestor_url_;
   ScopedVector<TestAImpl> a_bindings_;
 };
 
 class TestDelegate : public ApplicationManager::Delegate {
  public:
-  void AddMapping(const GURL& from, const GURL& to) {
-    mappings_[from] = to;
-  }
+  void AddMapping(const GURL& from, const GURL& to) { mappings_[from] = to; }
 
   // ApplicationManager::Delegate
   virtual GURL ResolveMappings(const GURL& url) override {
     auto it = mappings_.find(url);
     if (it != mappings_.end())
       return it->second;
     return url;
   }
 
   // ApplicationManager::Delegate
   virtual GURL ResolveURL(const GURL& url) override {
     GURL mapped_url = ResolveMappings(url);
     // The shell automatically map mojo URLs.
     if (mapped_url.scheme() == "mojo") {
       url::Replacements<char> replacements;
       replacements.SetScheme("file", url::Component(0, 4));
       mapped_url = mapped_url.ReplaceComponents(replacements);
     }
     return mapped_url;
   }
 
-  virtual void OnApplicationError(const GURL& url) override {
-  }
+  virtual void OnApplicationError(const GURL& url) override {}

[jamesr, 2015/02/11 23:44:48]

while you're here, this shouldn't have both "virtual" and "override" (the clang
checker is just ignoring this because of the test namespace i believe)

[viettrungluu, 2015/02/11 23:59:03]

Fixed. I wonder where else we make that mistake.

 
  private:
   std::map<GURL, GURL> mappings_;
 };
 
 class TestExternal : public ApplicationDelegate {
  public:
   TestExternal() : configure_incoming_connection_called_(false) {}
 
   virtual void Initialize(ApplicationImpl* app) override {
@@ skipping 305 matching lines @@
 TEST_F(ApplicationManagerTest, ExternalApp) {
   ApplicationPtr application;
   TestExternal external;
   std::vector<std::string> args;
   args.push_back("test");
   ApplicationImpl app(&external, GetProxy(&application));
   application_manager_->RegisterExternalApplication(GURL("mojo:test"), args,
                                                     application.Pass());
   loop_.Run();
   EXPECT_EQ(args, external.initialize_args());
-  application_manager_->ConnectToServiceByName(
-      GURL("mojo:test"), std::string());
+  application_manager_->ConnectToServiceByName(GURL("mojo:test"),
+                                               std::string());
   loop_.Run();
   EXPECT_TRUE(external.configure_incoming_connection_called());
 };
 
 }  // namespace mojo