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

Unified Diff: content/renderer/pepper/resource_converter.cc

Issue 264303002: PPAPI: Implement synchronous postMessage (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: defer some changes Created 6 years, 6 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: content/renderer/pepper/resource_converter.cc
diff --git a/content/renderer/pepper/resource_converter.cc b/content/renderer/pepper/resource_converter.cc
index 476e3d37413c5bd4e6726ee6a4271e16d6564217..f58e55225a5d26f4f4ceca0e8700d3a633e62679 100644
--- a/content/renderer/pepper/resource_converter.cc
+++ b/content/renderer/pepper/resource_converter.cc
@@ -274,6 +274,15 @@ void ResourceConverterImpl::Flush(const base::Callback<void(bool)>& callback) {
browser_vars_.clear();
}
+bool ResourceConverterImpl::FlushSync() {
+ if (!browser_host_create_messages_.empty()) {
+ // TODO(dmichael): Make these work; we have to make a synchronous path to
+ // create browser-side hosts. crbug.com/367896
+ return false;
+ }
+ return true;
+}
+
bool ResourceConverterImpl::ToV8Value(const PP_Var& var,
v8::Handle<v8::Context> context,
v8::Handle<v8::Value>* result) {

Powered by Google App Engine
This is Rietveld 408576698