wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 827591 - Be more careful changing declarations on page rules. r=dbaron

Browse Source
This commit is contained in:
Brendan Dahl 2013-01-09 10:33:09 -08:00
parent 7cc38ffb42
commit 0dd6b2e3a8
3 changed files with 25 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
layout/style/crashtests/827591-1.html Normal file
View File

@ -0,0 +1,19 @@
<!DOCTYPE html>
<html>
<head>
<style>
@page {}
</style>
<script>
function boom()
{
document.styleSheets[0].cssRules[0].style.paddingLeft = "initial";
}
</script>
</head>
<body onload="boom();"></body>
</html>

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

@ -84,3 +84,4 @@ load 806310-1.html
load 812824.html
load 822842.html
load 822766-1.html
load 827591-1.html

6
layout/style/nsCSSRules.cpp
View File

@ -2595,7 +2595,11 @@ void
nsCSSPageRule::ChangeDeclaration(css::Declaration* aDeclaration)
{
mImportantRule = nullptr;
mDeclaration = aDeclaration;
// Be careful to not assign to an nsAutoPtr if we would be assigning
// the thing it already holds.
if (aDeclaration != mDeclaration) {
mDeclaration = aDeclaration;
}
nsCSSStyleSheet* sheet = GetStyleSheet();
if (sheet) {