Intermittent activities_test.js | Contacts > Activities webcontacts/email activity Creates only one instance of action menu

RESOLVED FIXED

Status

Firefox OS
Gaia::Contacts
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: philor, Assigned: julienw)

Tracking

({intermittent-failure})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Updated

2 years ago
Summary: Intermittent /activities_test.js | Contacts > Activities webcontacts/email activity Creates only one instance of action menu → Intermittent activities_test.js | Contacts > Activities webcontacts/email activity Creates only one instance of action menu
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 210

2 years ago
17 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 10
* b2g-inbound: 3
* try: 2
* fx-team: 2

Platform breakdown:
* b2g-linux64: 13
* mulet-linux64: 4

For more details, see:
http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-01&endday=2015-10-01&tree=all

Comment 211

2 years ago
15 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 8
* fx-team: 3
* b2g-inbound: 3
* mozilla-central: 1

Platform breakdown:
* b2g-linux64: 8
* mulet-linux64: 7

For more details, see:
http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-02&endday=2015-10-02&tree=all

Comment 212

2 years ago
88 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 43
* fx-team: 23
* b2g-inbound: 17
* mozilla-central: 3
* try: 2

Platform breakdown:
* b2g-linux64: 49
* mulet-linux64: 39

For more details, see:
http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-09-28&endday=2015-10-04&tree=all

Comment 213

2 years ago
16 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 11
* fx-team: 4
* b2g-inbound: 1

Platform breakdown:
* b2g-linux64: 9
* mulet-linux64: 7

For more details, see:
http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-05&endday=2015-10-05&tree=all

Comment 214

2 years ago
17 automation job failures were associated with this bug yesterday.

Repository breakdown:
* b2g-inbound: 6
* mozilla-inbound: 5
* fx-team: 3
* try: 2
* mozilla-central: 1

Platform breakdown:
* b2g-linux64: 9
* mulet-linux64: 7
* linux32: 1

For more details, see:
http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-06&endday=2015-10-06&tree=all

Comment 215

2 years ago
65 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 28
* b2g-inbound: 21
* fx-team: 10
* mozilla-central: 4
* try: 2

Platform breakdown:
* b2g-linux64: 35
* mulet-linux64: 29
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-05&endday=2015-10-11&tree=all

Comment 216

2 years ago
39 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 21
* b2g-inbound: 14
* mozilla-central: 2
* fx-team: 2

Platform breakdown:
* b2g-linux64: 21
* mulet-linux64: 18

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-12&endday=2015-10-18&tree=all

Comment 217

2 years ago
19 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 13
* fx-team: 3
* b2g-inbound: 2
* mozilla-central: 1

Platform breakdown:
* b2g-linux64: 11
* mulet-linux64: 8

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-19&endday=2015-10-19&tree=all

Comment 218

2 years ago
58 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 30
* b2g-inbound: 17
* fx-team: 7
* mozilla-central: 3
* try: 1

Platform breakdown:
* b2g-linux64: 30
* mulet-linux64: 26
* b2g-emu-ics: 1
* b2g-device-image: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-19&endday=2015-10-25&tree=all

Comment 219

2 years ago
43 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 22
* fx-team: 13
* b2g-inbound: 6
* mozilla-central: 2

Platform breakdown:
* mulet-linux64: 32
* b2g-linux64: 10
* android-4-3-armv7-api11: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-10-26&endday=2015-11-01&tree=all

Comment 220

2 years ago
19 automation job failures were associated with this bug yesterday.

Repository breakdown:
* b2g-inbound: 7
* mozilla-inbound: 6
* fx-team: 5
* mozilla-central: 1

Platform breakdown:
* mulet-linux64: 11
* b2g-linux64: 8

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-11-05&endday=2015-11-05&tree=all

Comment 221

2 years ago
18 automation job failures were associated with this bug yesterday.

Repository breakdown:
* b2g-inbound: 5
* mozilla-inbound: 4
* gaia: 4
* fx-team: 3
* mozilla-central: 2

Platform breakdown:
* b2g-linux64: 8
* mulet-linux64: 6
* gaia: 4

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-11-06&endday=2015-11-06&tree=all

Comment 222

2 years ago
73 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 30
* b2g-inbound: 24
* fx-team: 12
* gaia: 4
* mozilla-central: 3

Platform breakdown:
* mulet-linux64: 35
* b2g-linux64: 34
* gaia: 4

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-11-02&endday=2015-11-08&tree=all

Comment 223

2 years ago
18 automation job failures were associated with this bug yesterday.

Repository breakdown:
* gaia: 10
* mozilla-inbound: 8

Platform breakdown:
* gaia: 10
* b2g-linux64: 5
* mulet-linux64: 3

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-11-09&endday=2015-11-09&tree=all
Blocks: 1222215
This is one of our top intermittents. Could you guys help out?
Flags: needinfo?(francisco)
Flags: needinfo?(felash)
(Assignee)

Comment 225

2 years ago
I can take it, it consistently fail for me locally.
Assignee: nobody → felash
Flags: needinfo?(felash)
(Assignee)

Comment 226

2 years ago
So, it looks like the click handler for the "add email" link is not registered when we tap on it, on a fast machine. But I don't yet understand how this can happen.
(Assignee)

Comment 227

2 years ago
Replacing `.click()` by `.scriptWith(function(el) { el.click(); })` in Contacts.addContactMultipleEmails works better, but I don't understand why...
Created attachment 8685704 [details] [review]
[gaia] julienw:1205907-fix-contacts-marionette-test > mozilla-b2g:master
(Assignee)

Comment 229

2 years ago
Comment on attachment 8685704 [details] [review]
[gaia] julienw:1205907-fix-contacts-marionette-test > mozilla-b2g:master

Hey Johan,

I admit I don't understand what I'm doing here, but it's fixing the problem.

I'd like to have your expert advice on why this could fix the issue.
Attachment #8685704 - Flags: feedback?(jlorenzo)

Comment 230

2 years ago
17 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 10
* fx-team: 5
* b2g-inbound: 2

Platform breakdown:
* mulet-linux64: 14
* b2g-linux64: 3

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-11-11&endday=2015-11-11&tree=all
(Assignee)

Comment 231

2 years ago
If .click() is implemented using [1] (the "clickElement" marionette command), it seems it actually wants to simulate a real mouse click.

It's actually way different than calling ".click()" (which is using the DOM API).

[1] https://github.com/SeleniumHQ/selenium/blob/9d157ae962ad5857ff28e7cbedcc0d57fd30d143/javascript/firefox-driver/js/wrappedElement.js#L63
(Assignee)

Comment 232

2 years ago
I think I have a clue: when displaying the "add contact" panel, we have a transition happening, and while it's happening nothing happens when I tap the "add email" (even when trying manually). I suspect this is happening here.
(Assignee)

Updated

2 years ago
Attachment #8685704 - Flags: feedback?(jlorenzo)
(Assignee)

Updated

2 years ago
Flags: needinfo?(francisco)
(Assignee)

Comment 233

2 years ago
Comment on attachment 8685704 [details] [review]
[gaia] julienw:1205907-fix-contacts-marionette-test > mozilla-b2g:master

Hey Francisco,

I didn't understand why we have "pointer-events: none" on the "next" panel, I think we shouldn't have it. So I removed it and it fixes the issue for me :)

(I totally understand we have it on the "current" panel though).

Note the integration test failure happens when the computer is too quick (contrary to other intermittent we see :) ).
Attachment #8685704 - Flags: review?(francisco)

Comment 234

2 years ago
73 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 36
* fx-team: 11
* b2g-inbound: 11
* gaia: 10
* mozilla-central: 5

Platform breakdown:
* mulet-linux64: 38
* b2g-linux64: 25
* gaia: 10

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-11-09&endday=2015-11-15&tree=all
Comment on attachment 8685704 [details] [review]
[gaia] julienw:1205907-fix-contacts-marionette-test > mozilla-b2g:master

Was tracking why we added that and come from:

https://bugzilla.mozilla.org/show_bug.cgi?id=935307

But right now with the current views (even without splitting), that's not needed anymore.

Tested on the phone and looking good, also tried to repro bug 935307 and couldn't so r+ on my side.

Thanks for the patch!
Attachment #8685704 - Flags: review?(francisco) → review+
(Assignee)

Comment 236

2 years ago
Wow, if this is a patch for bug 935307, this looks a lot like a workaround :) anyway we still block pointer events for the prev panel.

Thanks !
(Assignee)

Comment 237

2 years ago
master: 2a5d9511103e129473591043568c426f8f14f5f8
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
(Assignee)

Comment 238

2 years ago
Should this be uplifted to the nga branch as well ? (I can do it, I just want to know if necessary :) )
Flags: needinfo?(francisco)
Yes please, we still use that navigation object for subviews (like the search one in the list, or the import in settings)

Thanks again.
Flags: needinfo?(francisco)
(Assignee)

Comment 240

2 years ago
nga: 807ed988e6d4736851680fe6ad27b934be02c24f

Comment 241

2 years ago
18 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 10
* fx-team: 5
* b2g-inbound: 3

Platform breakdown:
* mulet-linux64: 14
* b2g-linux64: 4

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1205907&startday=2015-11-16&endday=2015-11-22&tree=all
(Assignee)

Comment 242

2 years ago
From the graph, it's 0 occurences since November 19th. \o/

b2g-inbound stops November 16th and then I guess we needed to wait for the patch propagation.
You need to log in before you can comment on or make changes to this bug.