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

Side by Side Diff: ppapi/proxy/ppb_broker_proxy.cc

Issue 630883002: replace OVERRIDE and FINAL with override and final in ppapi/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months 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
« no previous file with comments | « ppapi/proxy/ppb_audio_proxy.cc ('k') | ppapi/proxy/ppb_buffer_proxy.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #include "ppapi/proxy/ppb_broker_proxy.h" 5 #include "ppapi/proxy/ppb_broker_proxy.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "ppapi/c/pp_errors.h" 8 #include "ppapi/c/pp_errors.h"
9 #include "ppapi/c/trusted/ppb_broker_trusted.h" 9 #include "ppapi/c/trusted/ppb_broker_trusted.h"
10 #include "ppapi/proxy/enter_proxy.h" 10 #include "ppapi/proxy/enter_proxy.h"
(...skipping 12 matching lines...) Expand all
23 23
24 namespace ppapi { 24 namespace ppapi {
25 namespace proxy { 25 namespace proxy {
26 26
27 class Broker : public PPB_Broker_API, public Resource { 27 class Broker : public PPB_Broker_API, public Resource {
28 public: 28 public:
29 explicit Broker(const HostResource& resource); 29 explicit Broker(const HostResource& resource);
30 virtual ~Broker(); 30 virtual ~Broker();
31 31
32 // Resource overrides. 32 // Resource overrides.
33 virtual PPB_Broker_API* AsPPB_Broker_API() OVERRIDE; 33 virtual PPB_Broker_API* AsPPB_Broker_API() override;
34 34
35 // PPB_Broker_API implementation. 35 // PPB_Broker_API implementation.
36 virtual int32_t Connect( 36 virtual int32_t Connect(
37 scoped_refptr<TrackedCallback> connect_callback) OVERRIDE; 37 scoped_refptr<TrackedCallback> connect_callback) override;
38 virtual int32_t GetHandle(int32_t* handle) OVERRIDE; 38 virtual int32_t GetHandle(int32_t* handle) override;
39 39
40 // Called by the proxy when the host side has completed the request. 40 // Called by the proxy when the host side has completed the request.
41 void ConnectComplete(IPC::PlatformFileForTransit socket_handle, 41 void ConnectComplete(IPC::PlatformFileForTransit socket_handle,
42 int32_t result); 42 int32_t result);
43 43
44 private: 44 private:
45 bool called_connect_; 45 bool called_connect_;
46 scoped_refptr<TrackedCallback> current_connect_callback_; 46 scoped_refptr<TrackedCallback> current_connect_callback_;
47 47
48 // The plugin module owns the handle. 48 // The plugin module owns the handle.
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
227 // The easiest way to clean it up is to just put it in an object 227 // The easiest way to clean it up is to just put it in an object
228 // and then close it. This failure case is not performance critical. 228 // and then close it. This failure case is not performance critical.
229 // The handle could still leak if Send succeeded but the IPC later failed. 229 // The handle could still leak if Send succeeded but the IPC later failed.
230 base::SyncSocket temp_socket( 230 base::SyncSocket temp_socket(
231 IPC::PlatformFileForTransitToPlatformFile(foreign_socket_handle)); 231 IPC::PlatformFileForTransitToPlatformFile(foreign_socket_handle));
232 } 232 }
233 } 233 }
234 234
235 } // namespace proxy 235 } // namespace proxy
236 } // namespace ppapi 236 } // namespace ppapi
OLDNEW
« no previous file with comments | « ppapi/proxy/ppb_audio_proxy.cc ('k') | ppapi/proxy/ppb_buffer_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698