OLD | NEW |
---|---|
(Empty) | |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | |
bartfab (slow)
2014/03/13 13:20:00
Nit 1: s/2012/2014/
Nit 2: There are two variants
| |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef COMPONENTS_POLICY_CORE_BROWSER_CONFIGURATION_POLICY_HANDLER_PARAMETERS_H _ | |
6 #define COMPONENTS_POLICY_CORE_BROWSER_CONFIGURATION_POLICY_HANDLER_PARAMETERS_H _ | |
7 | |
8 namespace policy { | |
9 | |
bartfab (slow)
2014/03/13 13:20:00
Nit: Document what this struct is for.
| |
10 struct POLICY_EXPORT PolicyHandlerParameters { | |
bartfab (slow)
2014/03/13 13:20:00
Nit: #include "components/policy/policy_export.h"
| |
11 std::string active_username_hash; | |
bartfab (slow)
2014/03/13 13:20:00
1: Remove the "active_" prefix. When the Configura
| |
12 }; | |
13 | |
14 } // namespace policy | |
15 | |
16 #endif // COMPONENTS_POLICY_CORE_BROWSER_CONFIGURATION_POLICY_HANDLER_PARAMETER S_H_ | |
OLD | NEW |