Intermittent failure in content/html/document/test/test_bug448564.html | Test timed out.

RESOLVED FIXED in mozilla7

Status

()

Core
DOM: Core & HTML
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: mounir, Assigned: mounir)

Tracking

({intermittent-failure})

Trunk
mozilla7
intermittent-failure
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
s: talos-r3-fed64-046
62119 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/document/test/test_bug448564.html | Test timed out.

http://tinderbox.mozilla.org/showlog.cgi?log=Cedar/1304678289.1304679496.12575.gz
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)
(Assignee)

Comment 12

6 years ago
Created attachment 540425 [details] [diff] [review]
Patch v1
Assignee: nobody → mounir.lamouri
Status: NEW → ASSIGNED
Attachment #540425 - Flags: review?(jonas)
Attachment #540425 - Flags: feedback?(ehsan)
(Assignee)

Updated

6 years ago
Whiteboard: [orange] → [orange][needs review]
(Assignee)

Comment 13

6 years ago
Created attachment 540427 [details] [diff] [review]
Patch v1
Attachment #540425 - Attachment is obsolete: true
Attachment #540425 - Flags: review?(jonas)
Attachment #540425 - Flags: feedback?(ehsan)
Attachment #540427 - Flags: review?(jonas)
Attachment #540427 - Flags: feedback?(ehsan)
Comment on attachment 540427 [details] [diff] [review]
Patch v1

I'm not really sure how this patch is going to fix the orange.  Also, Doesn't this potentially change the semantics of the test?
(Assignee)

Comment 15

6 years ago
(In reply to comment #14)
> I'm not really sure how this patch is going to fix the orange.

I'm not sure either but I think it makes the test much saner and simple to understand. It might fix the orange in the way. Otherwise, it will be easier to track the issue.

> Also, Doesn't this potentially change the semantics of the test?

I don't think so. The idea of this test is to submit the three forms and make sure everything is okay. I'm still using the SJS and the same HTML code in the frames. The only difference is I'm checking the frames when all of them are done, not each one individually. I don't think that change can change the semantic of the test.
Attachment #540427 - Flags: feedback?(ehsan) → feedback+
Comment on attachment 540427 [details] [diff] [review]
Patch v1

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

r=me
Attachment #540427 - Flags: review?(jonas) → review+
(Assignee)

Comment 17

6 years ago
Pushed:
http://hg.mozilla.org/mozilla-central/rev/e00de9b8df15

Marking it as fixed and will reopen if a failure happens again.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [orange][needs review] → [orange]
Target Milestone: --- → mozilla7
Comment hidden (Treeherder Robot)
Keywords: intermittent-failure
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.