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

Side by Side Diff: sandbox/win/src/broker_services.cc

Issue 1210013007: clang/win: Fix warnings to prepare for building without -Wno-reorder. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@clang-unsequenced
Patch Set: Created 5 years, 5 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
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 "sandbox/win/src/broker_services.h" 5 #include "sandbox/win/src/broker_services.h"
6 6
7 #include <AclAPI.h> 7 #include <AclAPI.h>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
118 key |= policy->GetIntegrityLevel(); 118 key |= policy->GetIntegrityLevel();
119 119
120 return key; 120 return key;
121 } 121 }
122 122
123 } // namespace 123 } // namespace
124 124
125 namespace sandbox { 125 namespace sandbox {
126 126
127 BrokerServicesBase::BrokerServicesBase() 127 BrokerServicesBase::BrokerServicesBase()
128 : thread_pool_(NULL), job_port_(NULL), no_targets_(NULL), 128 : job_port_(NULL),
129 job_thread_(NULL) { 129 no_targets_(NULL),
130 job_thread_(NULL),
131 thread_pool_(NULL) {
130 } 132 }
131 133
132 // The broker uses a dedicated worker thread that services the job completion 134 // The broker uses a dedicated worker thread that services the job completion
133 // port to perform policy notifications and associated cleanup tasks. 135 // port to perform policy notifications and associated cleanup tasks.
134 ResultCode BrokerServicesBase::Init() { 136 ResultCode BrokerServicesBase::Init() {
135 if ((NULL != job_port_) || (NULL != thread_pool_)) 137 if ((NULL != job_port_) || (NULL != thread_pool_))
136 return SBOX_ERROR_UNEXPECTED_CALL; 138 return SBOX_ERROR_UNEXPECTED_CALL;
137 139
138 ::InitializeCriticalSection(&lock_); 140 ::InitializeCriticalSection(&lock_);
139 141
(...skipping 474 matching lines...) Expand 10 before | Expand all | Expand 10 after
614 return SBOX_ERROR_UNSUPPORTED; 616 return SBOX_ERROR_UNSUPPORTED;
615 617
616 base::string16 name = LookupAppContainer(sid); 618 base::string16 name = LookupAppContainer(sid);
617 if (name.empty()) 619 if (name.empty())
618 return SBOX_ERROR_INVALID_APP_CONTAINER; 620 return SBOX_ERROR_INVALID_APP_CONTAINER;
619 621
620 return DeleteAppContainer(sid); 622 return DeleteAppContainer(sid);
621 } 623 }
622 624
623 } // namespace sandbox 625 } // namespace sandbox
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698