[Webkit-unassigned] [Bug 165073] REGRESSION(r203047): [GTK][Stable] "notify::title" signal unreliably triggered multiple times
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sat Jan 14 20:38:05 PST 2017
https://bugs.webkit.org/show_bug.cgi?id=165073
Michael Catanzaro <mcatanzaro at igalia.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|[GTK][Stable] Regression: |REGRESSION(r203047):
|"notify::title" signal |[GTK][Stable]
|unreliably triggered |"notify::title" signal
|multiple times |unreliably triggered
| |multiple times
--- Comment #12 from Michael Catanzaro <mcatanzaro at igalia.com> ---
Ah wow, that fix was... really hidden quite well. OK, thanks for the help.
It took some squinting for me to see the fix, but it's straightforward. On our 2.14 branch, at the bottom of Document::setTitle, we have this:
// The DOM API has no method of specifying direction, so assume LTR.
updateTitle(StringWithDirection(title, LTR));
if (is<HTMLTitleElement>(m_titleElement.get()))
downcast<HTMLTitleElement>(*m_titleElement).setTextContent(title, ASSERT_NO_EXCEPTION);
else if (is<SVGTitleElement>(m_titleElement.get()))
downcast<SVGTitleElement>(*m_titleElement).setTextContent(title, ASSERT_NO_EXCEPTION);
We just need to move updateTitle from the top into an else clause. It should look like this:
if (is<HTMLTitleElement>(m_titleElement.get()))
downcast<HTMLTitleElement>(*m_titleElement).setTextContent(title, ASSERT_NO_EXCEPTION);
else if (is<SVGTitleElement>(m_titleElement.get()))
downcast<SVGTitleElement>(*m_titleElement).setTextContent(title, ASSERT_NO_EXCEPTION);
else
updateTitle(StringWithDirection(title, LTR));
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170115/0ba32a4a/attachment-0001.html>
More information about the webkit-unassigned
mailing list