Mojo: Make overriding OnConnectionError optional

mojo/service_manager/service_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/message_loop/message_loop.h"
 #include "mojo/public/cpp/bindings/allocation_scope.h"
 #include "mojo/public/cpp/environment/environment.h"
 #include "mojo/public/cpp/shell/application.h"
 #include "mojo/public/interfaces/shell/shell.mojom.h"
 #include "mojo/service_manager/service_loader.h"
@@ skipping 16 matching lines @@
 class TestServiceImpl : public InterfaceImpl<TestService> {
  public:
   explicit TestServiceImpl(TestContext* context) : context_(context) {
     ++context_->num_impls;
   }
 
   virtual ~TestServiceImpl() {
     --context_->num_impls;
   }
 
-  // InterfaceImpl<TestService> implementation.
-  virtual void OnConnectionError() OVERRIDE {
-  }
-
   // TestService implementation:
   virtual void Test(const mojo::String& test_string) OVERRIDE {
     context_->last_test_string = test_string.To<std::string>();
     client()->AckTest();
   }
 
  private:
   TestContext* context_;
 };
 
@@ skipping 223 matching lines @@
 
   std::string url("test:test3");
   MessagePipe pipe1;
   sm.Connect(GURL(url), pipe1.handle0.Pass());
   EXPECT_EQ(1, interceptor.call_count());
   EXPECT_EQ(url, interceptor.url_spec());
   EXPECT_EQ(1, default_loader->num_loads());
 }
 
 }  // namespace mojo