Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-17 23:35:34 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1402476. ExpectedOwnerForChild should not return a first-letter frame for a text child. r=emilio

Browse Source 
The expected owner is the DOM parent.  The first-letter then does some reparenting of the text style later.
This commit is contained in:
Boris Zbarsky 2017-10-02 16:36:47 -04:00
parent 18b234fd09
commit 4a228305f6
3 changed files with 20 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
layout/base/ServoRestyleManager.cpp
View File

@ -84,6 +84,12 @@ ExpectedOwnerForChild(const nsIFrame& aFrame)
return parent->IsLineFrame() ? parent->GetParent() : parent;
}
if (parent->IsLetterFrame()) {
// Things never have ::first-letter as their expected parent. Go
// on up to the ::first-letter's parent.
parent = parent->GetParent();
}
parent = FirstContinuationOrPartOfIBSplit(parent);
// We've handled already anon boxes and bullet frames, so now we're looking at

13
layout/base/crashtests/1402476.html Normal file
View File

@ -0,0 +1,13 @@
<html class="reftest-wait">
<script>
a = document.createElement("style")
a.appendChild(document.createTextNode("*:first-letter { }"))
document.documentElement.appendChild(a)
a.style.display = "contents"
requestIdleCallback(() => {
a.appendChild(document.createElement("x"));
a.lastChild.offsetWidth;
document.documentElement.className = "";
})
</script>
</html>

1
layout/base/crashtests/crashtests.list
View File

@ -505,3 +505,4 @@ load 1400438-1.html
load 1400599-1.html
load 1401739.html
load 1401840.html
load 1402476.html