Add a “timestamp” field to struct RemoteMediaPlayerState
Created attachment 423755 [details] Patch
Created attachment 423756 [details] Patch
Created attachment 423774 [details] Fix test failures and update TestExpectations
Comment on attachment 423774 [details] Fix test failures and update TestExpectations View in context: https://bugs.webkit.org/attachment.cgi?id=423774&action=review > Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.cpp:814 > + m_cachedState.timestamp = WallTime::now(); Nit: I think 'wallTime' might be more clear than 'timestamp'
Comment on attachment 423774 [details] Fix test failures and update TestExpectations View in context: https://bugs.webkit.org/attachment.cgi?id=423774&action=review >> Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.cpp:814 >> + m_cachedState.timestamp = WallTime::now(); > > Nit: I think 'wallTime' might be more clear than 'timestamp' Good idea! Will fix it after getting results from bots.
Created attachment 423805 [details] Patch for landing Revise the patch based on Eric's comment and fix a layout test failure in the previous version.
Created attachment 423806 [details] Patch for landing - v2
Committed r274816: <https://commits.webkit.org/r274816> All reviewed patches have been landed. Closing bug and clearing flags on attachment 423806 [details].
<rdar://problem/75714239>