Summary: | [Cocoa] Videos sometimes don't render when loaded in the GPU process | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Carlson <eric.carlson> | ||||||||
Component: | Media | Assignee: | Eric Carlson <eric.carlson> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | ews-watchlist, glenn, jer.noble, peng.liu6, philipj, sergio, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=230500 https://bugs.webkit.org/show_bug.cgi?id=230471 https://bugs.webkit.org/show_bug.cgi?id=232125 |
||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 230500 | ||||||||||
Attachments: |
|
Description
Eric Carlson
2021-09-20 12:02:07 PDT
Created attachment 438717 [details]
Patch
Comment on attachment 438717 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=438717&action=review r=me with nits: > Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.cpp:244 > + ALWAYS_LOG(LOGIDENTIFIER); Maybe, `ALWAYS_LOG(LOGIDENTIFIER, visible)` ? > Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.cpp:356 > + ALWAYS_LOG(LOGIDENTIFIER); Maybe, `ALWAYS_LOG(LOGIDENTIFIER, m_player->readyState())` ? Comment on attachment 438717 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=438717&action=review >> Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.cpp:244 >> + ALWAYS_LOG(LOGIDENTIFIER); > > Maybe, `ALWAYS_LOG(LOGIDENTIFIER, visible)` ? Good idea. >> Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.cpp:356 >> + ALWAYS_LOG(LOGIDENTIFIER); > > Maybe, `ALWAYS_LOG(LOGIDENTIFIER, m_player->readyState())` ? Ditto Created attachment 438718 [details]
Patch for landing
Created attachment 438721 [details]
Patch for landing
Committed r282789 (241924@main): <https://commits.webkit.org/241924@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 438721 [details]. |