Closed
Bug 1145585
Opened 9 years ago
Closed 9 years ago
[Vista] glitches in live streams caused by lazily-open-shared-handle
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla39
People
(Reporter: bmaris, Assigned: mattwoodrow)
References
Details
(Keywords: regression)
Attachments
(2 files)
4.25 MB,
image/gif
|
Details | |
1.22 KB,
patch
|
jrmuizel
:
review+
lmandel
:
approval-mozilla-aurora+
lmandel
:
approval-mozilla-beta+
lmandel
:
approval-mozilla-release-
|
Details | Diff | Splinter Review |
Affected OS`s: - Windows Vista only Affected Builds: - Firefox 37beta7 - Latest Nightly STR: 1. Start Firefox 2. Open a live stream (I reproduced on http://www.twitch.tv/ and http://www.douyutv.com/) Here is a list of streams: http://www.joindota.com/en/livestreams/ or https://new.livestream.com/watch Expected results: Stream runs without glitches Actual results: Lots of glitches can be seen during the stream Notes: 1. This is a regression. I was unable to get central or inbound regression but I did manage to get a range through tinderbox beta builds: First good - https://hg.mozilla.org/releases/mozilla-beta/rev/89cfa8ff9fc5 First bad - https://hg.mozilla.org/releases/mozilla-beta/rev/7d4016a05dd3 Pushlog: https://hg.mozilla.org/releases/mozilla-beta/pushloghtml?fromchange=89cfa8ff9fc5&tochange=7d4016a05dd3
Reporter | ||
Comment 1•9 years ago
|
||
Graphics Adapter Description AMD Radeon HD 6450 Adapter Drivers aticfx64 aticfx64 aticfx64 aticfx32 aticfx32 aticfx32 atiumd64 atidxx64 atidxx64 atiumdag atidxx32 atidxx32 atiumdva atiumd6a atitmm64 Adapter RAM 1024 Device ID 0x6779 Direct2D Enabled true DirectWrite Enabled true (7.0.6002.23200) Driver Date 12-6-2013 Driver Version 13.251.0.0 GPU #2 Active false GPU Accelerated Windows 1/1 Direct3D 11 (OMTC) Subsys ID 21251462 Vendor ID 0x1002 WebGL Renderer Google Inc. -- ANGLE (AMD Radeon HD 6450 Direct3D9Ex vs_3_0 ps_3_0) windowLayerManagerRemote true AzureCanvasBackend direct2d AzureContentBackend direct2d AzureFallbackCanvasBackend cairo AzureSkiaAccelerated 0
Updated•9 years ago
|
Keywords: regression
Comment 2•9 years ago
|
||
[Tracking Requested - why for this release]: Issue is Vista only, but is a regression and bad user experience on some popular websites.
status-firefox37:
--- → affected
status-firefox39:
--- → affected
tracking-firefox37:
--- → ?
tracking-firefox38:
--- → ?
tracking-firefox39:
--- → ?
Comment 3•9 years ago
|
||
I think that Video/Audio is a better place to start but this may also be a gfx issue. This is not YouTube so it's not MSE.
Component: Layout → Video/Audio
Comment 4•9 years ago
|
||
So I don't lose the comment, on irc Jeff said this was probably introduced by bug 1128170 or bug 1127925.
Comment 5•9 years ago
|
||
Can you try the build here to see if it reproduces? That should narrow things down between between those bugs: https://ftp-ssl.mozilla.org/pub/mozilla.org/firefox/try-builds/jmuizelaar@mozilla.com-666418bb7676
Reporter | ||
Comment 7•9 years ago
|
||
(In reply to Jeff Muizelaar [:jrmuizel] from comment #5) > Can you try the build here to see if it reproduces? That should narrow > things down between between those bugs: > https://ftp-ssl.mozilla.org/pub/mozilla.org/firefox/try-builds/ > jmuizelaar@mozilla.com-666418bb7676 This build is also affected by glitches on my system.
Flags: needinfo?(jmuizelaar)
Comment 8•9 years ago
|
||
Based on comment 7 this is caused by bug 1127925. ni? to mattwoodrow as a heads-up. Not sure if he has cycles to look into this right now though.
Blocks: 1127925
Flags: needinfo?(matt.woodrow)
Updated•9 years ago
|
Flags: needinfo?(jmuizelaar)
Summary: [Vista] glitches in live streams → [Vista] glitches in live streams caused by lazily-open-shared-handle
Comment 9•9 years ago
|
||
Bogdan, are the streams using flash or the video tag?
Flags: needinfo?(bogdan.maris)
Reporter | ||
Comment 10•9 years ago
|
||
(In reply to Jeff Muizelaar [:jrmuizel] from comment #9) > Bogdan, are the streams using flash or the video tag? Yes
Flags: needinfo?(bogdan.maris)
Reporter | ||
Comment 12•9 years ago
|
||
(In reply to Jeff Muizelaar [:jrmuizel] from comment #11) > Which one? http://www.twitch.tv/, http://www.douyutv.com/ and https://new.livestream.com/watch - require Flash. I did not encountered any live stream that did not requested Flash in order to work.
Flags: needinfo?(bogdan.maris)
Assignee | ||
Comment 13•9 years ago
|
||
Assignee: nobody → matt.woodrow
Flags: needinfo?(matt.woodrow)
Attachment #8582647 -
Flags: review?(jmuizelaar)
Assignee | ||
Comment 14•9 years ago
|
||
Comment on attachment 8582647 [details] [diff] [review] Hold a ref to the right texture Approval Request Comment [Feature/regressing bug #]: bug 1046550 and bug 1127925 [User impact if declined]: Video flickering, seems to happen mainly on vista (but should be able to happen on 7/8 too). [Describe test coverage new/current, TreeHerder]: Nothing yet. [Risks and why]: Very low risk, obvious typo. [String/UUID change made/needed]: None. This was initially broken by bug 1046550, though it's racy and apparently very rare. Bug 1127925 increased the window where it could happen which made it show up more.
Attachment #8582647 -
Flags: approval-mozilla-release?
Attachment #8582647 -
Flags: approval-mozilla-beta?
Attachment #8582647 -
Flags: approval-mozilla-aurora?
Comment 15•9 years ago
|
||
Comment on attachment 8582647 [details] [diff] [review] Hold a ref to the right texture Great simple fix for a very visible bug. Release- as we're not going to take this fix in 36. Beta+ Aurora+
Attachment #8582647 -
Flags: approval-mozilla-release?
Attachment #8582647 -
Flags: approval-mozilla-release-
Attachment #8582647 -
Flags: approval-mozilla-beta?
Attachment #8582647 -
Flags: approval-mozilla-beta+
Attachment #8582647 -
Flags: approval-mozilla-aurora?
Attachment #8582647 -
Flags: approval-mozilla-aurora+
Updated•9 years ago
|
Attachment #8582647 -
Flags: review?(jmuizelaar) → review+
Updated•9 years ago
|
Flags: qe-verify+
Reporter | ||
Comment 19•9 years ago
|
||
I retested on Firefox 37.0 RC and latest Aurora 38.0a2 under Windows Vista 64-bit and the glitches are gone.
Comment 20•9 years ago
|
||
I see this patch only landed on release, aurora, and beta but not central. Are there plans to land this on master? Thanks!
Flags: needinfo?(jmuizelaar)
Assignee | ||
Comment 21•9 years ago
|
||
Yes, but inbound seems to be closed every time I try.
Flags: needinfo?(jmuizelaar)
Comment 22•9 years ago
|
||
Set checkin-needed?
Assignee | ||
Comment 23•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4f8721033f5b
Comment 24•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4f8721033f5b
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Updated•9 years ago
|
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•