PPAPI: Force async callback invocation option.

ppapi/cpp/completion_callback.h

 // Copyright (c) 2011 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.
 
 #ifndef PPAPI_CPP_COMPLETION_CALLBACK_H_
 #define PPAPI_CPP_COMPLETION_CALLBACK_H_
 
 #include "ppapi/c/pp_completion_callback.h"
+#include "ppapi/c/pp_errors.h"
 #include "ppapi/cpp/logging.h"
 #include "ppapi/cpp/non_thread_safe_ref_count.h"
 
 namespace pp {
 
 // A CompletionCallback provides a wrapper around PP_CompletionCallback.
 class CompletionCallback {
  public:
   // Use this special constructor to create a 'blocking' CompletionCallback
   // that may be passed to a method to indicate that the calling thread should
   // be blocked until the asynchronous operation corresponding to the method
   // completes.
   struct Block {};
   CompletionCallback(Block) {
     cc_ = PP_BlockUntilComplete();
   }
 
   CompletionCallback(PP_CompletionCallback_Func func, void* user_data) {
     cc_ = PP_MakeCompletionCallback(func, user_data);
   }
 
+  CompletionCallback(PP_CompletionCallback_Func func, void* user_data,
+                     int32_t flags) {
+    cc_ = PP_MakeCompletionCallback(func, user_data);
+    cc_.flags = flags;
+  }
+
+  void set_flags(int32_t flags) { cc_.flags = flags; }
+
   // Call this method to explicitly run the CompletionCallback.  Normally, the
   // system runs a CompletionCallback after an asynchronous operation
   // completes, but programs may wish to run the CompletionCallback manually
   // in order to reuse the same code paths.
   void Run(int32_t result) {
     PP_DCHECK(cc_.func);
     PP_RunCompletionCallback(&cc_, result);
   }
 
+  bool IsOptional() const {
+    return  (cc_.func == NULL ||

[piman, 2011/06/07 17:32:14]

nit: only 1 space after return

[polina, 2011/06/09 23:53:51]

good eye! Done.

+             (cc_.flags & PP_COMPLETIONCALLBACK_FLAG_OPTIONAL) != 0);

[piman, 2011/06/07 17:32:14]

nit: Indent looks wrong

[polina, 2011/06/09 23:53:51]

could you be more specific please?

[piman, 2011/06/10 00:51:20]

4 spaces for continuations ?

[polina, 2011/06/10 19:33:19]

Not in this case because we have () and break after an operator.
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Return_Values

+  }
+
   const PP_CompletionCallback& pp_completion_callback() const { return cc_; }
 
+  int32_t MayForce(int32_t result) const {
+    if (result == PP_OK_COMPLETIONPENDING || IsOptional())
+      return result;
+    Module::Get()->core()->CallOnMainThread(0, *this, result);
+    return PP_OK_COMPLETIONPENDING;
+  }
+
  protected:
   PP_CompletionCallback cc_;
 };
 
 // CompletionCallbackFactory<T> may be used to create CompletionCallback
 // objects that are bound to member functions.
 //
 // If a factory is destroyed, then any pending callbacks will be cancelled
 // preventing any bound member functions from being called.  The CancelAll
 // method allows pending callbacks to be cancelled without destroying the
@@ skipping 11 matching lines @@
 // EXAMPLE USAGE:
 //
 //   class MyHandler {
 //    public:
 //     MyHandler() : factory_(this), offset_(0) {
 //     }
 //
 //     void ProcessFile(const FileRef& file) {
 //       CompletionCallback cc = factory_.NewCallback(&MyHandler::DidOpen);
 //       int32_t rv = fio_.Open(file, PP_FileOpenFlag_Read, cc);
-//       if (rv != PP_OK_COMPLETIONPENDING)
-//         cc.Run(rv);
+//       CHECK(rv == PP_OK_COMPLETIONPENDING);
 //     }
 //
 //    private:
 //     CompletionCallback NewCallback() {
 //       return factory_.NewCallback(&MyHandler::DidCompleteIO);
 //     }
 //
 //     void DidOpen(int32_t result) {
 //       if (result == PP_OK) {
 //         // The file is open, and we can begin reading.
 //         offset_ = 0;
 //         ReadMore();
 //       } else {
 //         // Failed to open the file with error given by 'result'.
 //       }
 //     }
 //
 //     void DidRead(int32_t result) {
 //       if (result > 0) {
 //         // buf_ now contains 'result' number of bytes from the file.
 //         ProcessBytes(buf_, result);
 //         offset_ += result;
 //         ReadMore();
 //       } else {
 //         // Done reading (possibly with an error given by 'result').
 //       }
 //     }
 //
 //     void ReadMore() {
-//       CompletionCallback cc = factory_.NewCallback(&MyHandler::DidRead);
+//       CompletionCallback cc =
+//           factory_.NewOptionalCallback(&MyHandler::DidRead);
 //       int32_t rv = fio_.Read(offset_, buf_, sizeof(buf_),
 //                              cc.pp_completion_callback());
 //       if (rv != PP_OK_COMPLETIONPENDING)
 //         cc.Run(rv);
 //     }
 //
 //     void ProcessBytes(const char* bytes, int32_t length) {
 //       // Do work ...
 //     }
 //
@@ skipping 24 matching lines @@
   void Initialize(T* object) {
     PP_DCHECK(object);
     PP_DCHECK(!object_);  // May only initialize once!
     object_ = object;
   }
 
   T* GetObject() {
     return object_;
   }
 
-  // Allocates a new, single-use CompletionCallback.  The CompletionCallback
-  // must be run in order for the memory allocated by NewCallback to be freed.
-  // If after passing the CompletionCallback to a PPAPI method, the method does
-  // not return PP_OK_COMPLETIONPENDING, then you should manually call the
-  // CompletionCallback's Run method otherwise memory will be leaked.
+  // Methods for allocating new, single-use CompletionCallbacks.
+  // The CompletionCallback must be run in order for the memory
+  // allocated by NewCallback to be freed. NewCallback() creates callbacks that
+  // will always run. NewOptionalCallback() creates callbacks that might
+  // not run if the method taking them can complete synchronously.
+  // Thus, if after passing the CompletionCallback to a PPAPI method, the
+  // method does not return PP_OK_COMPLETIONPENDING, then you should manually
+  // call the CompletionCallback's Run method, or memory will be leaked.
 
   template <typename Method>
   CompletionCallback NewCallback(Method method) {
     PP_DCHECK(object_);
     return NewCallbackHelper(Dispatcher0<Method>(method));
   }
 
+  template <typename Method>
+  CompletionCallback NewOptionalCallback(Method method) {
+    CompletionCallback cc = NewCallback(method);
+    cc.set_flags(PP_COMPLETIONCALLBACK_FLAG_OPTIONAL);
+    return cc;
+  }
+
   // A copy of "a" will be passed to "method" when the completion callback
   // runs.
   //
   // Method should be of type:
   //   void (T::*)(int32_t result, const A& a)
   //
   template <typename Method, typename A>
   CompletionCallback NewCallback(Method method, const A& a) {
     PP_DCHECK(object_);
     return NewCallbackHelper(Dispatcher1<Method, A>(method, a));
   }
 
+  template <typename Method, typename A>
+  CompletionCallback NewOptionalCallback(Method method, const A& a) {
+    CompletionCallback cc = NewCallback(method, a);
+    cc.set_flags(PP_COMPLETIONCALLBACK_FLAG_OPTIONAL);
+    return cc;
+  }
+
   // A copy of "a" and "b" will be passed to "method" when the completion
   // callback runs.
   //
   // Method should be of type:
   //   void (T::*)(int32_t result, const A& a, const B& b)
   //
   template <typename Method, typename A, typename B>
   CompletionCallback NewCallback(Method method, const A& a, const B& b) {
     PP_DCHECK(object_);
     return NewCallbackHelper(Dispatcher2<Method, A, B>(method, a, b));
   }
 
+  template <typename Method, typename A, typename B>
+  CompletionCallback NewOptionalCallback(Method method, const A& a,
+                                         const B& b) {
+    CompletionCallback cc = NewCallback(method, a, b);
+    cc.set_flags(PP_COMPLETIONCALLBACK_FLAG_OPTIONAL);
+    return cc;
+  }
+
  private:
   class BackPointer {
    public:
     typedef CompletionCallbackFactory<T, RefCount> FactoryType;
 
     BackPointer(FactoryType* factory)
         : factory_(factory) {
     }
 
     void AddRef() {
@@ skipping 111 matching lines @@
   CompletionCallbackFactory(const CompletionCallbackFactory&);
   CompletionCallbackFactory& operator=(const CompletionCallbackFactory&);
 
   T* object_;
   BackPointer* back_pointer_;
 };
 
 }  // namespace pp
 
 #endif  // PPAPI_CPP_COMPLETION_CALLBACK_H_