commit 63137e5ded659c02eb70d59ef38f99aecac4d076
author J. Richard Barnette <jrbarnette@chromium.org> Mon Oct 28 10:57:29 2013 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Mon Oct 28 20:17:19 2013 +0000
tree a0f01d7a1fc4fadd89850c67b962fc4351a0381f
parent 60fc59c3936dee428924a8fd7fbee3a73787cdca

Split GetECVersion to allow saner unit testing.

Previously, GetECVersion() accepted a (const char *) parameter that
was used solely as a flag meaning "unit test this function with
the specified data in place of mosys output".  This change splits
the function into GetECVersion() and ParseECVersion(), and only unit
tests the second part.

BUG=None
TEST=unit tests

Change-Id: Ic48d18c02bd1924f49a0d8f0034ccb1ae8b5231e
Reviewed-on: https://chromium-review.googlesource.com/174883
Commit-Queue: Richard Barnette <jrbarnette@chromium.org>
Tested-by: Richard Barnette <jrbarnette@chromium.org>
Reviewed-by: Chris Sosa <sosa@chromium.org>

omaha_request_params.cc

diff --git a/omaha_request_params.cc b/omaha_request_params.cc
index 9a729f0..ea25c74 100644
--- a/omaha_request_params.cc
+++ b/omaha_request_params.cc
@@ -81,7 +81,7 @@
   hwid_ = utils::GetHardwareClass();
   if (CollectECFWVersions()) {
     fw_version_ = utils::GetFirmwareVersion();
-    ec_version_ = utils::GetECVersion(NULL);
+    ec_version_ = utils::GetECVersion();
   }
 
   if (current_channel_ == target_channel_) {