Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1)

Unified Diff: remoting/host/desktop_session_proxy.cc

Issue 1509723003: Remove dependency on webrtc::SharedMemory::handle() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove_quic
Patch Set: Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/host/desktop_session_agent.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/desktop_session_proxy.cc
diff --git a/remoting/host/desktop_session_proxy.cc b/remoting/host/desktop_session_proxy.cc
index 4536500890870bdb57fada47d7f3b3b970fc5a02..6166d20aa5af2a0103411ab7ba2d89a85fa8a036 100644
--- a/remoting/host/desktop_session_proxy.cc
+++ b/remoting/host/desktop_session_proxy.cc
@@ -67,14 +67,6 @@ class DesktopSessionProxy::IpcSharedBufferCore
int id() { return id_; }
size_t size() { return size_; }
void* memory() { return shared_memory_.memory(); }
- webrtc::SharedMemory::Handle handle() {
-#if defined(OS_WIN)
- return shared_memory_.handle().GetHandle();
-#else
- return base::SharedMemory::GetFdFromSharedMemoryHandle(
- shared_memory_.handle());
-#endif
- }
private:
virtual ~IpcSharedBufferCore() {}
@@ -90,10 +82,8 @@ class DesktopSessionProxy::IpcSharedBufferCore
class DesktopSessionProxy::IpcSharedBuffer : public webrtc::SharedMemory {
public:
IpcSharedBuffer(scoped_refptr<IpcSharedBufferCore> core)
- : SharedMemory(core->memory(), core->size(),
- core->handle(), core->id()),
- core_(core) {
- }
+ : SharedMemory(core->memory(), core->size(), 0, core->id()),
+ core_(core) {}
private:
scoped_refptr<IpcSharedBufferCore> core_;
« no previous file with comments | « remoting/host/desktop_session_agent.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698