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

Unified Diff: content/renderer/pepper/content_decryptor_delegate.cc

Issue 301933002: Changing default_url to destination_url for consistency (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: removing nit Created 6 years, 7 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 | « content/renderer/pepper/content_decryptor_delegate.h ('k') | media/cdm/aes_decryptor_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/pepper/content_decryptor_delegate.cc
diff --git a/content/renderer/pepper/content_decryptor_delegate.cc b/content/renderer/pepper/content_decryptor_delegate.cc
index 38c0293f24cfefbc84c18b80ba4d194be363cbe7..a1076175b962f18019f26b6a483c068a176304ef 100644
--- a/content/renderer/pepper/content_decryptor_delegate.cc
+++ b/content/renderer/pepper/content_decryptor_delegate.cc
@@ -590,8 +590,7 @@ void ContentDecryptorDelegate::OnSessionCreated(uint32 session_id,
void ContentDecryptorDelegate::OnSessionMessage(uint32 session_id,
PP_Var message_var,
- PP_Var default_url_var) {
- // TODO(amogh.bihani): Replace all the default_url with destination_url.
+ PP_Var destination_url_var) {
if (session_message_cb_.is_null())
return;
@@ -603,18 +602,18 @@ void ContentDecryptorDelegate::OnSessionMessage(uint32 session_id,
message.assign(data, data + message_array_buffer->ByteLength());
}
- StringVar* default_url_string = StringVar::FromPPVar(default_url_var);
+ StringVar* destination_url_string = StringVar::FromPPVar(destination_url_var);
- if (!default_url_string) {
+ if (!destination_url_string) {
OnSessionError(session_id, media::MediaKeys::kUnknownError, 0);
return;
}
- GURL verified_gurl = GURL(default_url_string->value());
+ GURL verified_gurl = GURL(destination_url_string->value());
if (!verified_gurl.is_valid() && !verified_gurl.is_empty()) {
DLOG(WARNING) << "SessionMessage default_url is invalid : "
<< verified_gurl.possibly_invalid_spec();
- verified_gurl = GURL::EmptyGURL(); // Replace invalid default_url.
+ verified_gurl = GURL::EmptyGURL(); // Replace invalid destination_url.
}
session_message_cb_.Run(session_id, message, verified_gurl);
« no previous file with comments | « content/renderer/pepper/content_decryptor_delegate.h ('k') | media/cdm/aes_decryptor_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698