Cleanup template in CdmPromise

media/base/cdm_promise.h

 // 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 std::vector<std::vector<uint8> > KeyIdsVector;
+
 // Interface for promises being resolved/rejected in response to various
 // session actions. These may be called synchronously or asynchronously.
 // The promise must be resolved or rejected exactly once. It is expected that
 // the caller free the promise once it is resolved/rejected.
 //
 // This is only the base class, as parameter to resolve() varies.
 class MEDIA_EXPORT CdmPromise {
  public:
   // A superset of media::MediaKeys::Exception for UMA reporting.
   enum ResultCodeForUMA {
@@ skipping 27 matching lines @@
   // codes are not supported by the Key System. |error_message| is optional.
   virtual void reject(MediaKeys::Exception exception_code,
                       uint32 system_code,
                       const std::string& error_message);
 
   virtual ResolveParameterType GetResolveParameterType() const = 0;
 
  protected:
   CdmPromise();
   CdmPromise(PromiseRejectedCB reject_cb);
+  CdmPromise(const std::string& uma_name);
 
   // If constructed with a |uma_name| (which must be the name of a
   // CdmPromiseResult UMA), CdmPromise will report the promise result (success
   // or rejection code).
   CdmPromise(PromiseRejectedCB reject_cb, const std::string& uma_name);
 
+  // Called by all resolve() methods to do the check on |is_pending_|
+  // and call the UMA success if required.
+  void ResolveBase();

[sandersd (OOO until July 31), 2014/09/16 00:42:54]

As discussed, rename as ReportResolution().

+
+  // Called by all reject() methods to do the check on |is_pending_|
+  // and call the UMA with |exception_code| if required.
+  void RejectBase(MediaKeys::Exception exception_code);
+
   PromiseRejectedCB reject_cb_;
 
   // Keep track of whether the promise hasn't been resolved or rejected yet.
   bool is_pending_;
 
   // UMA to report result to.
   std::string uma_name_;
 
   DISALLOW_COPY_AND_ASSIGN(CdmPromise);
 };
 
-// Specialization for no parameter to resolve().
-template <>
-class MEDIA_EXPORT CdmPromiseTemplate<void> : public CdmPromise {
+// Used when there is no parameter to resolve().
+class MEDIA_EXPORT SimpleCdmPromise : public CdmPromise {
  public:
-  CdmPromiseTemplate(base::Callback<void(void)> resolve_cb,
-                     PromiseRejectedCB rejected_cb);
-  CdmPromiseTemplate(base::Callback<void(void)> resolve_cb,
-                     PromiseRejectedCB rejected_cb,
-                     const std::string& uma_name);
-  virtual ~CdmPromiseTemplate();
+  SimpleCdmPromise(base::Callback<void(void)> resolve_cb,
+                   PromiseRejectedCB rejected_cb);
+  SimpleCdmPromise(base::Callback<void(void)> resolve_cb,
+                   PromiseRejectedCB rejected_cb,
+                   const std::string& uma_name);
+  virtual ~SimpleCdmPromise();
   virtual void resolve();
   virtual ResolveParameterType GetResolveParameterType() const OVERRIDE;
 
  protected:
-  // Allow subclasses to completely override the implementation.
-  CdmPromiseTemplate();
+  SimpleCdmPromise();
+  SimpleCdmPromise(const std::string& uma_name);
 
  private:
   base::Callback<void(void)> resolve_cb_;
 
-  DISALLOW_COPY_AND_ASSIGN(CdmPromiseTemplate);
+  DISALLOW_COPY_AND_ASSIGN(SimpleCdmPromise);
 };
 
-template <>
-class MEDIA_EXPORT CdmPromiseTemplate<std::string> : public CdmPromise {
+class MEDIA_EXPORT NewSessionCdmPromise : public CdmPromise {
  public:
-  CdmPromiseTemplate(base::Callback<void(const std::string&)> resolve_cb,
-                     PromiseRejectedCB rejected_cb);
-  CdmPromiseTemplate(base::Callback<void(const std::string&)> resolve_cb,
-                     PromiseRejectedCB rejected_cb,
-                     const std::string& uma_name);
-  virtual ~CdmPromiseTemplate();
+  NewSessionCdmPromise(base::Callback<void(const std::string&)> resolve_cb,
+                       PromiseRejectedCB rejected_cb);
+  NewSessionCdmPromise(base::Callback<void(const std::string&)> resolve_cb,
+                       PromiseRejectedCB rejected_cb,
+                       const std::string& uma_name);
+  virtual ~NewSessionCdmPromise();
   virtual void resolve(const std::string& result);
   virtual ResolveParameterType GetResolveParameterType() const OVERRIDE;
 
  protected:
-  // Allow subclasses to completely override the implementation.
-  // TODO(jrummell): Remove when derived class SessionLoadedPromise
-  // (in ppapi_decryptor.cc) is no longer needed.
-  CdmPromiseTemplate();
+  NewSessionCdmPromise();
+  NewSessionCdmPromise(const std::string& uma_name);
 
  private:
   base::Callback<void(const std::string&)> resolve_cb_;
 
-  DISALLOW_COPY_AND_ASSIGN(CdmPromiseTemplate);
+  DISALLOW_COPY_AND_ASSIGN(NewSessionCdmPromise);
 };
 
-template <>
-class MEDIA_EXPORT CdmPromiseTemplate<KeyIdsVector> : public CdmPromise {
+class MEDIA_EXPORT KeyIdsPromise : public CdmPromise {
  public:
-  CdmPromiseTemplate(base::Callback<void(const KeyIdsVector&)> resolve_cb,
-                     PromiseRejectedCB rejected_cb);
-  CdmPromiseTemplate(base::Callback<void(const KeyIdsVector&)> resolve_cb,
-                     PromiseRejectedCB rejected_cb,
-                     const std::string& uma_name);
-  virtual ~CdmPromiseTemplate();
+  KeyIdsPromise(base::Callback<void(const KeyIdsVector&)> resolve_cb,
+                PromiseRejectedCB rejected_cb);
+  KeyIdsPromise(base::Callback<void(const KeyIdsVector&)> resolve_cb,
+                PromiseRejectedCB rejected_cb,
+                const std::string& uma_name);
+  virtual ~KeyIdsPromise();
   virtual void resolve(const KeyIdsVector& result);
   virtual ResolveParameterType GetResolveParameterType() const OVERRIDE;
 
+ protected:
+  KeyIdsPromise();
+  KeyIdsPromise(const std::string& uma_name);
+
  private:
   base::Callback<void(const KeyIdsVector&)> resolve_cb_;
 
-  DISALLOW_COPY_AND_ASSIGN(CdmPromiseTemplate);
+  DISALLOW_COPY_AND_ASSIGN(KeyIdsPromise);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_CDM_PROMISE_H_