corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1362761 : Remove Compact from NIGHTLY build for debug purpose. r=francois

Browse Source 
We suspect nsTArray Compact may cause a crash issue during SafeBrowsing
update. Temporarily remove it from NIGHTLY build, if crash still shows
at NIGHTLY build, we will restore it back.

MozReview-Commit-ID: 2wjbMykEbJ8
This commit is contained in:
DimiL
2017-10-11 16:56:06 +08:00
parent e25266583a
commit 811c853e4f
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
toolkit/components/url-classifier/nsUrlClassifierPrefixSet.cpp
View File

@@ -117,7 +117,10 @@ nsUrlClassifierPrefixSet::MakePrefixSet(const uint32_t* aPrefixes, uint32_t aLen
// Compact the previous element.
// Note there is always at least one element when we get here,
// because we created the first element before the loop.
// Bug 1362761 : Remove Compact from NIGHTLY build for debug purpose
#ifndef NIGHTLY_BUILD
mIndexDeltas.LastElement().Compact();
#endif
if (!mIndexDeltas.AppendElement(fallible)) {
return NS_ERROR_OUT_OF_MEMORY;
}
@@ -139,9 +142,12 @@ nsUrlClassifierPrefixSet::MakePrefixSet(const uint32_t* aPrefixes, uint32_t aLen
previousItem = aPrefixes[i];
}
// Bug 1362761 : Remove Compact from NIGHTLY build for debug purpose
#ifndef NIGHTLY_BUILD
mIndexDeltas.LastElement().Compact();
mIndexDeltas.Compact();
mIndexPrefixes.Compact();
#endif
LOG(("Total number of indices: %d", aLength));
LOG(("Total number of deltas: %d", totalDeltas));