Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-24 21:31:04 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1322570 Part 8 - Run debug code only if the style source is a gecko rule node. r=bholley

Browse Source 
Per bug 1322570 comment 27, the debug code is only useful in Gecko-backed
style system. To prevent the assertion in nsStyleContext::GetParent() when
running stylo, run them only in gecko.

MozReview-Commit-ID: DOcJ3T7vS1J

--HG--
extra : rebase_source : 33fd6caaa4ada9530fe8d874b827d91ac355e0c1
This commit is contained in:
Ting-Yu Lin 2017-03-20 17:05:16 +08:00
parent fd01a09628
commit 9e0bdb7f1e
1 changed files with 11 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
layout/tables/nsTableColGroupFrame.cpp
View File

@ -296,15 +296,18 @@ nsTableColGroupFrame::RemoveFrame(ChildListID aListID,
#ifdef DEBUG
nsIFrame* providerFrame;
nsStyleContext* psc = colFrame->GetParentStyleContext(&providerFrame);
if (colFrame->StyleContext()->GetParent() == psc) {
NS_ASSERTION(col->StyleContext() == colFrame->StyleContext() &&
col->GetContent() == colFrame->GetContent(),
"How did that happen??");
if (psc->StyleSource().IsGeckoRuleNodeOrNull()) {
// This check code is useful only in Gecko-backed style system.
if (colFrame->StyleContext()->GetParent() == psc) {
NS_ASSERTION(col->StyleContext() == colFrame->StyleContext() &&
col->GetContent() == colFrame->GetContent(),
"How did that happen??");
}
// else colFrame is being removed because of a frame
// reconstruct on it, and its style context is still the old
// one, so we can't assert anything about how it compares to
// col's style context.
}
// else colFrame is being removed because of a frame
// reconstruct on it, and its style context is still the old
// one, so we can't assert anything about how it compares to
// col's style context.
#endif
nextCol = col->GetNextCol();
RemoveFrame(kPrincipalList, col);