Add CDM allocator interface.

webkit/media/crypto/ppapi/clear_key_cdm.h

 // Copyright (c) 2012 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 WEBKIT_MEDIA_CRYPTO_PPAPI_CLEAR_KEY_CDM_H_
 #define WEBKIT_MEDIA_CRYPTO_PPAPI_CLEAR_KEY_CDM_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/synchronization/lock.h"
 #include "media/base/decryptor_client.h"
 #include "media/crypto/aes_decryptor.h"
 #include "webkit/media/crypto/ppapi/content_decryption_module.h"
 
 namespace media {
 class DecoderBuffer;
 }
 
 namespace webkit_media {
 
+class CdmAllocatorInterface;
+
 // Clear key implementation of the cdm::ContentDecryptionModule interface.
 class ClearKeyCdm : public cdm::ContentDecryptionModule {
  public:
   ClearKeyCdm();
   virtual ~ClearKeyCdm();
 
   // ContentDecryptionModule implementation.
+  virtual bool Initialize(CdmAllocatorInterface* allocator) OVERRIDE;
+
   virtual cdm::Status GenerateKeyRequest(const uint8_t* init_data,
                                          int init_data_size,
                                          cdm::KeyMessage* key_request) OVERRIDE;
 
   virtual cdm::Status AddKey(const char* session_id,
                              int session_id_size,
                              const uint8_t* key,
                              int key_size,
                              const uint8_t* key_id,
                              int key_id_size) OVERRIDE;
@@ skipping 49 matching lines @@
     scoped_array<uint8> key_message_;
     int key_message_length_;
     std::string default_url_;
   };
 
   Client client_;
   media::AesDecryptor decryptor_;
   // Protects the |client_| from being accessed by the |decryptor_|
   // simultaneously.
   base::Lock client_lock_;
+  // Allocates shared memory buffers using the CDM wrapper.

[xhwang, 2012/09/01 13:49:09]

Again, the CDM doesn't care what the allocator implementation is used.

[Tom Finegan, 2012/09/01 21:32:49]

My reasoning for this comment is that allocation of shared memory carries
implied alignment per
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/VzuISrCbq...

[ddorwin, 2012/09/04 09:53:15]

If alignment is the information you want to provide, just say so explicitly in
the CdmAllocator (then DCHECK it everywhere). I would not have known that this
implied alignment.

+  CdmAllocatorInterface* allocator_iface_;
 };
 
 }  // namespace webkit_media
 
 #endif  // WEBKIT_MEDIA_CRYPTO_PPAPI_CLEAR_KEY_CDM_H_