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

Side by Side Diff: ipc/file_descriptor_set_posix.h

Issue 649603004: Non-SFI NaCl: Batch-open resource files (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address review comments Created 6 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 IPC_FILE_DESCRIPTOR_SET_POSIX_H_ 5 #ifndef IPC_FILE_DESCRIPTOR_SET_POSIX_H_
6 #define IPC_FILE_DESCRIPTOR_SET_POSIX_H_ 6 #define IPC_FILE_DESCRIPTOR_SET_POSIX_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 13 matching lines...) Expand all
24 FileDescriptorSet(); 24 FileDescriptorSet();
25 25
26 // This is the maximum number of descriptors per message. We need to know this 26 // This is the maximum number of descriptors per message. We need to know this
27 // because the control message kernel interface has to be given a buffer which 27 // because the control message kernel interface has to be given a buffer which
28 // is large enough to store all the descriptor numbers. Otherwise the kernel 28 // is large enough to store all the descriptor numbers. Otherwise the kernel
29 // tells us that it truncated the control data and the extra descriptors are 29 // tells us that it truncated the control data and the extra descriptors are
30 // lost. 30 // lost.
31 // 31 //
32 // In debugging mode, it's a fatal error to try and add more than this number 32 // In debugging mode, it's a fatal error to try and add more than this number
33 // of descriptors to a FileDescriptorSet. 33 // of descriptors to a FileDescriptorSet.
34 static const size_t kMaxDescriptorsPerMessage = 7; 34 static const size_t kMaxDescriptorsPerMessage = 128;
35 35
36 // --------------------------------------------------------------------------- 36 // ---------------------------------------------------------------------------
37 // Interfaces for building during message serialisation... 37 // Interfaces for building during message serialisation...
38 38
39 // Add a descriptor to the end of the set. Returns false iff the set is full. 39 // Add a descriptor to the end of the set. Returns false iff the set is full.
40 bool AddToBorrow(base::PlatformFile fd); 40 bool AddToBorrow(base::PlatformFile fd);
41 // Add a descriptor to the end of the set and automatically close it after 41 // Add a descriptor to the end of the set and automatically close it after
42 // transmission. Returns false iff the set is full. 42 // transmission. Returns false iff the set is full.
43 bool AddToOwn(base::ScopedFD fd); 43 bool AddToOwn(base::ScopedFD fd);
44 44
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
110 // This contains the index of the next descriptor which should be consumed. 110 // This contains the index of the next descriptor which should be consumed.
111 // It's used in a couple of ways. Firstly, at destruction we can check that 111 // It's used in a couple of ways. Firstly, at destruction we can check that
112 // all the descriptors have been read (with GetNthDescriptor). Secondly, we 112 // all the descriptors have been read (with GetNthDescriptor). Secondly, we
113 // can check that they are read in order. 113 // can check that they are read in order.
114 mutable unsigned consumed_descriptor_highwater_; 114 mutable unsigned consumed_descriptor_highwater_;
115 115
116 DISALLOW_COPY_AND_ASSIGN(FileDescriptorSet); 116 DISALLOW_COPY_AND_ASSIGN(FileDescriptorSet);
117 }; 117 };
118 118
119 #endif // IPC_FILE_DESCRIPTOR_SET_POSIX_H_ 119 #endif // IPC_FILE_DESCRIPTOR_SET_POSIX_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698