From 670e362c28e8a92dda5dd537cca662d283ffc2a8 Mon Sep 17 00:00:00 2001 From: "sky@chromium.org" Date: Fri, 10 Sep 2010 20:14:37 +0000 Subject: Revert 59123 - Don't use the return value of rlz_lib::SendFinancialPing() as the return value of the chrome extension API, since a false does not represent an error. Added an optional callback to the API so that caller can determine if the ping was sent or not. This CL is to reapply change 58827, which was reverted due to a test break. I found an uninitialized variable in the underlying RLZ code, see http://code.google.com/p/rlz/source/detail?r=18 for the fix. This CL includes a bump of the RLZ library to get this fix. The tests have been expanded a bit too since 58827. BUG=54294 TEST=n/a Review URL: http://codereview.chromium.org/3350016 TBR=rogerta@google.com Review URL: http://codereview.chromium.org/3330019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59138 0039d316-1c4b-4281-b951-d872f2087c98 --- DEPS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'DEPS') diff --git a/DEPS b/DEPS index cddbb2c..4988a43 100644 --- a/DEPS +++ b/DEPS @@ -217,7 +217,7 @@ deps_os = { Var("nacl_revision")), "src/rlz": - (Var("googlecode_url") % "rlz") + "/trunk@18", + (Var("googlecode_url") % "rlz") + "/trunk@17", }, "mac": { "src/chrome/tools/test/reference_build/chrome_mac": -- cgit v1.1