From a860814a6c4a7206da22422329e48896889cf1f0 Mon Sep 17 00:00:00 2001 From: Mark Finkle Date: Thu, 2 Jan 2014 14:36:03 -0500 Subject: [PATCH] Bug 955870 - Do some quick checks before calling CheckURIVisited r=blassey --- mobile/android/components/build/nsAndroidHistory.cpp | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/mobile/android/components/build/nsAndroidHistory.cpp b/mobile/android/components/build/nsAndroidHistory.cpp index a9eaf2a3954f..a8c11e0f4c3e 100644 --- a/mobile/android/components/build/nsAndroidHistory.cpp +++ b/mobile/android/components/build/nsAndroidHistory.cpp @@ -42,8 +42,16 @@ nsAndroidHistory::RegisterVisitedCallback(nsIURI *aURI, Link *aContent) if (!aContent || !aURI) return NS_OK; + // Silently return if URI is something we would never add to DB. + bool canAdd; + nsresult rv = CanAddURI(aURI, &canAdd); + NS_ENSURE_SUCCESS(rv, rv); + if (!canAdd) { + return NS_OK; + } + nsAutoCString uri; - nsresult rv = aURI->GetSpec(uri); + rv = aURI->GetSpec(uri); if (NS_FAILED(rv)) return rv; NS_ConvertUTF8toUTF16 uriString(uri);