Intermittent TEST-UNEXPECTED-PASS | /html/semantics/embedded-content/media-elements/loading-the-media-resource/resource-selection-invoke-insert-source.html | invoking resource selection by inserting <source> - expected FAIL

RESOLVED FIXED in Firefox 53

Status

()

defect
P5
normal
RESOLVED FIXED
4 years ago
2 years ago

People

(Reporter: RyanVM, Assigned: ctai)

Tracking

({intermittent-failure})

Trunk
mozilla53
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox51 wontfix, firefox52 wontfix, firefox53 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

4 years ago
02:51:10 INFO - TEST-START | /html/semantics/embedded-content/media-elements/loading-the-media-resource/resource-selection-invoke-insert-source.html
02:51:10 INFO - PROCESS | 6291 | WARNING: content window passed to PrivateBrowsingUtils.isWindowPrivate. Use isContentWindowPrivate instead (but only for frame scripts).
02:51:10 INFO - PROCESS | 6291 | pbu_isWindowPrivate@resource://gre/modules/PrivateBrowsingUtils.jsm:25:14
02:51:10 INFO - PROCESS | 6291 | nsBrowserAccess.prototype.openURI@chrome://browser/content/browser.js:13865:25
02:51:10 INFO - PROCESS | 6291 | __marionetteFunc@dummy file:17:30
02:51:10 INFO - PROCESS | 6291 | @dummy file:23:3
02:51:10 INFO - PROCESS | 6291 | executeWithCallback@chrome://marionette/content/marionette-listener.js:684:5
02:51:10 INFO - PROCESS | 6291 | executeAsyncScript@chrome://marionette/content/marionette-listener.js:577:3
02:51:10 INFO - TEST-UNEXPECTED-PASS | /html/semantics/embedded-content/media-elements/loading-the-media-resource/resource-selection-invoke-insert-source.html | invoking resource selection by inserting <source> - expected FAIL
02:51:10 INFO - TEST-INFO | expected FAIL
02:51:10 INFO - TEST-OK | /html/semantics/embedded-content/media-elements/loading-the-media-resource/resource-selection-invoke-insert-source.html | took 264ms
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Inactive; closing (see bug 1180138).
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WORKSFORME
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Priority: -- → P5
Component: Audio/Video → Audio/Video: Playback
Comment hidden (mozreview-request)
Assignee: nobody → ctai

Comment 11

3 years ago
mozreview-review
Comment on attachment 8804577 [details]
Bug 1118337 - Enable resource-selection-invoke-insert-source.html.

https://reviewboard.mozilla.org/r/88488/#review87562
Attachment #8804577 - Flags: review?(jwwang) → review+

Comment 12

3 years ago
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9f7616d57abc
Enable resource-selection-invoke-insert-source.html. r=jwwang
Keywords: checkin-needed
Backed out for failing the test:

https://hg.mozilla.org/integration/autoland/rev/4d56330a4173ecda8b6e662f15b18fc4a91515de

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=9f7616d57abc34000f57dd03c1cd779033054995
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=5832722&repo=autoland

[task 2016-10-28T12:01:03.519770Z] 12:01:03     INFO - TEST-UNEXPECTED-FAIL | /html/semantics/embedded-content/media-elements/loading-the-media-resource/resource-selection-invoke-insert-source.html | invoking resource selection by inserting <source> - assert_unreached: Reached unreachable code
[task 2016-10-28T12:01:03.520514Z] 12:01:03     INFO - window.onload<@http://web-platform.test:8000/html/semantics/embedded-content/media-elements/loading-the-media-resource/resource-selection-invoke-insert-source.html:12:44
[task 2016-10-28T12:01:03.521099Z] 12:01:03     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1401:20
Flags: needinfo?(ctai)
I will look into it.
Flags: needinfo?(ctai)
Comment hidden (mozreview-request)
Attachment #8804577 - Attachment is obsolete: true

Comment 16

3 years ago
mozreview-review
Comment on attachment 8817943 [details]
Bug 1118337 - Enable resource-selection-invoke-insert-source.html.

https://reviewboard.mozilla.org/r/98132/#review98524

I can't see how this test is fixed by bug 1118337. You just tried to push the backed-out changes again without doing anything.
Depends on: 1222980
With patches of bug 1222980 => pass.
  see: https://treeherder.mozilla.org/#/jobs?repo=try&revision=183dff34b672c7d98c3ab156ff3a61e2c777145d
Without patches of bug 1222980 => fail.
  see: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4072fa5c24d1b9bb3e2e0fc94bef0129d2894b19 


(In reply to JW Wang [:jwwang] [:jw_wang] from comment #16)
> Comment on attachment 8817943 [details]
> Bug 1118337 - Enable resource-selection-invoke-insert-source.html.
> 
> https://reviewboard.mozilla.org/r/98132/#review98524
> 
> I can't see how this test is fixed by bug 1118337. You just tried to push
> the backed-out changes again without doing anything.

Comment 19

3 years ago
mozreview-review-reply
Comment on attachment 8817943 [details]
Bug 1118337 - Enable resource-selection-invoke-insert-source.html.

https://reviewboard.mozilla.org/r/98132/#review98524

So you mean this test wil be fixed by bug "1222980" instead of "1118337"? If so, Please fix the commit message.
Comment hidden (mozreview-request)

Comment 21

3 years ago
mozreview-review
Comment on attachment 8817943 [details]
Bug 1118337 - Enable resource-selection-invoke-insert-source.html.

https://reviewboard.mozilla.org/r/98132/#review99056
Attachment #8817943 - Flags: review?(jwwang) → review+
Please land Bug 1222980 and Bug 1118337 together. Thanks.
Keywords: checkin-needed

Comment 23

3 years ago
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cdd6a27a2737
Enable resource-selection-invoke-insert-source.html. r=jwwang
Keywords: checkin-needed
had to back this out because bug  1222980  had to backed out too, sorry
Flags: needinfo?(ctai)

Comment 25

3 years ago
Backout by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/51be64cf2cf0
Backed out changeset cdd6a27a2737 for mda test failures on a CLOSED TREE
Comment hidden (Intermittent Failures Robot)
Comment hidden (mozreview-request)
Fixed. See try result: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d1a70d43f9350fe19d2cd58a452437acc9dd5e90

Please land Bug 1222980 and Bug 1118337 together. Thanks.(In reply to Pulsebot from comment #25)
> Backout by cbook@mozilla.com:
> https://hg.mozilla.org/integration/autoland/rev/51be64cf2cf0
> Backed out changeset cdd6a27a2737 for mda test failures on a CLOSED TREE

(In reply to Carsten Book [:Tomcat] from comment #24)
> had to back this out because bug  1222980  had to backed out too, sorry
Flags: needinfo?(ctai)
Keywords: checkin-needed

Comment 29

2 years ago
Pushed by ihsiao@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5e3fce33c8c5
Enable resource-selection-invoke-insert-source.html. r=jwwang
Keywords: checkin-needed

Comment 30

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/5e3fce33c8c5
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.