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

shell/application_manager/application_manager_unittest.cc

Index: shell/application_manager/application_manager_unittest.cc
diff --git a/shell/application_manager/application_manager_unittest.cc b/shell/application_manager/application_manager_unittest.cc
index 5c3b2858f444fce5b78bb594c1a3ab45d8de3361..19cd0013111879d58f680009ace40463dd87aa15 100644
--- a/shell/application_manager/application_manager_unittest.cc
+++ b/shell/application_manager/application_manager_unittest.cc
@@ -390,9 +390,7 @@ class Tester : public ApplicationDelegate,
 
 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 {
@@ -414,8 +412,7 @@ class TestDelegate : public ApplicationManager::Delegate {
     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_;
@@ -741,8 +738,8 @@ TEST_F(ApplicationManagerTest, ExternalApp) {
                                                     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());
 };