Using GURL in place of std::string for destination_url

content/renderer/pepper/content_decryptor_delegate.cc

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 a36cf6f353965d041c421e3f538bd81c708c6b0b..278ed657baf8cf40dbe3f5b114a690f965a84d84 100644
--- a/content/renderer/pepper/content_decryptor_delegate.cc
+++ b/content/renderer/pepper/content_decryptor_delegate.cc
@@ -609,7 +609,15 @@ void ContentDecryptorDelegate::OnSessionMessage(uint32 session_id,
     return;
   }
 
-  session_message_cb_.Run(session_id, message, default_url_string->value());
+  GURL verified_gurl = GURL(default_url_string->value());

[xhwang, 2014/05/22 18:44:52]

I am surprised to see that we still have "default URL" used in various places.
We should replace all of them with "destination URL". Could you please help put
a TODO here?

+  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.
+  }
+
+  session_message_cb_.Run(
+      session_id, message, verified_gurl);
 }
 
 void ContentDecryptorDelegate::OnSessionReady(uint32 session_id) {