EME v0.1: Report defaultURL in KeyMessage.

webkit/media/webmediaplayer_proxy.cc

Index: webkit/media/webmediaplayer_proxy.cc
diff --git a/webkit/media/webmediaplayer_proxy.cc b/webkit/media/webmediaplayer_proxy.cc
index 13941bf2ad6b5518e57fabf3e200562119889c5e..13efcfa034f7ef6fe4e8d6a33ac11f1278de902b 100644
--- a/webkit/media/webmediaplayer_proxy.cc
+++ b/webkit/media/webmediaplayer_proxy.cc
@@ -166,9 +166,14 @@ void WebMediaPlayerProxy::KeyMessageTask(const std::string& key_system,
                                          int message_length,
                                          const std::string& default_url) {
   DCHECK(render_loop_->BelongsToCurrentThread());
-  if (webmediaplayer_)
+  if (webmediaplayer_) {
+    const GURL default_url_gurl(default_url);
+    DLOG_IF(WARNING, !default_url.empty() && !default_url_gurl.is_valid())
+        << "Invalid URL in default_url: " << default_url.c_str();

[xhwang, 2012/12/08 01:16:44]

do you need c_str() here?

[ddorwin, 2012/12/08 01:20:58]

Yes. Without it, no newline is added and all text following default_str is
ignored.

[scherkus (not reviewing), 2012/12/11 21:07:43]

huh? what text following default_str?

[ddorwin, 2012/12/14 20:52:27]

Removed. Bug in the sample code. I updated cdm.h separately to clarify.

     webmediaplayer_->OnKeyMessage(key_system, session_id,
-                                  message.Pass(), message_length, default_url);
+                                  message.Pass(), message_length,
+                                  default_url_gurl);
+  }
 }
 
 void WebMediaPlayerProxy::NeedKeyTask(const std::string& key_system,