Bug 1127534 - Remove assertion before creating a channel (r=sicking)

mozilla · Aug 2, 2015 · ccd1961 · ccd1961
1 parent 5d6e8c7
commit ccd1961
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/image/imgLoader.cpp b/image/imgLoader.cpp
@@ -789,7 +789,9 @@ NewImageChannel(nsIChannel** aResult,
     // we should always have a requestingNode, or we are loading something
     // outside a document, in which case the triggeringPrincipal
     // should always be the systemPrincipal.
-    MOZ_ASSERT(nsContentUtils::IsSystemPrincipal(triggeringPrincipal));
+    // However, there are two exceptions: one is Notifications and the
+    // other one is Favicons which create a channel in the parent prcoess
+    // in which case we can't get a requestingNode.
     rv = NS_NewChannel(aResult,
                        aURI,
                        triggeringPrincipal,