Bug 1211046 - Remove url parsing warning message for youtube embed check; r=cpeterson, a=lizzard
authorKyle Machulis <kyle@nonpolynomial.com>
Mon, 05 Oct 2015 10:54:44 -0700
changeset 296287 d9d25e8b20fa53c219c50c3d3f66a91527d664ab
parent 296286 216a745230ddfb20fb292a2c212b5c1653657b12
child 296288 a80ec0b5f87562c45f88d59a0f45bf26319e39d3
push id5245
push userraliiev@mozilla.com
push dateThu, 29 Oct 2015 11:30:51 +0000
treeherdermozilla-beta@dac831dc1bd0 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerscpeterson, lizzard
bugs1211046
milestone43.0a2
Bug 1211046 - Remove url parsing warning message for youtube embed check; r=cpeterson, a=lizzard
dom/base/nsObjectLoadingContent.cpp
--- a/dom/base/nsObjectLoadingContent.cpp
+++ b/dom/base/nsObjectLoadingContent.cpp
@@ -1489,17 +1489,17 @@ nsObjectLoadingContent::IsYoutubeEmbed()
   // If we can't analyze the URL, just pass on through.
   if(!tldService) {
     NS_WARNING("Could not get TLD service!");
     return false;
   }
   nsAutoCString currentBaseDomain;
   bool ok = NS_SUCCEEDED(tldService->GetBaseDomain(mURI, 0, currentBaseDomain));
   if (!ok) {
-    NS_WARNING("Could not parse plugin domain!");
+    // Data URIs won't parse correctly, so just fail silently here.
     return false;
   }
   nsAutoCString domain("youtube.com");
   if (StringEndsWith(domain, currentBaseDomain)) {
     return true;
   }
   return false;
 }