Changes ViewManagerServiceImpl so that it no longer subclasses InterfaceImpl

mojo/services/view_manager/client_connection.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 "mojo/services/view_manager/client_connection.h"
+
+#include "mojo/services/view_manager/connection_manager.h"
+#include "mojo/services/view_manager/view_manager_service_impl.h"
+
+namespace mojo {
+namespace service {
+
+ClientConnection::ClientConnection(scoped_ptr<ViewManagerServiceImpl> service)
+    : service_(service.Pass()), client_(nullptr) {
+}
+
+ClientConnection::~ClientConnection() {
+}
+
+DefaultClientConnection::DefaultClientConnection(
+    ConnectionManager* connection_manager,

[msw, 2014/11/14 01:42:18]

nit: swap the arg order to match the base class and init list use.

[sky, 2014/11/14 20:56:03]

Done.

+    scoped_ptr<ViewManagerServiceImpl> service_impl)
+    : ClientConnection(service_impl.Pass()),
+      connection_manager_(connection_manager),
+      binding_(service()) {
+  binding_.set_error_handler(this);
+}
+
+DefaultClientConnection::~DefaultClientConnection() {
+}
+
+void DefaultClientConnection::OnConnectionError() {
+  connection_manager_->OnConnectionError(this);
+}
+
+}  // namespace service
+}  // namespace mojo