Open
Bug 795716
Opened 12 years ago
Updated 2 years ago
Enable test_network_events.html
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
NEW
| blocking-basecamp | - |
People
(Reporter: mounir, Unassigned)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
Because of bug 795136, tests for bug 795711 have been disabled. We should make sure they are enabled as soon as bug 795136 is resolved.
|
||
Comment 1•12 years ago
|
||
Why is nomed for basecamp? We don't normally block on automation. Is there a special reason in this case?
|
Reporter | |
Comment 2•12 years ago
|
||
We don't block on tests? So it's fine to ship an untested feature?
|
|
Reporter | |
Comment 3•12 years ago
|
||
With the patch in bug 795711 landed, we would be good to re-enable this test.
|
||
Comment 4•12 years ago
|
||
Comment on attachment 666333 [details] [diff] [review]
Patch
r=me
Attachment #666333 -
Flags: review?(bzbarsky) → review+
|
|
Reporter | |
Comment 5•12 years ago
|
||
Sent to try with other patches:
https://tbpl.mozilla.org/?tree=Try&rev=6a6690d20239
|
||
Comment 6•12 years ago
|
||
(In reply to Mounir Lamouri (:mounir) from comment #2)
> We don't block on tests? So it's fine to ship an untested feature?
We need to test everything but we don't hold the release for individual tests. Of course, this can easily land :)
blocking-basecamp: ? → -
|
|
Reporter | |
Comment 7•12 years ago
|
||
For some reasons, debug builds and some Android builds time out on this test...
https://tbpl.mozilla.org/?tree=Try&rev=51f0b00da284
|
Assignee | |
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
|
||
Comment 8•2 years ago
|
||
The bug assignee is inactive on Bugzilla, so the assignee is being reset.
Assignee: mounir → nobody
Status: ASSIGNED → NEW
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•