Change AckEvent into request / response

mojo/examples/pepper_container_app/pepper_container_app.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/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "build/build_config.h"
 #include "mojo/examples/pepper_container_app/mojo_ppapi_globals.h"
@@ skipping 72 matching lines @@
     base::MessageLoop::current()->Quit();
   }
 
   virtual void OnBoundsChanged(const Rect& bounds) OVERRIDE {
     ppapi::ProxyAutoLock lock;
 
     if (plugin_instance_)
       plugin_instance_->DidChangeView(bounds);
   }
 
-  virtual void OnEvent(const Event& event) OVERRIDE {
+  virtual void OnEvent(const Event& event,
+                       const mojo::Callback<void()>& callback) OVERRIDE {
     if (event.location().is_null())

[darin (slow to review), 2014/03/31 03:24:05]

it might be good to leave a comment here checking if it makes sense to suppress
the callback in this case. the caller of OnEvent will be left hanging if we
return early here. is that ok?

       return;
 
     {
       ppapi::ProxyAutoLock lock;
 
       // TODO(yzshen): Handle events.
     }
-    viewport_->AckEvent(event);
+    callback.Run();
   }
 
   // MojoPpapiGlobals::Delegate implementation.
   virtual ScopedMessagePipeHandle CreateGLES2Context() OVERRIDE {
     MessagePipe gles2_pipe;
     viewport_->CreateGLES2Context(gles2_pipe.handle1.Pass());
     return gles2_pipe.handle0.Pass();
   }
 
  private:
@@ skipping 12 matching lines @@
 extern "C" PEPPER_CONTAINER_APP_EXPORT MojoResult CDECL MojoMain(
     MojoHandle shell_handle) {
   mojo::Environment env;
   mojo::GLES2Initializer gles2;
   base::MessageLoop run_loop;
   mojo::examples::PepperContainerApp app(shell_handle);
 
   run_loop.Run();
   return MOJO_RESULT_OK;
 }