Closed Bug 1069055 Opened 5 years ago Closed 5 years ago

Intermittent test_delete_focused_element.html | Should not arrive here.

Categories

(Core :: DOM: Device Interfaces, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla36
Tracking Status
firefox34 --- unaffected
firefox35 --- fixed
firefox36 --- fixed
firefox-esr31 --- unaffected

People

(Reporter: RyanVM, Assigned: timdream)

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

https://tbpl.mozilla.org/php/getParsedLog.php?id=48302031&tree=Mozilla-Inbound

b2g_ubuntu32_vm mozilla-inbound opt test mochitest-1 on 2014-09-17 11:37:52 PDT for push e3da4ca374cf
slave: tst-linux32-spot-307

11:52:01     INFO -  2969 INFO TEST-START | /tests/dom/inputmethod/mochitest/test_delete_focused_element.html
11:52:02     INFO -  ###################################### forms.js loaded
11:52:02     INFO -  ############################### browserElementPanning.js loaded
11:52:02     INFO -  ######################## BrowserElementChildPreload.js loaded
11:52:02     INFO -  file_test_app.html was loaded.
11:52:02     INFO -  2970 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_delete_focused_element.html | input was focused.
11:52:02     INFO -  2971 INFO TEST-UNEXPECTED-FAIL | /tests/dom/inputmethod/mochitest/test_delete_focused_element.html | Should not arrive here. - expected PASS
11:52:02     INFO -  2972 INFO TEST-OK | /tests/dom/inputmethod/mochitest/test_delete_focused_element.html | took 388ms
Bug 1030301 landed on inbound not too long before this started failing. Maybe related?
I'll take a look at this tomorrow.
I propose we remove this test and test the focused element removal by adding a few more steps in test_two_inputs.html. Does that sounds good?

Alternatively I can copy-paste test_two_inputs.html and replace blur() calls with removeChild calls.
Assignee: nobody → timdream
Status: NEW → ASSIGNED
Flags: needinfo?(xyuan)
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) (please ni?) from comment #25)
> I propose we remove this test and test the focused element removal by adding
> a few more steps in test_two_inputs.html. Does that sounds good?
Yes, please ;-)
Flags: needinfo?(xyuan)
Attachment #8513999 - Flags: review?(xyuan) → review+
Attached patch Patch for commitSplinter Review
Attachment #8513999 - Attachment is obsolete: true
Attachment #8514850 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/2aff0325a61a
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.