| commit | 540d942f9d32f08c2dafd5362f4d6d3babe420df | [log] [tgz] |
|---|---|---|
| author | Alex Deymo <deymo@chromium.org> | Thu Feb 27 11:17:31 2014 -0800 |
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | Fri Feb 28 06:44:34 2014 +0000 |
| tree | 6e2972ec73bf8e7286c1a40c5682b69341eebc76 | |
| parent | 2de23f54423b83ac0cd100282f5952c8bae17664 [diff] |
PolicyManager: Mark protected members as private. The Google C++ style guide requires members to be private having a getter and setter with fixed names. This patch moves the protected members to the private section on the base class for each provider and adds a protected setter to allow the sub-classes to set these members on Init(). Also, this patch uniforms the variable names prefixing them with var_. BUG=None TEST=Unit tests still passes. Change-Id: I4cbf7b7031b430991c12a5c7d9729bafaf359095 Reviewed-on: https://chromium-review.googlesource.com/187979 Reviewed-by: Gilad Arnold <garnold@chromium.org> Tested-by: Alex Deymo <deymo@chromium.org> Commit-Queue: Alex Deymo <deymo@chromium.org>