1808836 - C-C TB many warning lines: WARNING: 'event->mOriginalTarget != this', file /NEW-SSD/NREF-COMM-CENTRAL/mozilla/dom/xul/XULButtonElement.cpp:349

Status: RESOLVED FIXED

(Core :: XUL, enhancement)

Description

[ISHIKAWA, Chiaki]

I create a local binary of C-C TB DEBUG version for local testing.

I started to get many lines of the same warning when I run the TB binary.

[Parent 348217, Main Thread] WARNING: 'event->mOriginalTarget != this', file /NEW-SSD/NREF-COMM-CENTRAL/mozilla/dom/xul/XULButtonElement.cpp:349

I get 475 such warning lines for simply starting TB and receiving about 170 e-mail messages. That is a tad too excessive.

That comes from the following code: I see it was modified on Jan 4. So that is why I see them out of blue.
https://searchfox.org/mozilla-central/source/dom/xul/XULButtonElement.cpp#349

Now, TB may not be playing nicely with modified XUL. I have no idea.

Comment 1

[David Shin[:dshin]]

This happens on firefox debug build - enable menubar, click on any of them and hover through different menu items. Every time the focus moves, a warning is generated. Menubar seems to behave as expected otherwise?

Comment 2

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1808836 - Remove noisy warning.

This was useful when debugging the menu tests and so on, but the
condition is not unexpected nor rare and thus shouldn't warn.

(No review, trivial debug-only)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8f648891ec44
Remove noisy warning.

Comment 4

[ISHIKAWA, Chiaki]

(In reply to Emilio Cobos Álvarez (:emilio) from comment #2)

Created attachment 9312104 [details]
Bug 1808836 - Remove noisy warning.

This was useful when debugging the menu tests and so on, but the
condition is not unexpected nor rare and thus shouldn't warn.

(No review, trivial debug-only)

Thank you for the quick fix!

Comment 5

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/8f648891ec44