Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1548)

Unified Diff: media/blink/cdm_session_adapter.h

Issue 1072403009: Use std::vector<uint8_t> instead of uint8*/int for MediaKeys interface (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/blink/cdm_session_adapter.h
diff --git a/media/blink/cdm_session_adapter.h b/media/blink/cdm_session_adapter.h
index 5ed7550e2b19c23dcdeda7d402b04dd0c4784bf9..1156abf2dcd4adbf7f679a94f78a1153cfccac34 100644
--- a/media/blink/cdm_session_adapter.h
+++ b/media/blink/cdm_session_adapter.h
@@ -7,6 +7,7 @@
#include <map>
#include <string>
+#include <vector>
#include "base/basictypes.h"
#include "base/containers/hash_tables.h"
@@ -42,8 +43,7 @@ class CdmSessionAdapter : public base::RefCounted<CdmSessionAdapter> {
// Provides a server certificate to be used to encrypt messages to the
// license server.
- void SetServerCertificate(const uint8* server_certificate,
- int server_certificate_length,
+ void SetServerCertificate(const std::vector<uint8>& server_certificate,
xhwang 2015/04/21 04:47:11 we use |ceritificate| in MediaKeys... would be gre
jrummell 2015/04/21 22:59:36 Done.
scoped_ptr<SimpleCdmPromise> promise);
// Creates a new session and adds it to the internal map. The caller owns the
@@ -64,8 +64,7 @@ class CdmSessionAdapter : public base::RefCounted<CdmSessionAdapter> {
// Initializes a session with the |init_data_type|, |init_data| and
// |session_type| provided.
void InitializeNewSession(EmeInitDataType init_data_type,
- const uint8* init_data,
- int init_data_length,
+ const std::vector<uint8>& init_data,
MediaKeys::SessionType session_type,
scoped_ptr<NewSessionCdmPromise> promise);
@@ -76,8 +75,7 @@ class CdmSessionAdapter : public base::RefCounted<CdmSessionAdapter> {
// Updates the session specified by |session_id| with |response|.
void UpdateSession(const std::string& session_id,
- const uint8* response,
- int response_length,
+ const std::vector<uint8>& response,
scoped_ptr<SimpleCdmPromise> promise);
// Closes the session specified by |session_id|.

Powered by Google App Engine
This is Rietveld 408576698