mozilla
/
pjs
зеркало из https://github.com/mozilla/pjs.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Fix EOF handling after an !important. Bug 390318, r+sr+a=dbaron

This commit is contained in:
bzbarsky@mit.edu 2007-08-02 14:20:11 -07:00
Родитель 1c890eeda8
Коммит 718c18f740
9 изменённых файлов: 73 добавлений и 28 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

9
layout/reftests/bugs/390318-1-ref.html Normal file
Просмотреть файл

@ -0,0 +1,9 @@
<!DOCTYPE html>
<html>
<head>
<style>body { color: green; }</style>
</head>
<body>
This should be green
</body>
</html>

9
layout/reftests/bugs/390318-1a.html Normal file
Просмотреть файл

@ -0,0 +1,9 @@
<!DOCTYPE html>
<html>
<head>
<style>body { color: red; } body { color: green</style>
</head>
<body>
This should be green
</body>
</html>

9
layout/reftests/bugs/390318-1b.html Normal file
Просмотреть файл

@ -0,0 +1,9 @@
<!DOCTYPE html>
<html>
<head>
<style>body { color: red; } body { color: green !important</style>
</head>
<body>
This should be green
</body>
</html>

9
layout/reftests/bugs/390318-1c.html Normal file
Просмотреть файл

@ -0,0 +1,9 @@
<!DOCTYPE html>
<html>
<head>
<style>body { color: red; } body { color: green ;</style>
</head>
<body>
This should be green
</body>
</html>

9
layout/reftests/bugs/390318-1d.html Normal file
Просмотреть файл

@ -0,0 +1,9 @@
<!DOCTYPE html>
<html>
<head>
<style>body { color: red; } body { color: green !important ;</style>
</head>
<body>
This should be green
</body>
</html>

9
layout/reftests/bugs/390318-1e.html Normal file
Просмотреть файл

@ -0,0 +1,9 @@
<!DOCTYPE html>
<html>
<head>
<style>body { color: green; } body { color: red a</style>
</head>
<body>
This should be green
</body>
</html>

9
layout/reftests/bugs/390318-1f.html Normal file
Просмотреть файл

@ -0,0 +1,9 @@
<!DOCTYPE html>
<html>
<head>
<style>body { color: green; } body { color: red !important a</style>
</head>
<body>
This should be green
</body>
</html>

2
layout/reftests/bugs/reftest.list
Просмотреть файл

@ -327,3 +327,5 @@ random-if(MOZ_WIDGET_TOOLKIT=="cocoa") == 379316-2.html 379316-2-ref.html # bug
== 386470-1c.html 386470-1-ref.html
== 386920-1.html 386920-1-ref.html
== 387344-1.html 387344-1-ref.html
== 390318-1a.html 390318-1-ref.html
== 390318-1b.html 390318-1-ref.html

36
layout/style/nsCSSParser.cpp
Просмотреть файл

@ -3331,9 +3331,8 @@ CSSParserImpl::ParseDeclaration(nsresult& aErrorCode,
aMustCallValueAppended, aChanged);
return PR_TRUE;
}
else {
if (eCSSToken_Symbol == tk->mType) {
if ('!' == tk->mSymbol) {
if (eCSSToken_Symbol == tk->mType && '!' == tk->mSymbol) {
// Look for important ident
if (!GetToken(aErrorCode, PR_TRUE)) {
// Premature eof is not ok
@ -3355,24 +3354,12 @@ CSSParserImpl::ParseDeclaration(nsresult& aErrorCode,
// Not a !important declaration
UngetToken();
}
}
else {
// Not a !important declaration
UngetToken();
}
}
// Make sure valid property declaration is terminated with either a
// semicolon or a right-curly-brace (when aCheckForBraces is true).
// When aCheckForBraces is false, proper termination is either
// semicolon or EOF.
// semicolon, EOF or a right-curly-brace (this last only when
// aCheckForBraces is true).
if (!GetToken(aErrorCode, PR_TRUE)) {
if (aCheckForBraces) {
// Premature eof is not ok
REPORT_UNEXPECTED_EOF(PEDeclEndEOF);
ClearTempData(propID);
return PR_FALSE;
}
// EOF is a perfectly good way to end a declaration and declaration block
TransferTempData(aDeclaration, propID, isImportant,
aMustCallValueAppended, aChanged);
return PR_TRUE;
@ -3383,16 +3370,9 @@ CSSParserImpl::ParseDeclaration(nsresult& aErrorCode,
aMustCallValueAppended, aChanged);
return PR_TRUE;
}
if (!aCheckForBraces) {
// If we didn't hit eof and we didn't see a semicolon then the
// declaration is not properly terminated.
REPORT_UNEXPECTED_TOKEN(PEBadDeclEnd);
REPORT_UNEXPECTED(PEDeclDropped);
OUTPUT_ERROR();
ClearTempData(propID);
return PR_FALSE;
}
if ('}' == tk->mSymbol) {
if (aCheckForBraces && '}' == tk->mSymbol) {
// Unget the '}' so we'll be able to tell that this is the end
// of the declaration block when we unwind from here.
UngetToken();
TransferTempData(aDeclaration, propID, isImportant,
aMustCallValueAppended, aChanged);