Bug 816784 part 4 - Fix the rooting hazards that the analysis report.…

… r=sfink

WrapperMap::Enum::goToNext() is listed as a GC function because the *generated*
stack calls BrowserCompartmentMatcher::match() which leads a path to a FieldCall
of nsISupports.QueryInterface, however it is never the case. Add
nsContentUtils::IsExpandedPrincipal() to the ignore functions because it is more
narrowly scoped and therefore less likely to hide problems.



MozReview-Commit-ID: ACwkMtRiQk2

--HG--
extra : rebase_source : 2cadcc9f59096b4e5f693f39e2cab93da048f949

js/src/devtools/rootAnalysis/annotations.js

@@ -231,6 +231,8 @@ var ignoreFunctions = {
     // The big hammers.
     "PR_GetCurrentThread" : true,
     "calloc" : true,
+
+    "uint8 nsContentUtils::IsExpandedPrincipal(nsIPrincipal*)" : true,
 };
 
 function extraGCFunctions() {