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

Side by Side Diff: ppapi/proxy/proxy_channel.h

Issue 2476883002: Remove obsolete methods from IPC::Channel and related classes. (Closed)
Patch Set: Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef PPAPI_PROXY_PROXY_CHANNEL_H_ 5 #ifndef PPAPI_PROXY_PROXY_CHANNEL_H_
6 #define PPAPI_PROXY_PROXY_CHANNEL_H_ 6 #define PPAPI_PROXY_PROXY_CHANNEL_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/files/scoped_file.h" 10 #include "base/files/scoped_file.h"
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 bool Send(IPC::Message* msg) override; 95 bool Send(IPC::Message* msg) override;
96 96
97 // IPC::Listener implementation. 97 // IPC::Listener implementation.
98 void OnChannelError() override; 98 void OnChannelError() override;
99 99
100 // Will be NULL in some unit tests and if the remote side has crashed. 100 // Will be NULL in some unit tests and if the remote side has crashed.
101 IPC::SyncChannel* channel() const { 101 IPC::SyncChannel* channel() const {
102 return channel_.get(); 102 return channel_.get();
103 } 103 }
104 104
105 #if defined(OS_POSIX) && !defined(OS_NACL)
106 base::ScopedFD TakeRendererFD();
107 #endif
108
109 protected: 105 protected:
110 explicit ProxyChannel(); 106 explicit ProxyChannel();
111 107
112 // You must call this function before anything else. Returns true on success. 108 // You must call this function before anything else. Returns true on success.
113 // The delegate pointer must outlive this class, ownership is not 109 // The delegate pointer must outlive this class, ownership is not
114 // transferred. 110 // transferred.
115 virtual bool InitWithChannel(Delegate* delegate, 111 virtual bool InitWithChannel(Delegate* delegate,
116 base::ProcessId peer_pid, 112 base::ProcessId peer_pid,
117 const IPC::ChannelHandle& channel_handle, 113 const IPC::ChannelHandle& channel_handle,
118 bool is_client); 114 bool is_client);
(...skipping 20 matching lines...) Expand all
139 // remote side has crashed. 135 // remote side has crashed.
140 std::unique_ptr<IPC::SyncChannel> channel_; 136 std::unique_ptr<IPC::SyncChannel> channel_;
141 137
142 DISALLOW_COPY_AND_ASSIGN(ProxyChannel); 138 DISALLOW_COPY_AND_ASSIGN(ProxyChannel);
143 }; 139 };
144 140
145 } // namespace proxy 141 } // namespace proxy
146 } // namespace ppapi 142 } // namespace ppapi
147 143
148 #endif // PPAPI_PROXY_PROXY_CHANNEL_H_ 144 #endif // PPAPI_PROXY_PROXY_CHANNEL_H_
OLDNEW
« ipc/ipc_channel.h ('K') | « ipc/ipc_test_sink.cc ('k') | ppapi/proxy/proxy_channel.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698