| Summary: | [Media in GPU Process][MSE] The GPU Process may crash in the destructor of MediaPlayerPrivateMediaSourceAVFObjC | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Peng Liu <peng.liu6> | ||||||||||
| Component: | Media | Assignee: | Peng Liu <peng.liu6> | ||||||||||
| Status: | RESOLVED FIXED | ||||||||||||
| Severity: | Normal | CC: | eric.carlson, ews-watchlist, glenn, jer.noble, philipj, sergio, webkit-bug-importer, youennf | ||||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||||
| Version: | WebKit Nightly Build | ||||||||||||
| Hardware: | Unspecified | ||||||||||||
| OS: | Unspecified | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Peng Liu
2020-12-07 17:47:17 PST
Created attachment 415604 [details]
Patch
Comment on attachment 415604 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=415604&action=review > Source/WebKit/GPUProcess/media/cocoa/RemoteMediaPlayerProxyCocoa.mm:-60 > - m_inlineLayerHostingContext->setRootLayer(m_player->platformLayer()); It is not clear why removing this line is fine. Is it redundant? If so, can you add this info in change log? Comment on attachment 415604 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=415604&action=review >> Source/WebKit/GPUProcess/media/cocoa/RemoteMediaPlayerProxyCocoa.mm:-60 >> - m_inlineLayerHostingContext->setRootLayer(m_player->platformLayer()); > > It is not clear why removing this line is fine. > Is it redundant? If so, can you add this info in change log? The purpose of this callback is to notify the DOM that the player's rendering mode is changed, including the case that the layer is destroyed. So I think it was a bug to operate on the layer. I just realized that we need to forward the callback to the Web process (HTMLMediaElement). I will upload a new patch. Created attachment 415651 [details]
Patch
Created attachment 415653 [details]
Fix GTK/WPE build failures
Created attachment 415657 [details]
Fix a build failure on the wincairo port
Committed r270563: <https://trac.webkit.org/changeset/270563> All reviewed patches have been landed. Closing bug and clearing flags on attachment 415657 [details]. |