mojo: Update content handler API

mojo/application/content_handler.cc

Index: mojo/application/content_handler.cc
diff --git a/mojo/application/content_handler.cc b/mojo/application/content_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..9d68cb746e782a1b40e20efd76068d4fd0426337
--- /dev/null
+++ b/mojo/application/content_handler.cc
@@ -0,0 +1,117 @@
+// 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/application/content_handler.h"
+
+#include <set>
+
+#include "base/bind.h"
+#include "base/callback.h"
+#include "base/threading/platform_thread.h"
+#include "mojo/application/application_runner_chromium.h"
+#include "mojo/common/message_pump_mojo.h"
+#include "mojo/public/cpp/application/application_connection.h"
+#include "mojo/public/cpp/application/application_delegate.h"
+#include "mojo/public/cpp/application/application_impl.h"
+#include "mojo/public/cpp/application/interface_factory_impl.h"
+#include "mojo/public/cpp/bindings/interface_impl.h"
+#include "mojo/services/public/interfaces/content_handler/content_handler.mojom.h"
+
+namespace mojo {
+
+namespace {
+
+class ApplicationThread : public base::PlatformThread::Delegate {
+ public:
+  ApplicationThread(
+      scoped_refptr<base::MessageLoopProxy> handler_thread,
+      const base::Callback<void(ApplicationThread*)>& termination_callback,
+      ContentHandlerFactory::Delegate* handler_delegate,
+      ShellPtr shell,
+      URLResponsePtr response)
+      : handler_thread_(handler_thread),
+        termination_callback_(termination_callback),
+        handler_delegate_(handler_delegate),
+        shell_(shell.Pass()),
+        response_(response.Pass()) {}
+
+ private:
+  void ThreadMain() override {
+    base::MessageLoop loop(common::MessagePumpMojo::Create());

[Aaron Boodman, 2014/10/31 16:40:46]

I thought you wanted to support content handlers that didn't use message loops.
Are you thinking that those would not use this infrastructure?

+    application_ =
+        handler_delegate_->CreateApplication(shell_.Pass(), response_.Pass());
+    loop.Run();
+    handler_thread_->PostTask(FROM_HERE,
+                              base::Bind(termination_callback_, this));
+  }
+
+  scoped_refptr<base::MessageLoopProxy> handler_thread_;
+  base::Callback<void(ApplicationThread*)> termination_callback_;
+  ContentHandlerFactory::Delegate* handler_delegate_;
+  ShellPtr shell_;
+  URLResponsePtr response_;
+  scoped_ptr<ContentHandlerFactory::HandledApplicationHolder> application_;
+
+  DISALLOW_COPY_AND_ASSIGN(ApplicationThread);
+};
+
+class ContentHandlerImpl : public InterfaceImpl<ContentHandler> {
+ public:
+  explicit ContentHandlerImpl(ContentHandlerFactory::Delegate* delegate)
+      : delegate_(delegate) {}
+  ~ContentHandlerImpl() {
+    for (auto thread : active_threads_) {
+      base::PlatformThread::Join(thread.second);
+      delete thread.first;
+    }
+  }
+
+ private:
+  // Overridden from ContentHandler:
+  virtual void StartApplication(ShellPtr shell,
+                                URLResponsePtr response) override {
+    ApplicationThread* thread = new ApplicationThread(
+        base::MessageLoopProxy::current(),
+        base::Bind(&ContentHandlerImpl::OnThreadEnd, base::Unretained(this)),
+        delegate_,
+        shell.Pass(),
+        response.Pass());
+    base::PlatformThreadHandle handle;
+    bool launched = base::PlatformThread::Create(0, thread, &handle);
+    DCHECK(launched);
+    active_threads_[thread] = handle;
+  }
+
+  void OnThreadEnd(ApplicationThread* thread) {
+    DCHECK(active_threads_.find(thread) != active_threads_.end());
+    base::PlatformThreadHandle handle = active_threads_[thread];
+    active_threads_.erase(thread);
+    base::PlatformThread::Join(handle);
+    delete thread;
+    if (!active_threads_.size()) {
+      ApplicationImpl::Terminate();
+    }
+  }
+
+  ContentHandlerFactory::Delegate* delegate_;
+  std::map<ApplicationThread*, base::PlatformThreadHandle> active_threads_;
+
+  DISALLOW_COPY_AND_ASSIGN(ContentHandlerImpl);
+};
+
+}  // namespace
+
+ContentHandlerFactory::ContentHandlerFactory(Delegate* delegate)
+    : delegate_(delegate) {
+}
+
+ContentHandlerFactory::~ContentHandlerFactory() {
+}
+
+void ContentHandlerFactory::Create(ApplicationConnection* connection,
+                                   InterfaceRequest<ContentHandler> request) {
+  BindToRequest(new ContentHandlerImpl(delegate_), &request);
+}
+
+}  // namespace mojo