Intermittent widget/tests/test_assign_event_data.html | WidgetKeyboardEvent (keyup during composition) (gCallPreventDefault=false): failed to get duplicated event

NEW
Unassigned

Status

()

defect
P5
normal
3 years ago
2 years ago

People

(Reporter: intermittent-bug-filer, Unassigned)

Tracking

({intermittent-failure, leave-open})

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [stockwell disabled]tpi:-)

Attachments

(1 attachment)

Whiteboard: tpi:-
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
this seems to be android specific.  It also seems that we have sort of a rollercoaster effect here with a spike for a few weeks then some more up and downs.

I see a similar bugs, all appear to be the same for android debug:
Bug 1285414: Intermittent widget/tests/test_assign_event_data.html | WidgetKeyboardEvent (keyup during composition) (gCallPreventDefault=false): failed to get duplicated event
Bug 1308173: Intermittent widget/tests/test_assign_event_data.html | WidgetKeyboardEvent (keyup during composition) (gCallPreventDefault=true): failed to get duplicated event
See Also: → 1308173
Whiteboard: tpi:- → [stockwell needswork]tpi:-
:gbrown, can you look into this, possibly I think we just need to determine if we get value on android, or maybe there is something more useful we can harvest out of the logs.
Flags: needinfo?(gbrown)
Assignee: nobody → gbrown
Flags: needinfo?(gbrown)
Associated logcats have warnings just before the failures:

https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=78844250&lineNumber=1854
https://public-artifacts.taskcluster.net/ZO5wFMLjRo6L0oCQTn4hBQ/0/public/test_info//logcat-emulator-5554.log

02-20 08:36:02.714   770   788 I GeckoDump: ⰲ겿{"action":"test_status","time":1487608562712,"thread":null,"pid":null,"source":"mochitest","test":"/tests/widget/tests/test_assign_event_data.html","subtest":"InternalScrollAreaEvent (MozScrolledAreaChanged, shrinking) (gCallPreventDefault=false): mismatch (explicitOriginalTarget)","status":"PASS","js_source":"TestRunner.js"}ⰲ겿
02-20 08:36:03.674   770   788 I Gecko   : [770] WARNING: 'mIsInEditAction', file /home/worker/workspace/build/src/editor/libeditor/EditorBase.cpp, line 1889
02-20 08:36:04.175   770   788 I Gecko   : [770] WARNING: 'mIsInEditAction', file /home/worker/workspace/build/src/editor/libeditor/EditorBase.cpp, line 1889
02-20 08:36:04.804   770   788 I GeckoDump: ⰲ겿{"action":"test_status","time":1487608564776,"thread":null,"pid":null,"source":"mochitest","test":"/tests/widget/tests/test_assign_event_data.html","subtest":"WidgetKeyboardEvent (keyup during composition) (gCallPreventDefault=false): failed to get duplicated event","status":"FAIL","expected":"PASS","stack":"    gCallback@widget/tests/test_assign_event_data.html:667:5\n","js_source":"TestRunner.js"}ⰲ겿
02-20 08:36:04.894   770   788 I GeckoDump: ⰲ겿{"action":"test_status","time":1487608564849,"thread":null,"pid":null,"source":"mochitest","test":"/tests/widget/tests/test_assign_event_data.html","subtest":"WidgetKeyboardEvent (keyup during composition) (gCallPreventDefault=false): count of attribute of the event must be larger than 0","status":"FAIL","expected":"PASS","stack":"    gCallback@widget/tests/test_assign_event_data.html:668:5\n","js_source":"TestRunner.js"}ⰲ겿
02-20 08:36:05.165   770   788 I Gecko   : [770] WARNING: 'mIsInEditAction', file /home/worker/workspace/build/src/editor/libeditor/EditorBase.cpp, line 1889
02-20 08:36:05.474   770   788 I Gecko   : [770] WARNING: 'mIsInEditAction', file /home/worker/workspace/build/src/editor/libeditor/EditorBase.cpp, line 1889
02-20 08:36:05.884   770   788 I GeckoDump: ⰲ겿{"action":"test_status","time":1487608565868,"thread":null,"pid":null,"source":"mochitest","test":"/tests/widget/tests/test_assign_event_data.html","subtest":"WidgetKeyboardEvent (keydown during composition) (gCallPreventDefault=false): failed to get duplicated event","status":"PASS","js_source":"TestRunner.js"}ⰲ겿

:masayuki - Do you have an idea of what's going wrong here? (It only seems to happen, intermittently, on Android Debug.)
Flags: needinfo?(masayuki)
Although, we should fix the warnings, but it shouldn't cause any orange because the warnings mean the method is called unexpectedly (but it does nothing).
https://dxr.mozilla.org/mozilla-central/rev/106a96755d3bcebe64bbbc3b521d65d262ba9c02/editor/libeditor/EditorBase.cpp#1889

Looks like that the failure may be possible if doTest() for next test is called before gCallback. Although, I don't have any ideas of the reason.

https://dxr.mozilla.org/mozilla-central/source/widget/tests/test_assign_event_data.html#200,204,214
The test creates composition and commit it immediately. If the orange is only in Android, something after TextInputProcessor dispatches composition events and keyup event might be handled asynchronously?

jchen: do you have some ideas?
Flags: needinfo?(masayuki) → needinfo?(nchen)
jchen: Also, if a fix is not apparent, would it be okay to skip this test on Android Debug? (It looks like we could continue to run reliably on Android Opt.)
> would it be okay to skip this test on Android Debug?

I think so because the tested path is only XP part, not platform dependent. The reason why the test depends on Android is, the path to fire events from widget depends on platform specific code in widget. But the testing code is under dom/events.

# But of course, it's the best way to test it on all platforms.
I would wait and see if bug 1137567 part 6 that just landed changes things, but I suppose we can disable it for debug.
Flags: needinfo?(nchen)
Comment on attachment 8843276 [details] [diff] [review]
disable test for android/debug

Review of attachment 8843276 [details] [diff] [review]:
-----------------------------------------------------------------

Failures continue after the landing in bug 1137567:

https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=83697590110c895052be413677425def87208bde

so let's go ahead and disable this on Android Debug.
Attachment #8843276 - Flags: review?(gbrown) → review+
Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/86e1748ab478
Intermittent widget/tests/test_assign_event_data.html. disable for android debug. r=gbrown
Assignee: gbrown → nobody
Keywords: leave-open
Whiteboard: [stockwell needswork]tpi:- → [stockwell disabled]tpi:-
Bulk priority update of open intermittent test failure bugs. 

P3 => P5

https://bugzilla.mozilla.org/show_bug.cgi?id=1381960
Priority: P3 → P5
You need to log in before you can comment on or make changes to this bug.