diff options
author | rdsmith@google.com <rdsmith@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-22 17:25:26 +0000 |
---|---|---|
committer | rdsmith@google.com <rdsmith@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-22 17:25:26 +0000 |
commit | c4058fb9d4e9203cec16ea7641d7ddc0fd07f2a4 (patch) | |
tree | 6c8738a1175345876588f9ffde67fb8a84497a85 /chrome | |
parent | 45ac1e4802f7265c354af298d92ee087bb062dbc (diff) | |
download | chromium_src-c4058fb9d4e9203cec16ea7641d7ddc0fd07f2a4.zip chromium_src-c4058fb9d4e9203cec16ea7641d7ddc0fd07f2a4.tar.gz chromium_src-c4058fb9d4e9203cec16ea7641d7ddc0fd07f2a4.tar.bz2 |
Initial commit of CookieMonster statistics.
Specifically:
* Number of cookies, recorded every ten minutes of active browsing (i.e. cookies being requested from CookieMonster)
* Last access time, recorded when cookie accessed.
* Last access time for evicted cookies, recorded on eviction
* Time until cookie expires, recorded on insertion.
* Reason for a cookie being removed from store, recorded when removed.
* Size of batch update to persistent cookie store and whether it succeeded or failed, recorded at success/failure.
BUG=4005
TEST=net_unittests CookieMonster.* on Linux
Review URL: http://codereview.chromium.org/2718011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50477 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/browser/net/sqlite_persistent_cookie_store.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/chrome/browser/net/sqlite_persistent_cookie_store.cc b/chrome/browser/net/sqlite_persistent_cookie_store.cc index a098c44..30833bf 100644 --- a/chrome/browser/net/sqlite_persistent_cookie_store.cc +++ b/chrome/browser/net/sqlite_persistent_cookie_store.cc @@ -231,7 +231,9 @@ void SQLitePersistentCookieStore::Backend::Commit() { break; } } - transaction.Commit(); + bool succeeded = transaction.Commit(); + UMA_HISTOGRAM_ENUMERATION("net.CookieBackingStoreUpdateResults", + succeeded ? 0 : 1, 2); } // Fire off a close message to the background thread. We could still have a |