<a class="header-button" href="https://bugzilla-dev.allizom.org/home" title="Go to home page"> Bugzilla

Comment 4

•

10 years ago

I'm not an expert in the webplatform tests, but IIUC from bug 1066467, these tests were supposed to be disabled for being unstable. http://mxr.mozilla.org/mozilla-central/source/testing/web-platform/meta/media-source/mediasource-duration.html.ini I'm wondering if the recent upstream update somehow re-enabled them?

Flags: needinfo?(ryanvm) → needinfo?(james)

James Graham [:jgraham]

Comment 5

•

10 years ago

So consistently getting timeouts isn't an unexpected result. Hence it doesn't mark the run orange. You only get orange if you got a timeout but expected something else (e.g. PASS). I think the recent update actually re-disabled tests that were supposed to be enabled (bug 1087165).

Flags: needinfo?(james)

JW Wang [:jwwang] [:jw_wang]

Assignee

Comment 6

•

10 years ago

This test was enabled, about 4 days before the first report in comment 0, in https://hg.mozilla.org/integration/mozilla-inbound/rev/2839cfaaa883 I don't know of any changes that may have made this unstable, but we can disable the particular subtest.

Assignee: nobody → karlt

Blocks: 1066467

Comment hidden (Legacy TBPL/Treeherder Robot)

submit_timestamp: 2014-11-01T09:49:11 log: https://treeherder.mozilla.org/ui/logviewer.html#?repo=fx-team&job_id=1066514 repository: fx-team who: philringnalda[at]gmail[dot]com machine: tst-linux64-spot-896 buildname: Ubuntu VM 12.04 x64 fx-team pgo test web-platform-tests-4 revision: c8eefd3e3ac3 TEST-UNEXPECTED-FAIL | /media-source/mediasource-duration.html | Test seek starts on duration truncation below currentTime - assert_equals: Event types match. expected "seeking" but got "timeupdate" TEST-UNEXPECTED-FAIL | /media-source/mediasource-duration.html | Test endOfStream completes previous seek to truncated duration - assert_equals: Event types match. expected "seeking" but got "timeupdate" TEST-UNEXPECTED-TIMEOUT | /media-source/mediasource-duration.html | Test setting same duration multiple times does not fire duplicate durationchange - Test timed out TEST-UNEXPECTED-OK | /media-source/mediasource-duration.html | expected TIMEOUT TEST-UNEXPECTED-TIMEOUT | /quirks-mode/hashless-hex-color.html | expected OK TEST-UNEXPECTED-FAIL | /resource-timing/test_resource_timing.html | PerformanceEntry has correct name, initiatorType, startTime, and duration (iframe) - assert_approx_equals: expected 1425.576489 +/- 20 but got 1524.724894 TEST-UNEXPECTED-FAIL | /resource-timing/test_resource_timing.html | window.performance.getEntriesByName() and window.performance.getEntriesByNameType() return same data (link) - assert_equals: should have a single entry for each resource (without type) expected 1 but got 0 TEST-UNEXPECTED-FAIL | /resource-timing/test_resource_timing.html | PerformanceEntry has correct name, initiatorType, startTime, and duration (link) - actual is undefined(stack: resource_load/<@http://web-platform.test:8000/resource-timing/test_resource_timing.js:151:9 TEST-UNEXPECTED-FAIL | /resource-timing/test_resource_timing.html | PerformanceEntry has correct order of timing attributes (link) - actual is undefined(stack: test@http://web-platform.test:8000/resource-timing/test_resource_timing.js:161:9 TEST-UNEXPECTED-FAIL | /resource-timing/test_resource_timing.html | PerformanceEntry has correct name, initiatorType, startTime, and duration (xmlhttprequest) - assert_approx_equals: expected 1530.447955 +/- 20 but got 1573.55237 Return code: 1

Comment hidden (Legacy TBPL/Treeherder Robot)

Comment 12

•

10 years ago

Depends on bug 1088472 comment 1.

Depends on: 1088472

Comment hidden (Legacy TBPL/Treeherder Robot)

Assignee

Updated

•

10 years ago

OS: Linux → All

Hardware: x86 → All

Comment hidden (Legacy TBPL/Treeherder Robot)

https://hg.mozilla.org/integration/mozilla-inbound/rev/849e49ab0a01

Assignee

Comment 24

•

10 years ago

Keywords: leave-open

Whiteboard: [disabled]

Comment hidden (Legacy TBPL/Treeherder Robot)

Carsten Book [:Tomcat]

Reporter

Comment 26

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/849e49ab0a01

Comment hidden (Legacy TBPL/Treeherder Robot)

Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)

Updated

•

10 years ago

Priority: -- → P5

https://hg.mozilla.org/integration/mozilla-inbound/rev/822005621954

Assignee

Comment 29

•

10 years ago

Flags: in-testsuite+

Keywords: leave-open

Whiteboard: [disabled]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e4012e4f4570

Assignee

Comment 30

•

10 years ago

Carsten Book [:Tomcat]

Reporter

Comment 31

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/822005621954

Status: NEW → RESOLVED

Closed: 10 years ago

Resolution: --- → FIXED

Target Milestone: --- → mozilla38

Comment 32

•

10 years ago

Do we need to enable these on aurora/beta too?

status-firefox36: --- → affected

status-firefox37: --- → affected

status-firefox38: --- → fixed

status-firefox-esr31: --- → unaffected

Flags: needinfo?(karlt)

Ralph Giles (:rillian)

Comment 33

•

10 years ago

Attached patch Enable remaining mediasoruce-duratin subtests (deleted) — Details — Splinter Review

Yes, we would like this fix uplifted. Nominating patch from comment #31. Approval Request Comment [Feature/regressing bug #]: MSE [User impact if declined]: Less consistent testing (including intermittant failures) [Describe test coverage new/current, TreeHerder]: Landed on m-c. [Risks and why]: Test only, so the risk is just more orange on treeherder. [String/UUID change made/needed]: None.

Attachment #8552582 - Flags: approval-mozilla-beta?

Attachment #8552582 - Flags: approval-mozilla-aurora?

Comment 34

•

10 years ago

Comment on attachment 8552582 [details] [diff] [review] Enable remaining mediasoruce-duratin subtests Test-only changes don't need approval.

Attachment #8552582 - Flags: approval-mozilla-beta?

Attachment #8552582 - Flags: approval-mozilla-aurora?

Assignee

Comment 35

•

10 years ago

(In reply to Ryan VanderMeulen [:RyanVM UTC-5] from comment #32) > Do we need to enable these on aurora/beta too? I would wait a couple of days to check that these really are no longer intermittent. I don't know exactly what would have changed behavior, and it is hard to verify on try because the failures were not frequent.

Flags: needinfo?(karlt)

Assignee

Comment 36

•

10 years ago

I see these were just re-enabled on aurora. That's fine too.

Comment 37

•

10 years ago

Fun times, I'd already pushed them by the time I saw that comment. Oh well, we'll see I guess. Also, they're fully disabled on beta, so we'll need a different patch there. https://hg.mozilla.org/releases/mozilla-aurora/rev/1711d9b3f9f4

status-firefox37: affected → fixed

Flags: needinfo?(karlt)

Keywords: branch-patch-needed

Assignee

Comment 38

•

10 years ago

(In reply to Ryan VanderMeulen [:RyanVM UTC-5] from comment #37) > Also, they're fully disabled on beta, so we'll need a different patch there. See bug 1098126. The patch here should apply fine on top of that.

Flags: needinfo?(karlt)

Ralph Giles (:rillian)

Comment 39

•

10 years ago

https://hg.mozilla.org/releases/mozilla-beta/rev/d918f7ea93fe I merged it manually to make it look like central; hopefully that will work.

status-firefox36: affected → fixed

[SV Manager] Florin Mezei, QA (:FlorinMezei)

Updated

•

10 years ago

Keywords: branch-patch-needed

Updated

•

10 years ago

Flags: qe-verify-

Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)

Comment 40

•

10 years ago

I had to back this out in https://hg.mozilla.org/releases/mozilla-beta/rev/a3202b0fe6a7 on beta for wpt-4 bustage: https://treeherder.mozilla.org/logviewer.html#?job_id=235734&repo=mozilla-beta

Status: RESOLVED → REOPENED

status-firefox36: fixed → affected

Flags: needinfo?(karlt)

Resolution: FIXED → ---

Updated

•

10 years ago

Depends on: 1125776

Comment hidden (Legacy TBPL/Treeherder Robot)

Assignee

Comment 47

•

10 years ago

(In reply to Wes Kocher (:KWierso) from comment #40) > I had to back this out in > https://hg.mozilla.org/releases/mozilla-beta/rev/a3202b0fe6a7 on beta for > wpt-4 bustage: > > https://treeherder.mozilla.org/logviewer.html#?job_id=235734&repo=mozilla- > beta That looks similar to changes in results from from bug 1096089 comment 25 and bug 1116007. We can leave things disabled on beta 36. jya has changes in bug 1125776 to address recent m-c intermittents, which seem different from those originally filed here.

Assignee

Updated

•

10 years ago

Flags: needinfo?(karlt)

Comment hidden (Legacy TBPL/Treeherder Robot)

Comment 55

•

10 years ago

This will have been fixed by bug 1125776, bug 1130826 and bug 1130839

Updated

•

10 years ago

Status: REOPENED → RESOLVED

Closed: 10 years ago → 10 years ago

Resolution: --- → FIXED

Updated

•

10 years ago

status-firefox36: affected → wontfix