| commit | 231a851550786d6054a2935d52b59278d4305427 | [log] [tgz] |
|---|---|---|
| author | Alex Deymo <[email protected]> | Fri Mar 21 12:56:10 2014 -0700 |
| committer | chrome-internal-fetch <[email protected]> | Sat Mar 22 01:44:25 2014 +0000 |
| tree | 54fdf8bd2315ab2ac32a30c4f25e963103efed3f | |
| parent | 53556eccd206bacd5c9c8bb6605bcceb1bcb6190 [diff] [blame] |
PolicyManager: Disallow copy and assign on Variable objects. Per style guide, we don't allow copy constructors on objects that don't need to be copied. BUG=None TEST=Build and unittest. Change-Id: Ib6efcc8ad9860394239d84216f730a4896cf78f9 Reviewed-on: https://chromium-review.googlesource.com/191077 Reviewed-by: Gilad Arnold <[email protected]> Commit-Queue: Alex Deymo <[email protected]> Tested-by: Alex Deymo <[email protected]>
diff --git a/policy_manager/fake_variable.h b/policy_manager/fake_variable.h index 36c3848..b1de9d2 100644 --- a/policy_manager/fake_variable.h +++ b/policy_manager/fake_variable.h
@@ -53,6 +53,8 @@ private: // The pointer returned by GetValue(). scoped_ptr<const T> ptr_; + + DISALLOW_COPY_AND_ASSIGN(FakeVariable); }; } // namespace chromeos_policy_manager