Open Bug 1316953 Opened 3 years ago Updated 3 months ago

Intermittent browser/components/translation/test/browser_translation_infobar.js | timeout waiting for the info bar to reappear -

Categories

(Firefox :: Translation, defect)

defect
Not set

Tracking

()

People

(Reporter: intermittent-bug-filer, Unassigned)

Details

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

Attachments

(2 files, 1 obsolete file)

Whiteboard: [stockwell unknown]
There have been a total of 31 failures in the last week, according to Orange Factor.

Occurrences per platform:
-OS X 10.10: 17
-linux64-stylo-disabled: 7
-Linux x64: 6
-macosx64-nightly: 1

Occurrences per build type:
-debug: 1
-pgo: 4
-opt: 26


Here is a recent log file and a snippet with the failure:

https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=159292493&lineNumber=4666

03:03:10     INFO - TEST-PASS | browser/components/translation/test/browser_translation_infobar.js | no 'translate' notification after clicking the url bar icon - 
03:03:10     INFO - Check that clicking the url bar icon reopens the info bar
03:03:10     INFO - TEST-PASS | browser/components/translation/test/browser_translation_infobar.js | translate icon shown - 
03:03:10     INFO - TEST-PASS | browser/components/translation/test/browser_translation_infobar.js | translated icon not shown - 
03:03:10     INFO - Buffered messages logged at 03:03:07
03:03:10     INFO - Console message: [JavaScript Error: "The character encoding of the plain text document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the file needs to be declared in the transfer protocol or file needs to use a byte order mark as an encoding signature." {file: "data:text/plain,test%20page" line: 0}]
03:03:10     INFO - Buffered messages finished
03:03:10     INFO - TEST-UNEXPECTED-FAIL | browser/components/translation/test/browser_translation_infobar.js | timeout waiting for the info bar to reappear - 
03:03:10     INFO - Stack trace:
03:03:10     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition/interval<:17
03:03:10     INFO - Not taking screenshot here: see the one that was previously logged
03:03:10     INFO - TEST-UNEXPECTED-FAIL | browser/components/translation/test/browser_translation_infobar.js | there's a 'translate' notification -
Flags: needinfo?(felipc)
Whiteboard: [stockwell unknown] → [stockwell needswork]
In the last 7 days we have 32 failures.
They occur mostly on OS X 10.10 (opt and debug), Linux x64 (opt and pgo), linux64-stylo-disabled (opt), linux64-stylo-disabled (opt), macosx64-nightly (opt).
Recent log example: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=165668931&lineNumber=3359
and a relevant part of it
15:05:56     INFO - TEST-UNEXPECTED-FAIL | browser/components/translation/test/browser_translation_infobar.js | timeout waiting for the info bar to reappear - 
15:05:56     INFO - Stack trace:
15:05:56     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition/interval<:17
15:05:56     INFO - Not taking screenshot here: see the one that was previously logged
15:05:56     INFO - TEST-UNEXPECTED-FAIL | browser/components/translation/test/browser_translation_infobar.js | there's a 'translate' notification - 
:Felipe Gomes, can you please take a look at this?
Whiteboard: [stockwell unknown] → [stockwell needswork]
Hi,
This bug just had a major spike in the last day going from 18 failures to 46 failures overnight.

It fails on linux and osx platforms on all buildtypes.

Here is a recent log:
https://treeherder.mozilla.org/logviewer.html#?job_id=194245292&repo=mozilla-inbound&lineNumber=3924

TEST-PASS | browser/components/translation/test/browser_translation_infobar.js | translated icon not shown - 
00:01:50     INFO - Console message: [JavaScript Error: "The character encoding of the plain text document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the file needs to be declared in the transfer protocol or file needs to use a byte order mark as an encoding signature." {file: "data:text/plain,test page" line: 0}]
00:01:50     INFO - Buffered messages finished
00:01:50     INFO - TEST-UNEXPECTED-FAIL | browser/components/translation/test/browser_translation_infobar.js | timeout waiting for the info bar to reappear - 
00:01:50     INFO - Stack trace:
00:01:50     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition/interval<:17
00:01:50     INFO - setInterval handler*chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition:15
00:01:50     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:run_tests:211
00:01:50     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:test/<:86
00:01:50     INFO - promise callback*chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:test:80
00:01:50     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1137
00:01:50     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:999
00:01:50     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
00:01:50     INFO - Not taking screenshot here: see the one that was previously logged
00:01:50     INFO - TEST-UNEXPECTED-FAIL | browser/components/translation/test/browser_translation_infobar.js | there's a 'translate' notification - 
00:01:50     INFO - Stack trace:
00:01:50     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:run_tests/<:212
00:01:50     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:moveOn:32
00:01:50     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition/interval<:18
00:01:50     INFO - setInterval handler*chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition:15
00:01:50     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:run_tests:211
00:01:50     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:test/<:86
00:01:50     INFO - promise callback*chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:test:80
00:01:50     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1137
00:01:50     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:999
00:01:50     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
00:01:50     INFO - GECKO(1039) | MEMORY STAT | vsize 4403MB | residentFast 309MB | heapAllocated 104MB
00:01:50     INFO - TEST-OK | browser/components/translation/test/browser_translation_infobar.js | took 3569ms
00:01:50     INFO - checking window state
00:01:50     INFO - TEST-START | browser/components/translation/test/browser_translation_telemetry.js

Felipe Gomes: Can you please take a look at this bug?
Whiteboard: [stockwell unknown] → [stockwell needswork]
Attached patch Disable test on mac and linux (obsolete) — Splinter Review
Most of the failures are on Linux (32&64) and OSX-10-10.
Attachment #9002169 - Flags: review?(jmaher)
Comment on attachment 9002169 [details] [diff] [review]
Disable test on mac and linux

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

::: browser/components/translation/test/browser.ini
@@ +11,4 @@
>  skip-if = os == 'win' && !e10s # bug 1374446
>  [browser_translation_telemetry.js]
>  [browser_translation_infobar.js]
> +skip-if = (os == 'mac') || (os == 'linux') # Bug 1316953

I would like to keep the 'verify' clause, so please make this:
skip-if = (verify && debug && (os == 'mac')) || (os == 'mac') || (os == 'linux') # Bug 1316953
Attachment #9002169 - Flags: review?(jmaher) → review-
Modified the patch so that it includes the verify clause.
Attachment #9002169 - Attachment is obsolete: true
Attachment #9002459 - Flags: review?(jmaher)
Comment on attachment 9002459 [details] [diff] [review]
Disable test on mac and linux

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

thanks!
Attachment #9002459 - Flags: review?(jmaher) → review+
Whiteboard: [stockwell disable-recommended] → [stockwell disabled]
Pushed by shindli@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/60f12b694614
Disable browser_translation_infobar.js on mac and linux for frequent failures. r=jmaher
Keywords: checkin-needed
This bug has failed 33 times in the last 7 days, failures occur on windows platfomrs on debug build type.

Log:
https://treeherder.mozilla.org/logviewer.html#?job_id=196826545&repo=mozilla-central&lineNumber=23335

 TEST-PASS | browser/components/translation/test/browser_translation_infobar.js | translated icon not shown - 
05:58:59     INFO - Buffered messages logged at 05:58:56
05:58:59     INFO - Console message: [JavaScript Error: "The character encoding of the plain text document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the file needs to be declared in the transfer protocol or file needs to use a byte order mark as an encoding signature." {file: "data:text/plain,test page" line: 0}]
05:58:59     INFO - Buffered messages finished
05:58:59     INFO - TEST-UNEXPECTED-FAIL | browser/components/translation/test/browser_translation_infobar.js | timeout waiting for the info bar to reappear - 
05:58:59     INFO - Stack trace:
05:58:59     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition/interval<:17
05:58:59     INFO - setInterval handler*chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition:15
05:58:59     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:run_tests:211
05:58:59     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:test/<:86
05:58:59     INFO - promise callback*chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:test:80
05:58:59     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1133
05:58:59     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:995
05:58:59     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
05:58:59     INFO - Not taking screenshot here: see the one that was previously logged
05:58:59     INFO - TEST-UNEXPECTED-FAIL | browser/components/translation/test/browser_translation_infobar.js | there's a 'translate' notification - 
05:58:59     INFO - Stack trace:
05:58:59     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:run_tests/<:212
05:58:59     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:moveOn:32
05:58:59     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition/interval<:18
05:58:59     INFO - setInterval handler*chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:waitForCondition:15
05:58:59     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:run_tests:211
05:58:59     INFO - chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:test/<:86
05:58:59     INFO - promise callback*chrome://mochitests/content/browser/browser/components/translation/test/browser_translation_infobar.js:test:80
05:58:59     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1133
05:58:59     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:995
05:58:59     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
05:58:59     INFO - GECKO(2720) | MEMORY STAT | vsize 2098933MB | vsizeMaxContiguous 128758483MB | residentFast 288MB | heapAllocated 90MB
05:58:59     INFO - TEST-OK | browser/components/translation/test/browser_translation_infobar.js | took 4465ms
05:58:59     INFO - GECKO(2720) | ++DOCSHELL 000002E2C533C800 == 1 [pid = 6568] [id = {2565dde6-8a39-460d-a9fd-00fb52e71907}]
05:58:59     INFO - GECKO(2720) | ++DOMWINDOW == 7 (000002E2C5314000) [pid = 6568] [serial = 7] [outer = 0000000000000000]
05:58:59     INFO - GECKO(2720) | ++DOMWINDOW == 8 (000002E2CCC77000) [pid = 6568] [serial = 8] [outer = 000002E2C5314000]
05:58:59     INFO - checking window state
05:58:59     INFO - GECKO(2720) | ++DOMWINDOW == 9 (000002E2CD175000) [pid = 6568] [serial = 9] [outer = 000002E2C5314000]
05:58:59     INFO - TEST-START | browser/components/translation/test/browser_translation_telemetry.js

:Felipe Gomes: Can you please take a look at this bug?
Whiteboard: [stockwell disabled] → [stockwell disabled] [stockwell needswork]
(In reply to Dorel Luca [:dluca] from comment #74)
> This bug has failed 33 times in the last 7 days, failures occur on windows
> platforms on debug build type.

I'm also gonna disable it for all debug builds.  The translation infobar will probably be removed in the near future, so it's not a big deal
Flags: needinfo?(felipc)
Assignee: nobody → felipc
Status: NEW → ASSIGNED
Attachment #9006072 - Flags: review+
Pushed by felipc@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9dbafdb2ecb6
Disable browser_translation_infobar.js on debug builds. r=me
This test is still running on windows opt, and disabling it on other platforms has brought the number of failures down significantly. I don't think it needs more attention at the moment.
Assignee: felipc → nobody
Status: ASSIGNED → NEW
Whiteboard: [stockwell disabled] [stockwell needswork] → [stockwell]
You need to log in before you can comment on or make changes to this bug.