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

Some spot error handling improvements to fix a crash in http://checkmate/raptor/js98/test3.html

This commit is contained in:
kipp%netscape.com 1998-11-09 16:21:18 +00:00
Родитель 0cae727734
Коммит 58eb43e339
2 изменённых файлов: 20 добавлений и 14 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

17
content/html/content/src/nsGenericHTMLElement.cpp
Просмотреть файл

@ -641,7 +641,7 @@ nsDOMCSSAttributeDeclaration::StylePropertyChanged(const nsString& aPropertyName
if (nsnull != mContent) {
nsIDocument *doc;
result = mContent->GetDocument(doc);
if (NS_OK == result) {
if (NS_SUCCEEDED(result) && (nsnull != doc)) {
result = doc->AttributeChanged(mContent, nsHTMLAtoms::style, aHint);
NS_RELEASE(doc);
}
@ -2880,11 +2880,13 @@ nsGenericHTMLElement::TriggerLink(nsIPresContext& aPresContext,
PRBool aClick)
{
nsILinkHandler* handler;
if (NS_OK == aPresContext.GetLinkHandler(&handler) && (nsnull != handler)) {
nsresult rv = aPresContext.GetLinkHandler(&handler);
if (NS_SUCCEEDED(rv) && (nsnull != handler)) {
// Resolve url to an absolute url
nsIURL* docURL = nsnull;
nsIDocument* doc;
if (NS_OK == GetDocument(doc)) {
rv = GetDocument(doc);
if (NS_SUCCEEDED(rv) && (nsnull != doc)) {
docURL = doc->GetDocumentURL();
NS_RELEASE(doc);
}
@ -2897,9 +2899,7 @@ nsGenericHTMLElement::TriggerLink(nsIPresContext& aPresContext,
absURLSpec = aURLSpec;
}
if (nsnull != docURL) {
NS_RELEASE(docURL);
}
NS_IF_RELEASE(docURL);
// Now pass on absolute url to the click handler
if (aClick) {
@ -2935,10 +2935,13 @@ nsGenericHTMLLeafElement::CopyInnerTo(nsIHTMLContent* aSrcContent,
nsresult
nsGenericHTMLLeafElement::GetChildNodes(nsIDOMNodeList** aChildNodes)
{
nsDOMSlots *slots = GetDOMSlots();
nsDOMSlots* slots = GetDOMSlots();
if (nsnull == slots->mChildNodes) {
slots->mChildNodes = new nsChildContentList(nsnull);
if (nsnull == slots->mChildNodes) {
return NS_ERROR_OUT_OF_MEMORY;
}
NS_ADDREF(slots->mChildNodes);
}

17
layout/html/content/src/nsGenericHTMLElement.cpp
Просмотреть файл

@ -641,7 +641,7 @@ nsDOMCSSAttributeDeclaration::StylePropertyChanged(const nsString& aPropertyName
if (nsnull != mContent) {
nsIDocument *doc;
result = mContent->GetDocument(doc);
if (NS_OK == result) {
if (NS_SUCCEEDED(result) && (nsnull != doc)) {
result = doc->AttributeChanged(mContent, nsHTMLAtoms::style, aHint);
NS_RELEASE(doc);
}
@ -2880,11 +2880,13 @@ nsGenericHTMLElement::TriggerLink(nsIPresContext& aPresContext,
PRBool aClick)
{
nsILinkHandler* handler;
if (NS_OK == aPresContext.GetLinkHandler(&handler) && (nsnull != handler)) {
nsresult rv = aPresContext.GetLinkHandler(&handler);
if (NS_SUCCEEDED(rv) && (nsnull != handler)) {
// Resolve url to an absolute url
nsIURL* docURL = nsnull;
nsIDocument* doc;
if (NS_OK == GetDocument(doc)) {
rv = GetDocument(doc);
if (NS_SUCCEEDED(rv) && (nsnull != doc)) {
docURL = doc->GetDocumentURL();
NS_RELEASE(doc);
}
@ -2897,9 +2899,7 @@ nsGenericHTMLElement::TriggerLink(nsIPresContext& aPresContext,
absURLSpec = aURLSpec;
}
if (nsnull != docURL) {
NS_RELEASE(docURL);
}
NS_IF_RELEASE(docURL);
// Now pass on absolute url to the click handler
if (aClick) {
@ -2935,10 +2935,13 @@ nsGenericHTMLLeafElement::CopyInnerTo(nsIHTMLContent* aSrcContent,
nsresult
nsGenericHTMLLeafElement::GetChildNodes(nsIDOMNodeList** aChildNodes)
{
nsDOMSlots *slots = GetDOMSlots();
nsDOMSlots* slots = GetDOMSlots();
if (nsnull == slots->mChildNodes) {
slots->mChildNodes = new nsChildContentList(nsnull);
if (nsnull == slots->mChildNodes) {
return NS_ERROR_OUT_OF_MEMORY;
}
NS_ADDREF(slots->mChildNodes);
}