Open Bug 1285414 Opened 9 years ago Updated 2 years ago

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

Categories

(Core :: Widget, defect, P5)

Product:
Core
Component:
Widget
Type:
defect

Tracking

()

People

(Reporter: intermittent-bug-filer, Unassigned)

References

Details

(Keywords: intermittent-failure, leave-open, Whiteboard: [stockwell disabled]tpi:-)

Attachments

(1 file)

disable test for android/debug
746 bytes, patch
gbrown
: review+
Details | Diff | Splinter Review
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)
Attached patch disable test for android/debug — — Splinter Review
Attachment #8843276 - Flags: review?(gbrown)
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
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: