Work in progress for end-to-end bindings

mojo/shell/app_container.cc

 // Copyright 2013 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/shell/app_container.h"
 
 #include "base/bind.h"
 #include "base/callback_forward.h"
 #include "base/callback_helpers.h"
 #include "base/file_util.h"
@@ skipping 49 matching lines @@
 
 AppContainer::~AppContainer() {
 }
 
 void AppContainer::Load(const GURL& app_url) {
   request_ = context_->loader()->Load(app_url, this);
 }
 
 void AppContainer::DidCompleteLoad(const GURL& app_url,
                                    const base::FilePath& app_path) {
+  Handle shell_handle;
   Handle app_handle;
-  MojoResult result = CreateMessagePipe(&shell_handle_, &app_handle);
+  MojoResult result = CreateMessagePipe(&shell_handle, &app_handle);
   if (result < MOJO_RESULT_OK) {
     // Failure..
   }
 
+  hello_world_service_.reset(
+    new examples::HelloWorldServiceImpl(shell_handle));
+
   // Launch the app on its own thread.
   // TODO(beng): Create a unique thread name.
   thread_.reset(new base::Thread("app_thread"));
   thread_->Start();
   thread_->message_loop_proxy()->PostTaskAndReply(
       FROM_HERE,
       base::Bind(&LaunchAppOnThread, app_path, app_handle),
       base::Bind(&AppContainer::AppCompleted, weak_factory_.GetWeakPtr()));
 
-  const char* hello_msg = "Hello";
-  result = WriteMessage(shell_handle_, hello_msg,
-                        static_cast<uint32_t>(strlen(hello_msg)+1),
-                        NULL, 0, MOJO_WRITE_MESSAGE_FLAG_NONE);
-  if (result < MOJO_RESULT_OK) {
-    // Failure..
-  }
-
   // TODO(beng): This should be created on demand by the NativeViewportService
   //             when it is retrieved by the app.
-  native_viewport_controller_.reset(
-      new services::NativeViewportController(context_, shell_handle_));
+  // native_viewport_controller_.reset(
+  //     new services::NativeViewportController(context_, shell_handle_));
 }
 
 void AppContainer::AppCompleted() {
-  native_viewport_controller_->Close();
+  hello_world_service_.reset();
+  // native_viewport_controller_->Close();

[darin (slow to review), 2013/11/14 00:05:34]

nit: commented out code should have a TODO

[DaveMoore, 2013/11/14 23:46:54]

Done.

 
   thread_.reset();
-  Close(shell_handle_);
 }
 
 }  // namespace shell
 }  // namespace mojo