Closed
Bug 1330589
Opened 8 years ago
Closed 7 years ago
stylo: Talos job "tcanvasmark" failed with timeout
Categories
(Testing :: Talos, defect, P3)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: shinglyu, Unassigned)
References
Details
Attachments
(3 files)
https://treeherder.mozilla.org/#/jobs?repo=try&revision=cc113d4a2d9acd3eb7cd30542bf1f079e7ae5705&selectedJob=66082391
TEST-UNEXPECTED-ERROR | tcanvasmark | timeout
Reporter | ||
Updated•8 years ago
|
Component: CSS Parsing and Computation → Talos
Product: Core → Testing
Reporter | ||
Comment 1•8 years ago
|
||
The error message:
17:27:49 INFO - PROCESS | 4890 | ERROR:geckoservo::glue: Unnecessary call to traverse_subtree
17:27:49 INFO - PROCESS | 4890 | ERROR:geckoservo::glue: Unnecessary call to traverse_subtree
17:27:49 INFO - PROCESS | 4890 | ERROR:geckoservo::glue: Unnecessary call to traverse_subtree
17:27:49 INFO - PROCESS | 4890 | ERROR:geckoservo::glue: Unnecessary call to traverse_subtree
17:27:49 INFO - PROCESS | 4890 | ERROR:geckoservo::glue: Unnecessary call to traverse_subtree
17:27:49 INFO - PROCESS | 2030 | *************************
17:27:49 INFO - PROCESS | 2030 | A coding exception was thrown and uncaught in a Task.
17:27:49 INFO - PROCESS | 2030 |
17:27:49 ERROR - PROCESS | 2030 | Full message: TypeError: NetworkError when attempting to fetch resource.
17:27:49 INFO - PROCESS | 2030 | Full stack:
17:27:49 INFO - PROCESS | 2030 | *************************
17:37:49 INFO - Terminating psutil.Process(pid=2030, name='firefox')
17:37:49 INFO - TEST-UNEXPECTED-ERROR | tcanvasmark | timeout
17:37:49 ERROR - Traceback (most recent call last):
17:37:49 INFO - File "/home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/run_tests.py", line 202, in run_tests
17:37:49 INFO - talos_results.add(mytest.runTest(browser_config, test))
17:37:49 INFO - File "/home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/ttest.py", line 70, in runTest
17:37:49 INFO - return self._runTest(browser_config, test_config, setup)
17:37:49 INFO - File "/home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/ttest.py", line 174, in _runTest
17:37:49 INFO - if counter_management else None),
17:37:49 INFO - File "/home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/talos_process.py", line 117, in run_browser
17:37:49 INFO - raise TalosError("timeout")
17:37:49 INFO - TalosError: timeout
17:37:49 INFO - TEST-INFO took 902868ms
17:37:49 INFO - SUITE-END | took 1216s
17:37:50 ERROR - Return code: 2
17:37:50 WARNING - setting return code to 2
17:37:50 ERROR - # TBPL FAILURE #
Reporter | ||
Comment 2•8 years ago
|
||
Saw this in the developer console.
Reporter | ||
Comment 3•8 years ago
|
||
The problem is that the testing/talos/talos/tests/canvasmark/index.html test is not running. So the test suite fails.
Reporter | ||
Comment 4•8 years ago
|
||
Got this trace while trying to run
Game.centerFillText(ctx, "Please wait... Loading Images...", "18pt Helvetica", GameHandler.height/2, "#eee");
*************************
console.error:
Message: TypeError: rule is null
Stack:
_getImported/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:939:1
_run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:917:23
walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:801:7
Promise*scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:734:11
schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:765:7
Promise.prototype.then@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:450:5
_handleResultValue@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:387:7
_run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:319:13
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:273:3
asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:247:14
spawn@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:161:12
_getImported@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:933:12
_addStyleSheets/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:915:29
_run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:273:3
asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:247:14
spawn@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:161:12
_addStyleSheets@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:888:12
StyleSheetsActor<.getStyleSheets<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:839:26
_run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:273:3
asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:247:14
handler@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1082:19
onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/main.js:1776:15
receiveMessage@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/transport/transport.js:761:7
enter@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/script.js:350:5
_pushThreadPause@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/script.js:533:5
_pauseAndRespond@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/script.js:741:7
hit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/breakpoint.js:172:12
onRenderScene@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:6239:13
frame@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:282:13
start@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:109:7
onloadHandler/g_splashImg.onload@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:6091:7
EventHandlerNonNull*onloadHandler@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:6087:4
EventListener.handleEvent*@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:6077:1
_getImported/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:939:1
_run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:917:23
walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:801:7
Promise*scheduleWalkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:734:11
schedulePromise@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:765:7
Promise.prototype.then@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:450:5
_handleResultValue@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:387:7
_run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:319:13
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:273:3
asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:247:14
spawn@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:161:12
_getImported@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:933:12
_addStyleSheets/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:915:29
_run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:273:3
asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:247:14
spawn@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:161:12
_addStyleSheets@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:888:12
StyleSheetsActor<.getStyleSheets<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/stylesheets.js:839:26
_run@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:311:39
TaskImpl@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:273:3
asyncFunction@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/task.js:247:14
handler@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/protocol.js:1082:19
onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/main.js:1776:15
receiveMessage@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/transport/transport.js:761:7
enter@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/script.js:350:5
_pushThreadPause@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/script.js:533:5
_pauseAndRespond@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/script.js:741:7
hit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/server/actors/breakpoint.js:172:12
onRenderScene@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:6239:13
frame@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:282:13
start@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:109:7
onloadHandler/g_splashImg.onload@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:6091:7
EventHandlerNonNull*onloadHandler@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:6087:4
EventListener.handleEvent*@file:///home/shinglyu/workspace/stylo/stylo-incubator-2/testing/talos/talos/tests/canvasmark/scripts/canvasmark_v6.js:6077:1
console.error:
Protocol error (unknownError): rule is null
Reporter | ||
Comment 5•8 years ago
|
||
Hi Cameron,
Do you have any idea about our canvas support? Looks like it hangs when trying to call fillText on a canvas?
Flags: needinfo?(cam)
Comment 6•8 years ago
|
||
We currently throw an exception from some canvas methods, since we haven't got the ability to ask Servo for an nsStyleContext based on a given list of rules.
http://searchfox.org/mozilla-central/rev/30fcf167af036aeddf322de44a2fadd370acfd2f/dom/canvas/CanvasRenderingContext2D.cpp#2649
That might be the cause of the timeout.
Flags: needinfo?(cam)
Reporter | ||
Comment 7•8 years ago
|
||
I've isolated the test case, it's caused by setting the font:
var c = document.getElementById("canvas");
var ctx = c.getContext("2d");
ctx.font = "18pt Helvetica" // <<< GOTCHA!
ctx.fillText("Hello World", 25, 25)
(Also attached the html and screenshot)
Is there anything blocking us from implementing it? Or is there anyone already working on it?
Flags: needinfo?(cam)
Reporter | ||
Comment 8•8 years ago
|
||
Reporter | ||
Comment 9•8 years ago
|
||
The left is the expected result (using firefox-dev edition), the left is stylo and the error message.
Comment 10•8 years ago
|
||
(In reply to Shing Lyu [:shinglyu] from comment #7)
> I've isolated the test case, it's caused by setting the font:
>
> var c = document.getElementById("canvas");
> var ctx = c.getContext("2d");
> ctx.font = "18pt Helvetica" // <<< GOTCHA!
> ctx.fillText("Hello World", 25, 25)
>
> (Also attached the html and screenshot)
>
> Is there anything blocking us from implementing it? Or is there anyone
> already working on it?
That's bug 1324700, nobody's working on it yet though.
Flags: needinfo?(cam)
Reporter | ||
Updated•8 years ago
|
Summary: [Stylo] Talos job "c" failed with timeout → [Stylo] Talos job "tcanvasmark" failed with timeout
Reporter | ||
Comment 11•8 years ago
|
||
The "c" job contains two subtests "tresize" and "tcanvasmark". Only "tcanvasmark" is failing.
Updated•8 years ago
|
Priority: -- → P3
Summary: [Stylo] Talos job "tcanvasmark" failed with timeout → stylo: Talos job "tcanvasmark" failed with timeout
Comment 12•8 years ago
|
||
The minimal test case in comment 8 looks fine now.
Shing, is there anything what I can do for this?
Flags: needinfo?(shing.lyu)
Reporter | ||
Comment 13•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&filter-searchStr=stylo&selectedJob=104379974
^ tcanvasmark test seems to run just fine now. Let's close it as WORKSFORME.
Flags: needinfo?(shing.lyu)
Reporter | ||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•