Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(647)

Unified Diff: mojo/examples/pepper_container_app/pepper_container_app.cc

Issue 218763002: Change AckEvent into request / response (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/examples/pepper_container_app/pepper_container_app.cc
diff --git a/mojo/examples/pepper_container_app/pepper_container_app.cc b/mojo/examples/pepper_container_app/pepper_container_app.cc
index ec45a45c96b6ba9ad4aa0d409d501df740481966..992ffdc6123dd538d9f20a6f6452031b3f07426d 100644
--- a/mojo/examples/pepper_container_app/pepper_container_app.cc
+++ b/mojo/examples/pepper_container_app/pepper_container_app.cc
@@ -90,7 +90,8 @@ class PepperContainerApp: public Application,
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
return;
@@ -99,7 +100,7 @@ class PepperContainerApp: public Application,
// TODO(yzshen): Handle events.
}
- viewport_->AckEvent(event);
+ callback.Run();
}
// MojoPpapiGlobals::Delegate implementation.

Powered by Google App Engine
This is Rietveld 408576698