1284074 - Intermittent dom/media/tests/mochitest/test_peerConnection_removeThenAddVideoTrackNoBundle.html | application crashed [@ mozilla::layers::TextureClient::SetLastFwdTransactionId(unsigned __int64)]

Status: RESOLVED FIXED

(Core :: WebRTC, defect, P3)

Description

[Treeherder Bug Filer]

Filed by: [e-mail]

https://treeherder.mozilla.org/logviewer.html#?job_id=10255456&repo=fx-team

http://archive.mozilla.org/pub/firefox/tinderbox-builds/fx-team-win32-debug/1467558502/fx-team_win7_vm-debug_test-mochitest-media-bm137-tests1-windows-build80.txt.gz

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Any graphics changes around a week ago that might explain this and the various dupes?

Comment 2

[Sotaro Ikeda [:sotaro]]

> Assertion failure: mFwdTransactionId < aTransactionId

The above assertion seems like a false alarm. The assertion needs to be changed. It seems like a potential problem, since Bug 1252835. Bug 1280839 might affect to the ImageBridge's task timing change.

Comment 3

[Sotaro Ikeda [:sotaro]]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #1)
> Any graphics changes around a week ago that might explain this and the
> various dupes?

various dupes also hit the assertion. It seems to be caused by the same cause.

Comment 4

[Sotaro Ikeda [:sotaro]]

Attachment: patch - Relax assertion check

Relax the assertion, since there could be a case that same TextureClient is handled multiple times in one transaction.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bfdd3618b368
Relax assertion check r=nical

Comment 6

[Carsten Book [:Tomcat]]

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