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

Bug 497242. GetElementsByName should not find non-HTML elements. r+sr=sicking

This commit is contained in:
Boris Zbarsky 2009-06-19 16:57:39 -04:00
Родитель 56b8bba799
Коммит 56540642ff
3 изменённых файлов: 47 добавлений и 2 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

6
content/html/document/src/nsHTMLDocument.cpp
Просмотреть файл

@ -2267,8 +2267,10 @@ nsHTMLDocument::MatchNameAttribute(nsIContent* aContent, PRInt32 aNamespaceID,
{
NS_PRECONDITION(aContent, "Must have content node to work with!");
nsString* elementName = static_cast<nsString*>(aData);
return aContent->AttrValueIs(kNameSpaceID_None, nsGkAtoms::name,
*elementName, eCaseMatters);
return
aContent->GetNameSpaceID() == kNameSpaceID_XHTML &&
aContent->AttrValueIs(kNameSpaceID_None, nsGkAtoms::name,
*elementName, eCaseMatters);
}
NS_IMETHODIMP

1
content/html/document/test/Makefile.in
Просмотреть файл

@ -94,6 +94,7 @@ _TEST_FILES = test_bug1682.html \
test_bug481647.html \
test_bug482659.html \
test_bug486741.html \
test_bug497242.xhtml \
$(NULL)
libs:: $(_TEST_FILES)

42
content/html/document/test/test_bug497242.xhtml Normal file
Просмотреть файл

@ -0,0 +1,42 @@
<html xmlns="http://www.w3.org/1999/xhtml">
<!--
https://bugzilla.mozilla.org/show_bug.cgi?id=497242
-->
<head>
<title>Test for Bug 497242</title>
<script type="application/javascript" src="/MochiKit/packed.js"></script>
<script type="application/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
<link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/>
</head>
<body>
<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=497242">Mozilla Bug 497242</a>
<p id="display"></p>
<div id="content" style="display: none">
<form name="foo"/>
<form name="foo"/>
<form name="bar"/>
<form name="bar" xmlns=""/>
</div>
<pre id="test">
<script type="application/javascript">
<![CDATA[
/** Test for Bug 497242 **/
is(document.getElementsByName("foo").length, 2,
"Should find both forms with name 'foo'");
is(document.getElementsByName("foo")[0],
document.getElementsByTagName("form")[0],
"Unexpected first foo");
is(document.getElementsByName("foo")[1],
document.getElementsByTagName("form")[1],
"Unexpected second foo");
is(document.getElementsByName("bar").length, 1,
"Should find only the HTML form with name 'bar'");
is(document.getElementsByName("bar")[0],
document.getElementsByTagName("form")[2],
"Unexpected bar");
]]>
</script>
</pre>
</body>
</html>