Closed Bug 972826 Opened 10 years ago Closed 10 years ago

Intermittent test_load_same_resource.html | Test timed out.

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla33
Tracking Flags:
Tracking Status
firefox31 --- wontfix
firefox32 --- fixed
firefox33 --- fixed
firefox-esr24 --- unaffected
firefox-esr31 --- fixed
b2g-v1.3 --- unaffected
b2g-v1.3T --- unaffected
b2g-v1.4 --- fixed
b2g-v2.0 --- fixed
b2g-v2.1 --- fixed

People

(Reporter: mayhemer, Assigned: jwwang)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

972826_fix_test_load_same_resource-v2.patch
2.21 KB, patch
cajbir
: review+
Details | Diff | Splinter Review 
TEST-UNEXPECTED-FAIL | /tests/content/media/test/test_load_same_resource.html | Test timed out.

https://tbpl.mozilla.org/php/getParsedLog.php?id=34662682&tree=Try
https://tbpl.mozilla.org/php/getParsedLog.php?id=34666609&tree=Try
Keywords: intermittent-failure
(In reply to Honza Bambas (:mayhemer) from comment #0)
> TEST-UNEXPECTED-FAIL |
> /tests/content/media/test/test_load_same_resource.html | Test timed out.
> 
> https://tbpl.mozilla.org/php/getParsedLog.php?id=34662682&tree=Try
> https://tbpl.mozilla.org/php/getParsedLog.php?id=34666609&tree=Try

Hi Honza,
What does the "HTTP cache v2" in the bug title mean? Is this bug related to HTTP or network?
(In reply to JW Wang[:jwwang] from comment #6)
> (In reply to Honza Bambas (:mayhemer) from comment #0)
> > TEST-UNEXPECTED-FAIL |
> > /tests/content/media/test/test_load_same_resource.html | Test timed out.
> > 
> > https://tbpl.mozilla.org/php/getParsedLog.php?id=34662682&tree=Try
> > https://tbpl.mozilla.org/php/getParsedLog.php?id=34666609&tree=Try
> 
> Hi Honza,
> What does the "HTTP cache v2" in the bug title mean? Is this bug related to
> HTTP or network?

It means it happens only with the new HTTP cache v2 on.  Not with the default preference setting.
We (necko people) believe these test failures are caused by problems in the actual tests rather then in the new http cache, hence moving to the A/V component.
Component: Networking: Cache → Video/Audio
According comment 8 and 9 this is not reproducible only with the http cache v2.
No longer blocks: cache2tests
Summary: HTTP cache v2: Intermittent TEST-UNEXPECTED-FAIL | test_load_same_resource.html | Test timed out → Intermittent TEST-UNEXPECTED-FAIL | test_load_same_resource.html | Test timed out
Attached patch part1_fix_v1.patch (obsolete) — — Splinter Review 
There is only one h264 decoder on b2g emulator. We have to release the decoder of the original element so that the cloned one can start playing. Note that since cloning decoder is an async operation and the cloned elemnt shares media cache with the original, we can't reset 'src' until the decoder is cloned and media cache is completely buffered.

try: https://tbpl.mozilla.org/?tree=Try&rev=8756154da084
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Attachment #8398354 - Flags: review?(cajbir.bugzilla)
Attachment #8398354 - Flags: review?(cajbir.bugzilla) → review+
Summary: Intermittent TEST-UNEXPECTED-FAIL | test_load_same_resource.html | Test timed out → Intermittent test_load_same_resource.html | Test timed out.
Blocks: 1022453
The v1.patch is less than ideal. Since bug 995090 is fixed, we don't have to wait until the playback reaches the end to delete the decoder of the original element.
Attachment #8398354 - Attachment is obsolete: true
Attachment #8458406 - Flags: review?(cajbir.bugzilla)
Attachment #8458406 - Flags: review?(cajbir.bugzilla) → review+
Try on B2G: https://tbpl.mozilla.org/?tree=Try&rev=a8da8dd2506d
No failure in test_load_same_resource.html so far.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/248b8863ee1c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: