Add a flag field to PP_CompletionCallback to control if the callback should

always be invoked asynchronously on success or error or skipped if the
operation can complete synchronously without blocking. Keep the default
behavior as-is  until clients update their code. Bump revisions of all
interfaces that take callbacks as args. Update browser interface function
implementations and C++ layer to force callbacks if sync option is not set.
Change ppapi/tests to run tests involving callbacks with both flag options.

BUG=79376
TEST=ppapi_tests + bots
Review URL: http://codereview.chromium.org/6899055

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91205 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 2 deletions

diff --git a/remoting/client/plugin/pepper_port_allocator_session.cc b/remoting/client/plugin/pepper_port_allocator_session.cc
index 8c5c598..ec7544e9 100644
--- a/remoting/client/plugin/pepper_port_allocator_session.cc
+++ b/remoting/client/plugin/pepper_port_allocator_session.cc
@@ -121,7 +121,7 @@ class PepperURLFetcher {
     fetch_callback_.reset(fetch_callback);
 
     pp::CompletionCallback callback =
-        callback_factory_.NewCallback(&PepperURLFetcher::DidOpen);
+        callback_factory_.NewOptionalCallback(&PepperURLFetcher::DidOpen);
     int rv = loader_.Open(request, callback);
     if (rv != PP_OK_COMPLETIONPENDING)
       callback.Run(rv);
@@ -130,7 +130,7 @@ class PepperURLFetcher {
  private:
   void ReadMore() {
     pp::CompletionCallback callback =
-        callback_factory_.NewCallback(&PepperURLFetcher::DidRead);
+        callback_factory_.NewOptionalCallback(&PepperURLFetcher::DidRead);
     int rv = loader_.ReadResponseBody(buf_, sizeof(buf_), callback);
     if (rv != PP_OK_COMPLETIONPENDING)
       callback.Run(rv);