diff options
| author | oshima@chromium.org <oshima@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-17 13:12:19 +0000 |
|---|---|---|
| committer | oshima@chromium.org <oshima@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-17 13:12:19 +0000 |
| commit | d86b4d58f9cfbd14f225075a1efe9baa7410a83d (patch) | |
| tree | 3049cffd49d5ea0ab2c89a96e1c21de1a1a9ec4b /rlz/lib/financial_ping.h | |
| parent | 26329c0092e737f2b824452eaf2cd2c5775fedce (diff) | |
| download | chromium_src-d86b4d58f9cfbd14f225075a1efe9baa7410a83d.zip chromium_src-d86b4d58f9cfbd14f225075a1efe9baa7410a83d.tar.gz chromium_src-d86b4d58f9cfbd14f225075a1efe9baa7410a83d.tar.bz2 | |
Add periodic check to see if context has been reset.
Context may be set to NULL when shutdown is requested
while trying to send ping. With this CL, rlz will cancel
the request when it detects the NULL context.
non chromeos chrome used to wait even after context is set
to NULL, so maybe I should do this only for chromeos. Please
let me know what you think.
This also fixes the edge race case where the g_context
may be set to NULL in the middle of FinancialPing::PingServer.
BUG=261377
Review URL: https://codereview.chromium.org/26929004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229114 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'rlz/lib/financial_ping.h')
| -rw-r--r-- | rlz/lib/financial_ping.h | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/rlz/lib/financial_ping.h b/rlz/lib/financial_ping.h index 081ecba..63c20a1 100644 --- a/rlz/lib/financial_ping.h +++ b/rlz/lib/financial_ping.h @@ -58,6 +58,13 @@ class FinancialPing { ~FinancialPing() {} }; +#if defined(RLZ_NETWORK_IMPLEMENTATION_CHROME_NET) +namespace test { +void ResetSendFinancialPingInterrupted(); +bool WasSendFinancialPingInterrupted(); +} // namespace test +#endif + } // namespace rlz_lib |