Closed
Bug 980972
Opened 10 years ago
Closed 10 years ago
Detect marionette script timeout
Categories
(Firefox OS Graveyard :: Gaia::PerformanceTest, defect, P1)
Tracking
(b2g-v1.3T fixed)
RESOLVED
FIXED
1.4 S5 (11apr)
| Tracking | Status | |
|---|---|---|
| b2g-v1.3T | --- | fixed |
People
(Reporter: hub, Assigned: hub)
References
Details
(Keywords: perf, Whiteboard: [c=automation p=1 s= u=])
Attachments
(1 file)
We should detect marionette timeout. Like in the performanceHelper.observe(); performanceHelper.waitForPerfEvent(); pattern. This could help us track regressions in the testing.
|
Assignee | |
Updated•10 years ago
|
|
|
Assignee | |
Updated•10 years ago
|
|
|
Assignee | |
Comment 2•10 years ago
|
||
(In reply to Hubert Figuiere [:hub] from comment #1) > Bug 981009 must be fixed first. Not really.
|
|
Assignee | |
Updated•10 years ago
|
Priority: -- → P1
|
|
Assignee | |
Comment 3•10 years ago
|
||
Attachment #8389377 -
Flags: review?(felash)
|
|
Assignee | |
Updated•10 years ago
|
Whiteboard: [c=automation p= s= u=] → [c=automation p=1 s= u=]
|
||
Comment 4•10 years ago
|
||
Hey Hubert, I won't be able to review this before my holidays. So either this will wait until 24th March, or you'll need to find another reviewer.
|
|
Assignee | |
Comment 5•10 years ago
|
||
Comment on attachment 8389377 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17092 James, do you think you can review this?
Attachment #8389377 -
Flags: review?(felash) → review?(jlal)
|
|
Assignee | |
Updated•10 years ago
|
Summary: Detect marionette timeout → Detect marionette script timeout
|
|
Assignee | |
Comment 6•10 years ago
|
||
Comment on attachment 8389377 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17092 Julien, do you mind since you are back from PTO? Thanks.
Attachment #8389377 -
Flags: review?(jlal) → review?(felash)
|
|
||
Comment 7•10 years ago
|
||
Comment on attachment 8389377 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17092 comments on github thanks!
Attachment #8389377 -
Flags: review?(felash)
|
|
Assignee | |
Comment 8•10 years ago
|
||
Comment on attachment 8389377 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17092 Updated the PR.
Attachment #8389377 -
Flags: review?(felash)
|
|
||
Comment 9•10 years ago
|
||
Comment on attachment 8389377 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/17092 r=me except if you want to do the comment about the "done" function, in that case I'd like another review please :)
Attachment #8389377 -
Flags: review?(felash) → review+
|
|
Assignee | |
Comment 10•10 years ago
|
||
I got hit by an unrelated intermittent Travis error. All green now. Merged https://github.com/mozilla-b2g/gaia/commit/fb0aec1ef13f915a9739fc889df88ceda07c7b53
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
|
||
Updated•10 years ago
|
Target Milestone: --- → 1.4 S5 (11apr)
|
|
Assignee | |
Comment 11•10 years ago
|
||
v1.3 uplift https://github.com/mozilla-b2g/gaia/commit/0fea32c302b5f22ce030db3b987c119cb94e922e
status-b2g-v1.3T:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•