Make sure that Content Handled application can be connected multiple times.

sky/viewer/document_view.cc

Index: sky/viewer/document_view.cc
diff --git a/sky/viewer/document_view.cc b/sky/viewer/document_view.cc
index 499829902106d4db3980bd489355785522bf5cb6..b0058e16aafba4a134389ecde3ee2068ec44d6d7 100644
--- a/sky/viewer/document_view.cc
+++ b/sky/viewer/document_view.cc
@@ -67,19 +67,23 @@ mojo::Target WebNavigationPolicyToNavigationTarget(
 static int s_next_debugger_id = 1;
 
 DocumentView::DocumentView(
+    const base::Closure& destruction_callback,
+    mojo::ServiceProviderPtr provider,
     mojo::URLResponsePtr response,
-    mojo::ShellPtr shell,
+    mojo::Shell* shell,
     scoped_refptr<base::MessageLoopProxy> compositor_thread)
-    : response_(response.Pass()),
-      shell_(shell.Pass()),
+    : destruction_callback_(destruction_callback),
+      response_(response.Pass()),
+      shell_(shell),
       web_view_(NULL),
       root_(NULL),
-      view_manager_client_factory_(shell_.get(), this),
+      view_manager_client_factory_(shell_, this),
       inspector_service_factory_(this),
       compositor_thread_(compositor_thread),
       debugger_id_(s_next_debugger_id++),
       weak_factory_(this) {
-  shell_.set_client(this);
+  exported_services_.AddService(&view_manager_client_factory_);
+  mojo::WeakBindToPipe(&exported_services_, provider.PassMessagePipe());
 }
 
 DocumentView::~DocumentView() {
@@ -87,21 +91,13 @@ DocumentView::~DocumentView() {
     web_view_->close();
   if (root_)
     root_->RemoveObserver(this);
+  destruction_callback_.Run();
 }
 
 base::WeakPtr<DocumentView> DocumentView::GetWeakPtr() {
   return weak_factory_.GetWeakPtr();
 }
 
-void DocumentView::AcceptConnection(const mojo::String& requestor_url,
-                                    mojo::ServiceProviderPtr provider) {
-  exported_services_.AddService(&view_manager_client_factory_);
-  mojo::WeakBindToPipe(&exported_services_, provider.PassMessagePipe());
-}
-
-void DocumentView::Initialize(mojo::Array<mojo::String> args) {
-}
-
 void DocumentView::OnEmbed(
     mojo::ViewManager* view_manager,
     mojo::View* root,
@@ -211,7 +207,7 @@ mojo::NavigatorHost* DocumentView::NavigatorHost() {
 }
 
 mojo::Shell* DocumentView::Shell() {
-  return shell_.get();
+  return shell_;
 }
 
 void DocumentView::OnViewBoundsChanged(mojo::View* view,