commit f967ebeb1851548a37c01db2bc4f78b8db2d2f99
author Alex Deymo <deymo@chromium.org> Mon May 05 14:46:17 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed May 07 17:52:59 2014 +0000
tree 9bea7d79f07979ca1263ca6040a82a60c5e8e0da
parent 7e8ac06833aba4d851257fe287b38c98af4e226b

PolicyManager: Move payload_size to int64_t.

Code style encourages signed types instead of unsigned types and the
value being exposed is already signed. This patch fixes that for the
payload_size variable and adapts the BoxedValue::ValuePrinter
implementations to use the int64_t and uint64_t types.

BUG=None
TEST=Unit tests updated

Change-Id: I21310c59d8c2654c43cac27265055c8577341562
Reviewed-on: https://chromium-review.googlesource.com/198269
Reviewed-by: Alex Deymo <deymo@chromium.org>
Commit-Queue: Alex Deymo <deymo@chromium.org>
Tested-by: Alex Deymo <deymo@chromium.org>

policy_manager/fake_updater_provider.h

diff --git a/policy_manager/fake_updater_provider.h b/policy_manager/fake_updater_provider.h
index c9fbdc6..e8fee63 100644
--- a/policy_manager/fake_updater_provider.h
+++ b/policy_manager/fake_updater_provider.h
@@ -41,7 +41,7 @@
     return &var_new_version_;
   }
 
-  virtual FakeVariable<size_t>* var_payload_size() override {
+  virtual FakeVariable<int64_t>* var_payload_size() override {
     return &var_payload_size_;
   }
 
@@ -76,7 +76,7 @@
   FakeVariable<double> var_progress_{"progress", kVariableModePoll};
   FakeVariable<Stage> var_stage_{"stage", kVariableModePoll};
   FakeVariable<std::string> var_new_version_{"new_version", kVariableModePoll};
-  FakeVariable<size_t> var_payload_size_{"payload_size", kVariableModePoll};
+  FakeVariable<int64_t> var_payload_size_{"payload_size", kVariableModePoll};
   FakeVariable<std::string> var_curr_channel_{
       "curr_channel", kVariableModePoll};
   FakeVariable<std::string> var_new_channel_{"new_channel", kVariableModePoll};