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

Unified Diff: chromecast/media/cdm/browser_cdm_cast.cc

Issue 1100763002: Inject CanAddURLToHistory into TopSitesImpl (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@prefs
Patch Set: Fix error introduced during rebase 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
« no previous file with comments | « chromecast/media/cdm/browser_cdm_cast.h ('k') | chromeos/CHROMEOS_LKGM » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromecast/media/cdm/browser_cdm_cast.cc
diff --git a/chromecast/media/cdm/browser_cdm_cast.cc b/chromecast/media/cdm/browser_cdm_cast.cc
index 33c10c5d90af02991a96a72fe62fd7d71c414dac..07c3c3e5500e9d5e94d08ffc2ced5aa611543336 100644
--- a/chromecast/media/cdm/browser_cdm_cast.cc
+++ b/chromecast/media/cdm/browser_cdm_cast.cc
@@ -68,7 +68,7 @@ void BrowserCdmCast::LoadSession(
}
void BrowserCdmCast::OnSessionMessage(const std::string& session_id,
- const std::vector<uint8>& message,
+ const std::vector<uint8_t>& message,
const GURL& destination_url) {
// Note: Message type is not supported in Chromecast. Do our best guess here.
::media::MediaKeys::MessageType message_type =
@@ -99,31 +99,6 @@ void BrowserCdmCast::OnSessionKeysChange(
player_tracker_impl_->NotifyNewKey();
}
-void BrowserCdmCast::SetServerCertificateHelper(
- const std::vector<uint8>& certificate_data,
- scoped_ptr<::media::SimpleCdmPromise> promise) {
- SetServerCertificate(certificate_data.data(),
- certificate_data.size(),
- promise.Pass());
-}
-
-void BrowserCdmCast::CreateSessionAndGenerateRequestHelper(
- ::media::MediaKeys::SessionType session_type,
- ::media::EmeInitDataType init_data_type,
- const std::vector<uint8>& init_data,
- scoped_ptr<::media::NewSessionCdmPromise> promise) {
- CreateSessionAndGenerateRequest(
- session_type, init_data_type, init_data.data(), init_data.size(),
- promise.Pass());
-}
-
-void BrowserCdmCast::UpdateSessionHelper(
- const std::string& session_id,
- const std::vector<uint8>& response,
- scoped_ptr<::media::SimpleCdmPromise> promise) {
- UpdateSession(session_id, response.data(), response.size(), promise.Pass());
-}
-
// A macro runs current member function on |cdm_loop_| thread.
#define FORWARD_ON_CDM_THREAD(param_fn, ...) \
cdm_loop_->PostTask( \
@@ -160,29 +135,26 @@ BrowserCdmCast* BrowserCdmCastUi::browser_cdm_cast() const {
}
void BrowserCdmCastUi::SetServerCertificate(
- const uint8* certificate_data,
- int certificate_data_length,
+ const std::vector<uint8_t>& certificate,
scoped_ptr<::media::SimpleCdmPromise> promise) {
DCHECK(thread_checker_.CalledOnValidThread());
FORWARD_ON_CDM_THREAD(
- SetServerCertificateHelper,
- std::vector<uint8>(certificate_data,
- certificate_data + certificate_data_length),
+ SetServerCertificate,
+ certificate,
base::Passed(&promise));
}
void BrowserCdmCastUi::CreateSessionAndGenerateRequest(
::media::MediaKeys::SessionType session_type,
::media::EmeInitDataType init_data_type,
- const uint8* init_data,
- int init_data_length,
+ const std::vector<uint8_t>& init_data,
scoped_ptr<::media::NewSessionCdmPromise> promise) {
DCHECK(thread_checker_.CalledOnValidThread());
FORWARD_ON_CDM_THREAD(
- CreateSessionAndGenerateRequestHelper,
+ CreateSessionAndGenerateRequest,
session_type,
init_data_type,
- std::vector<uint8>(init_data, init_data + init_data_length),
+ init_data,
base::Passed(&promise));
}
@@ -197,14 +169,13 @@ void BrowserCdmCastUi::LoadSession(
void BrowserCdmCastUi::UpdateSession(
const std::string& session_id,
- const uint8* response,
- int response_length,
+ const std::vector<uint8_t>& response,
scoped_ptr<::media::SimpleCdmPromise> promise) {
DCHECK(thread_checker_.CalledOnValidThread());
FORWARD_ON_CDM_THREAD(
- UpdateSessionHelper,
+ UpdateSession,
session_id,
- std::vector<uint8>(response, response + response_length),
+ response,
base::Passed(&promise));
}
« no previous file with comments | « chromecast/media/cdm/browser_cdm_cast.h ('k') | chromeos/CHROMEOS_LKGM » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698