Closed Bug 1118337 Opened 6 years ago Closed 4 years ago

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

Categories

(Core :: Audio/Video: Playback, defect, P5)

x86_64
Linux
defect

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox51 --- wontfix
firefox52 --- wontfix
firefox53 --- fixed

People

(Reporter: RyanVM, Assigned: ctai)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

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
Inactive; closing (see bug 1180138).
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Priority: -- → P5
Component: Audio/Video → Audio/Video: Playback
Assignee: nobody → ctai
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+
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)
Attachment #8804577 - Attachment is obsolete: true
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 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 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
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)
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
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
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
https://hg.mozilla.org/mozilla-central/rev/5e3fce33c8c5
Status: REOPENED → RESOLVED
Closed: 5 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.