Revert 59138 - Revert 59123 - Don't use the return value of rlz_lib::SendFinancialPin...
commitf0eb8fe0a5426d157b02326d94c6c76ff6f6b7bc
authorrogerta@google.com <rogerta@google.com@0039d316-1c4b-4281-b951-d872f2087c98>
Mon, 13 Sep 2010 18:18:18 +0000 (13 18:18 +0000)
committerrogerta@google.com <rogerta@google.com@0039d316-1c4b-4281-b951-d872f2087c98>
Mon, 13 Sep 2010 18:18:18 +0000 (13 18:18 +0000)
treece415302e4293700c8f4d99f63075e6bb9e4f208
parentffa59d622418c4093bc4adefe80201af0ab11e56
Revert 59138 - 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

Review URL: http://codereview.chromium.org/3330019

TBR=sky@chromium.org
Review URL: http://codereview.chromium.org/3323028

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59248 0039d316-1c4b-4281-b951-d872f2087c98
DEPS
chrome/browser/extensions/extension_rlz_apitest.cc
chrome/browser/extensions/extension_rlz_module.cc
chrome/common/extensions/api/extension_api.json
chrome/test/data/extensions/api_test/rlz/test.js