From d9b9142c0c0b8eef96dacc7e72f7cb9067d0beec Mon Sep 17 00:00:00 2001
From: Daniel Holbert <dholbert@cs.stanford.edu>
Date: Fri, 4 Dec 2015 12:01:43 -0800
Subject: [PATCH] Bug 1230351: Replace ThirdPartyUtil::IsThirdPartyInternal's
 warning-spammy NS_ENSURE_ARG with an equivalent check that lacks the warning.
 r=mrbkap

---
 dom/base/ThirdPartyUtil.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/dom/base/ThirdPartyUtil.cpp b/dom/base/ThirdPartyUtil.cpp
index 47cb4067929..6e8c819e85b 100644
--- a/dom/base/ThirdPartyUtil.cpp
+++ b/dom/base/ThirdPartyUtil.cpp
@@ -46,7 +46,9 @@ ThirdPartyUtil::IsThirdPartyInternal(const nsCString& aFirstDomain,
                                      nsIURI* aSecondURI,
                                      bool* aResult)
 {
-  NS_ENSURE_ARG(aSecondURI);
+  if (!aSecondURI) {
+    return NS_ERROR_INVALID_ARG;
+  }
 
   // Get the base domain for aSecondURI.
   nsCString secondDomain;