Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixing bug 26528 "cloneNode() doesn't copy event handlers".

Browse Source 
r=brendan@mozilla.org sr=jst@netscape.com a=asa@mozilla.org
This patch causes event handlers set as attributes on an element
to be registered as event handlers on the cloned element as well.
This patch also fixes bug 71937 "value of cloned input
type=button is not displayed correctly".


git-svn-id: svn://10.0.0.236/trunk@116855 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
hidday%geocities.com 2002-03-19 13:46:33 +00:00
parent 14eba63cec
commit 2785da72ed
1 changed files with 21 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
mozilla/content/html/content/src/nsGenericHTMLElement.cpp
View File

@ -338,14 +338,28 @@ nsGenericHTMLElement::CopyInnerTo(nsIContent* aSrcContent,
nsGenericHTMLElement* aDst,
PRBool aDeep)
{
nsresult result = NS_OK;
nsresult rv = NS_OK;
if (nsnull != mAttributes) {
result = mAttributes->Clone(&(aDst->mAttributes));
if (mAttributes) {
PRInt32 index, count;
GetAttrCount(count);
nsCOMPtr<nsIAtom> name, prefix;
PRInt32 namespace_id;
nsAutoString value;
if (NS_SUCCEEDED(result)) {
for (index = 0; index < count; index++) {
rv = GetAttrNameAt(index, namespace_id, *getter_AddRefs(name),
*getter_AddRefs(prefix));
NS_ENSURE_SUCCESS(rv, rv);
rv = GetAttr(namespace_id, name, value);
NS_ENSURE_SUCCESS(rv, rv);
rv = aDst->SetAttr(namespace_id, name, value, PR_FALSE);
NS_ENSURE_SUCCESS(rv, rv);
}
if (NS_SUCCEEDED(rv)) {
nsHTMLValue val;
nsresult rv = aDst->GetHTMLAttribute(nsHTMLAtoms::style, val);
rv = aDst->GetHTMLAttribute(nsHTMLAtoms::style, val);
if (rv == NS_CONTENT_ATTR_HAS_VALUE &&
val.GetUnit() == eHTMLUnit_ISupports) {
@ -355,7 +369,7 @@ nsGenericHTMLElement::CopyInnerTo(nsIContent* aSrcContent,
if (rule) {
nsCOMPtr<nsICSSRule> ruleClone;
result = rule->Clone(*getter_AddRefs(ruleClone));
rv = rule->Clone(*getter_AddRefs(ruleClone));
val.SetISupportsValue(ruleClone);
aDst->SetHTMLAttribute(nsHTMLAtoms::style, val, PR_FALSE);
@ -371,7 +385,7 @@ nsGenericHTMLElement::CopyInnerTo(nsIContent* aSrcContent,
aDst->SetContentID(id);
return result;
return rv;
}
nsresult