Remove unnecessary RemoteCDMInstanceSessionIdentifier argument from RemoteCDMFactory::addSession(). I noticed this when fixing Bug 209712.
Created attachment 395452 [details] Patch v1
Comment on attachment 395452 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=395452&action=review > Source/WebKit/WebProcess/GPU/media/RemoteCDMFactory.cpp:84 > + m_sessions.set(session->identifier(), WTFMove(session)); This also means that we could use a HashSet instead of a HashMap.
Comment on attachment 395452 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=395452&action=review >> Source/WebKit/WebProcess/GPU/media/RemoteCDMFactory.cpp:84 >> + m_sessions.set(session->identifier(), WTFMove(session)); > > This also means that we could use a HashSet instead of a HashMap. That’s a slightly more challenging refactoring, but it’s one that would save memory.
Committed r259539: <https://trac.webkit.org/changeset/259539> All reviewed patches have been landed. Closing bug and clearing flags on attachment 395452 [details].
<rdar://problem/61304706>