From 772d548d6997833e9d4cd2d5649a6aadcad30124 Mon Sep 17 00:00:00 2001 From: "rogerta@google.com" Date: Fri, 10 Sep 2010 19:01:49 +0000 Subject: 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 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59123 0039d316-1c4b-4281-b951-d872f2087c98 --- DEPS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'DEPS') diff --git a/DEPS b/DEPS index 4988a43..cddbb2c 100644 --- a/DEPS +++ b/DEPS @@ -217,7 +217,7 @@ deps_os = { Var("nacl_revision")), "src/rlz": - (Var("googlecode_url") % "rlz") + "/trunk@17", + (Var("googlecode_url") % "rlz") + "/trunk@18", }, "mac": { "src/chrome/tools/test/reference_build/chrome_mac": -- cgit v1.1