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
--- a/js/src/devtools/rootAnalysis/annotations.js
+++ b/js/src/devtools/rootAnalysis/annotations.js
@@ -226,16 +226,18 @@ var ignoreFunctions = {
// VTune internals that lazy-load a shared library and make IndirectCalls.
"iJIT_IsProfilingActive" : true,
"iJIT_NotifyEvent": true,
// The big hammers.
"PR_GetCurrentThread" : true,
"calloc" : true,
+
+ "uint8 nsContentUtils::IsExpandedPrincipal(nsIPrincipal*)" : true,
};
function extraGCFunctions() {
return ["ffi_call"];
}
function isProtobuf(name)
{