summaryrefslogtreecommitdiffstats
path: root/chrome/browser/metrics/metrics_service.cc
Commit message (Collapse)AuthorAgeFilesLines
* Automatically adapt to faster/slower uploads of renderer histogramsjar@chromium.org2009-05-281-29/+45
| | | | | | | | | | | | | | | | | | | | | This replaces the current time based approach (chrome is given N seconds to upload all renederer histograms) with an asynch callback approach that waits until all renderers have responded (with their updates). It uses a fall-back timer to ensure that a hung renderer won't delay things forever as well. This causes faster (and complete) updates in about:histograms as well as generally assuring complete updates during UMA gatherings. This code was contributed by Raman Tenneti in CL 42496 http://codereview.chromium.org/42496 bug=12850 r=raman Review URL: http://codereview.chromium.org/113473 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17123 0039d316-1c4b-4281-b951-d872f2087c98
* Use a NotificationRegistrar to listen for notifications.pkasting@chromium.org2009-05-221-43/+34
| | | | | | | BUG=2381 Review URL: http://codereview.chromium.org/114040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16792 0039d316-1c4b-4281-b951-d872f2087c98
* Add a RENDERER_PROCESS_CLOSED notification that gets sent when we close thempcomplete@google.com2009-05-051-3/+4
| | | | | | | | | | RenderProcessHost's process_. This is similar to RENDER_PROCESS_TERMINATED, but the latter is not always sent since the process could get reused. It also takes the place of RENDERER_PROCESS_CRASHED, where this one contains a boolean Details indicating whether the process crashed (true) or just exited (false). Review URL: http://codereview.chromium.org/100277 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15261 0039d316-1c4b-4281-b951-d872f2087c98
* ImportantFileWriterphajdan.jr@chromium.org2009-04-281-3/+2
| | | | | | | | | | | | | | Introducing a class for writing important files, preventing their corruption during writing. Switched PrefService to use it. Other classes will be switched in future changesets. TEST=This may affect things using preferences. Make sure that changes in preferences don't get lost, and that you don't get excessive disk activity when changing preferences. http://crbug.com/10618 Review URL: http://codereview.chromium.org/83001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14717 0039d316-1c4b-4281-b951-d872f2087c98
* Do not send stats from Chromium builds.kuchhal@chromium.org2009-04-271-6/+12
| | | | | | | | | BUG=1435533 Review URL: http://codereview.chromium.org/93109 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14665 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Do not send stats from Chromium builds."kuchhal@chromium.org2009-04-241-8/+6
| | | | | | | | | | | | This reverts commit 4a0189db765fe94f5df360c0907a585e26c9f681. Revert "Fix build break." This reverts commit eada493cf28975e56395aa3e2d3a0dc267a7f2c7. Review URL: http://codereview.chromium.org/92162 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14479 0039d316-1c4b-4281-b951-d872f2087c98
* Fix build break.kuchhal@chromium.org2009-04-241-1/+1
| | | | | | | | | BUG=1435533 Review URL: http://codereview.chromium.org/93154 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14478 0039d316-1c4b-4281-b951-d872f2087c98
* Do not send stats from Chromium builds.kuchhal@chromium.org2009-04-241-6/+8
| | | | | | | | | BUG=1435533 Review URL: http://codereview.chromium.org/93109 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14474 0039d316-1c4b-4281-b951-d872f2087c98
* Don't overload the meaning of the RENDERER_PROCESS_TERMINATED notification, ↵jam@chromium.org2009-04-131-4/+3
| | | | | | | | | | | | instead create a new one for crashing. The old way of using notifications was incorrect since a RenderProcessHost might have sent only one notification even though a new renderer might have been created after a crash. BUG=9379 Review URL: http://codereview.chromium.org/66069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13629 0039d316-1c4b-4281-b951-d872f2087c98
* Review URL: http://codereview.chromium.org/62160robertshield@google.com2009-04-131-0/+1
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13600 0039d316-1c4b-4281-b951-d872f2087c98
* The profile Source parameter of a BookmarkModelLoaded event can be null ↵tim@chromium.org2009-04-091-3/+5
| | | | | | | | | | during testing. MetricsService and WebContents weren't handling this case properly. Review URL: http://codereview.chromium.org/66007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13475 0039d316-1c4b-4281-b951-d872f2087c98
* Fix underflow that would cause either an access violation or an infinite ↵robertshield@google.com2009-03-181-2/+3
| | | | | | | | loop if we were ever to get a "chrome_config" node back in metrics response data. Review URL: http://codereview.chromium.org/42335 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11983 0039d316-1c4b-4281-b951-d872f2087c98
* The MetricsService are not initialized unless the user have opted injar@chromium.org2009-03-081-9/+13
| | | | | | | | | See contributed CL 28315 by Xharze BUG=2278 Review URL: http://codereview.chromium.org/40274 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11219 0039d316-1c4b-4281-b951-d872f2087c98
* Add a set of long-running metrics to Chrome that are sent up only at ↵robertshield@google.com2009-03-041-18/+39
| | | | | | | | uninstall time via the uninstall survey page.These uninstall metrics are collected according to the same opt-in policy as the existing UMA code. They are stored along with other prefs in the browser's Local State file. At uninstall time, the Local State file is copied to a temporary location during the file deletion stage and then read to extract the uninstall metrics. If the user selected to have metrics reported, the uninstall metrics are then sent up to the uninstall survey page that is currently opened. Review URL: http://codereview.chromium.org/27092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10859 0039d316-1c4b-4281-b951-d872f2087c98
* revert broken change 10833nsylvain@chromium.org2009-03-031-25/+16
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10837 0039d316-1c4b-4281-b951-d872f2087c98
* Port DictionaryValue to use string16 instead of wstring.dsh@google.com2009-03-031-16/+25
| | | | | | | Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=10818 Review URL: http://codereview.chromium.org/31014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10833 0039d316-1c4b-4281-b951-d872f2087c98
* Revert dsh's change 10818nsylvain@chromium.org2009-03-031-23/+14
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10821 0039d316-1c4b-4281-b951-d872f2087c98
* Port DictionaryValue to use string16 instead of wstring.dsh@google.com2009-03-031-14/+23
| | | | | | Review URL: http://codereview.chromium.org/31014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10818 0039d316-1c4b-4281-b951-d872f2087c98
* Fix compilations errors introduced by revision 10712.maruel@chromium.org2009-03-021-4/+5
| | | | | | | TBR=deanm Review URL: http://codereview.chromium.org/27347 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10721 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of wstring variants of StringToFoo.dsh@google.com2009-02-281-6/+6
| | | | | | Review URL: http://codereview.chromium.org/28281 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10664 0039d316-1c4b-4281-b951-d872f2087c98
* Initial support for Renderer Side Histogramsjar@chromium.org2009-02-251-0/+12
| | | | | | | | | | Patch contributed by Raman Tenneti see also patch number 21038 Review URL: http://codereview.chromium.org/27034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10330 0039d316-1c4b-4281-b951-d872f2087c98
* Use string for Histogram names since these are all ASCII anyway.dsh@google.com2009-02-241-5/+5
| | | | | | | | Wide-character literals cause problems between platforms. Review URL: http://codereview.chromium.org/28046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10276 0039d316-1c4b-4281-b951-d872f2087c98
* Re-re-roll use of ongoing logs to gather stability metricsjar@chromium.org2009-02-241-3/+1
| | | | | | | | | | | | | | This was rolled back to try to use only the initial logs to gather statibility stats, because we were having trouble capturing data in ongoing logs. We now resolved the problem (which was actually only on the stability branch), so we'll roll back to using this method and verify it in the dev branch. tbr=mal Review URL: http://codereview.chromium.org/27061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10238 0039d316-1c4b-4281-b951-d872f2087c98
* Discard crash data during first run of a new version.jar@chromium.org2009-02-231-1/+36
| | | | | | | | | | | | | | | Avoid confusion associated with uploading crash data from an old version during the first run of a new version. Carlos: Please confirm that you're fine with discarding metrics about installer and Render in Sbox. Evan: Please confirm that I got all the usage/crash stats. r=cpu,evanm,mal Review URL: http://codereview.chromium.org/28017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10211 0039d316-1c4b-4281-b951-d872f2087c98
* Make plugin_service.cc compile on Posix.jhawkins@chromium.org2009-02-201-1/+1
| | | | | | Review URL: http://codereview.chromium.org/20531 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10102 0039d316-1c4b-4281-b951-d872f2087c98
* Add a comment to a metrics_service.cc NOTIMPLEMENTED.evan@chromium.org2009-02-191-0/+3
| | | | | | Review URL: http://codereview.chromium.org/21523 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10037 0039d316-1c4b-4281-b951-d872f2087c98
* Fix build break.paul@chromium.org2009-02-131-1/+1
| | | | | | | TBR=evan git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9721 0039d316-1c4b-4281-b951-d872f2087c98
* Port most of MetricsService to Mac and linux.paul@chromium.org2009-02-131-10/+25
| | | | | | Review URL: http://codereview.chromium.org/20338 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9720 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor plugin process code in the browser process so that the ↵jam@chromium.org2009-02-111-32/+35
| | | | | | | | browser/about:memory/task manager/metrics code doesn't depend on PluginProcessHost pointers. In a future changelist I'll add one central child process registry in the browser process. Review URL: http://codereview.chromium.org/20196 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9621 0039d316-1c4b-4281-b951-d872f2087c98
* Force all stability stats into "initial log" (don't use ongoing log)jar@chromium.org2009-02-091-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | The goal of this checkin is to effectively roll-back our stats gathering methodology for stability stats to ONLY use the initial log. This has some negative consequences (time lag, and bias against data from happy ongoing users). We're doing this because there appears to be a BIG survivability differnce between initial and ongoing logs. Perhaps we are even getting multiple recordings of data from initial logs. Until this is corrected, it is not safe to send part of the stability info (crashes) only in the initial logs, while we send pageloads in part via ongoing logs. To absolutely minimize risk, I decided to leave the inter-log delay at 20 minutes, so that there is no chance of changing the log saving resource utilization (i.e., going over the limits). It was at 5 when we started the process, but since it has no impact on the initial log, it won't harm the stability stats, and it may reduce load on the front end log saving servers. bug=1564631 r=huanr,evanm Review URL: http://codereview.chromium.org/21160 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9385 0039d316-1c4b-4281-b951-d872f2087c98
* Finish taking out render_messages.h includes from headers.jam@chromium.org2009-02-051-0/+1
| | | | | | Review URL: http://codereview.chromium.org/20072 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9209 0039d316-1c4b-4281-b951-d872f2087c98
* Remove most header file dependencies on the notification type list. It isbrettw@chromium.org2009-02-011-43/+51
| | | | | | | | | | | really painful to add more types, since lots of headers include the notification service to derive from the notification observer. This splits that out, so much less of the project should end up including notification_types.h ---Paths modified but not in any changelist: Review URL: http://codereview.chromium.org/19744 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@9020 0039d316-1c4b-4281-b951-d872f2087c98
* Change kMetricsURL towtc@chromium.org2009-01-301-3/+1
| | | | | | | | | | | | | https://clients4.google.com/firefox/metrics/collect, which has a certificate that matches the host name. Remove the net::LOAD_IGNORE_CERT_COMMON_NAME_INVALID load flag. R=evanm BUG=b/876021 Review URL: http://codereview.chromium.org/19450 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8940 0039d316-1c4b-4281-b951-d872f2087c98
* Make purify happy by fixing memory leak that shows up during shutdown.kuchhal@chromium.org2009-01-281-0/+8
| | | | | | | | | BUG=6528 Review URL: http://codereview.chromium.org/19407 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8780 0039d316-1c4b-4281-b951-d872f2087c98
* Discard corrupt old logs in UMA uploadjar@chromium.org2009-01-211-6/+11
| | | | | | | | | | | | Detect logs that are unacceptable to the server, and don't retransmit them (it won't help to send them again). bug= 1505736 r=huanr,evanm Review URL: http://codereview.chromium.org/18444 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8361 0039d316-1c4b-4281-b951-d872f2087c98
* Add more production logging to diagnose upload problemjar@chromium.org2009-01-201-27/+29
| | | | | | | | | | | | | | | It appears that some logs get "stuck" trying to be sent, so I've changed a number of DLOG() calls to LOG() calls. I also added a bit of info to the LOG() calls, and made minor edits. bug=1505736 r=evanm Review URL: http://codereview.chromium.org/18354 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8304 0039d316-1c4b-4281-b951-d872f2087c98
* Move functions required by the rest of the browser from RenderProcessHost to ↵brettw@chromium.org2009-01-161-1/+1
| | | | | | | | an interface and move the implementation to BrowserRenderProcessHost. This will allow me to write render view unit tests without using the actual renderer, but there are no tests yet. Review URL: http://codereview.chromium.org/18132 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8188 0039d316-1c4b-4281-b951-d872f2087c98
* Move metrics files into a subdirben@chromium.org2009-01-161-0/+1755
Review URL: http://codereview.chromium.org/18302 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8156 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-07 09:41:55 +0000