Closed Bug 665677 Opened 10 years ago Closed 9 years ago

Intermittent test_bug330705-2.xul | The first box element is still focused after blur() has been called on the second box element

Categories

(Core :: DOM: Events, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla15
Tracking Status
firefox-esr10 --- fixed

People

(Reporter: philor, Assigned: enndeakin)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla-Inbound/1308597920.1308599553.32513.gz
Rev3 Fedora 12 mozilla-inbound opt test mochitest-other on 2011/06/20 12:25:20
s: talos-r3-fed-059

1537 INFO TEST-START | chrome://mochitests/content/chrome/content/xul/content/test/test_bug330705-2.xul
1538 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/content/xul/content/test/test_bug330705-2.xul | The first box element is still focused after blur() has been called on the second box element
1539 INFO TEST-END | chrome://mochitests/content/chrome/content/xul/content/test/test_bug330705-2.xul | finished in 26ms
Attached patch Patch (obsolete) — Splinter Review
Looks like the cause is focus/blur events are blocked by script blocker maybe due to loading or reflowing the page. Maybe, this fixes the random orange.

https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=ec4095b36699
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Attachment #579612 - Flags: review?(bugs)
Attachment #579612 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/53cc21dc210f
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
I thought it was bad practice to add timers. They should only be added when they are really needed, no?
(In reply to Martijn Wargers [:mw22] (QA - IRC nick: mw22) from comment #207)
> I thought it was bad practice to add timers. They should only be added when
> they are really needed, no?

I think that they are necessary.

The first timer in focus handler prevents nesting DOM events.

The second timer in doTest() checks whether there are no delayed blur events.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #217)
> Ah, I see. I'll post a patch.

Er, no I misunderstood the failure. I have no idea for now.
Assignee: masayuki → nobody
Whiteboard: [orange][inbound] → [orange]