Add Promises for EME (Chromium side)

media/base/cdm_promise.h

Index: media/base/cdm_promise.h
diff --git a/media/base/cdm_promise.h b/media/base/cdm_promise.h
new file mode 100644
index 0000000000000000000000000000000000000000..6569c45f1f1481039ca318fc5f03bc4ba6eb27cb
--- /dev/null
+++ b/media/base/cdm_promise.h
@@ -0,0 +1,73 @@
+// 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.
+
+#ifndef MEDIA_BASE_CDM_PROMISE_H_
+#define MEDIA_BASE_CDM_PROMISE_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/callback.h"
+#include "media/base/media_export.h"
+#include "media/base/media_keys.h"
+
+namespace media {
+
+typedef base::Callback<void(MediaKeys::MediaKeysException exception_code,
+                            uint32 system_code,
+                            const std::string& error_message)>
+    PromiseRejectedCB;
+
+// Interface for promises being resolved/rejected in response to various
+// session actions. These may be called synchronously or asynchronously.
+// Only one method may be called on any promise. It is expected that the
+// caller free the promise once it is resolved/rejected.
+template <typename T>
+class MEDIA_EXPORT CdmPromise {
+ public:
+  CdmPromise(base::Callback<void(const T&)> resolve_cb,
+             PromiseRejectedCB rejected_cb);
+  virtual ~CdmPromise();
+
+  // Used to indicate that promise has successfully resolved.

[ddorwin, 2014/05/13 22:44:02]

Be consistent with reject comment: i.e. operation succeeded.

[jrummell, 2014/05/15 22:38:09]

Done.

+  virtual void resolve(const T& result);
+
+  // Used to indicate that the operation failed. |exception_code| must be
+  // specified. |system_code| is a Key System-specific value for the error
+  // that occurred, or 0 if there is no associated status code or such status
+  // codes are not supported by the Key System. |error_message| is optional.
+  virtual void reject(MediaKeys::MediaKeysException exception_code,
+                      uint32 system_code,
+                      const std::string& error_message);
+
+ private:
+  base::Callback<void(const T&)> resolve_cb_;
+  PromiseRejectedCB rejected_cb_;
+  bool is_pending_;
+
+  DISALLOW_COPY_AND_ASSIGN(CdmPromise);
+};
+
+// Specialization for no parameter to resolve().
+template <>
+class MEDIA_EXPORT CdmPromise<void> {
+ public:
+  CdmPromise(base::Callback<void()> resolve_cb, PromiseRejectedCB rejected_cb);

[ddorwin, 2014/05/13 22:44:02]

Use base::Closure? Or maybe that's more confusing because it doesn't look like
above.

[jrummell, 2014/05/15 22:38:09]

Changed it to be more like the above call.

+  virtual ~CdmPromise();
+  virtual void resolve();
+  virtual void reject(MediaKeys::MediaKeysException exception_code,
+                      uint32 system_code,
+                      const std::string& error_message);
+
+ private:
+  base::Callback<void()> resolve_cb_;
+  PromiseRejectedCB rejected_cb_;
+  bool is_pending_;
+
+  DISALLOW_COPY_AND_ASSIGN(CdmPromise);
+};
+
+}  // namespace media
+
+#endif  // MEDIA_BASE_CDM_PROMISE_H_