Chris Waterson Reporter
	Description • 25 years ago

Avoid reflow if SetAttribute() sets attribute to identical value.

	Mike Shaver (:shaver -- probably not reading bugmail closely) Assignee
	Comment 2 • 25 years ago

Ah, dammit.  I wanted to get this in for M13.  Vidur, do you think it's too
risky for the performance gain?

Waterson: you said you depended on this behaviour?  Is that still true?

	Chris Waterson Reporter
	Comment 3 • 25 years ago

Some of the RDF template code depends on this behavior. Since there is currently
no mechanism for "knowing" when a datasource is added/removed from the composite
datasource, this was a dirty hack to force template-generated content to be
rebuilt.

I can't remember off the top of my head what will break if you make this change.
My guess is that, if you did it for HTML elements only, -nothing- would break
(most templates are generated using XUL elements).

I'll file a bug vs. myself to add a "datasource add/remove" listener interface
on the composite datasource, and make the template generator respond
appropriately.

	Chris Waterson Reporter
	Comment 4 • 25 years ago

shaver: ignore bug 24110. just whack away and we'll fix what gets broke.

	Mike Shaver (:shaver -- probably not reading bugmail closely) Assignee
	Comment 5 • 25 years ago

I've got the first part of this fix (the part that actually looks like the old
== new optimization) in my tree, so it'll probably land soon.

(About freaking time, sorry.)

	Mike Shaver (:shaver -- probably not reading bugmail closely) Assignee
	Comment 6 • 25 years ago

Fixed, baby.

	Jan Carpenter
	Comment 7 • 24 years ago

Mass update of qa contact

	Prashant Desale
	Comment 8 • 24 years ago

QA contact Update

	Prashant Desale
	Comment 9 • 23 years ago

Updating QA contact to Shivakiran Tummala.

	Sivakiran Tummala
	Comment 10 • 22 years ago

verified, fix checked in