summaryrefslogtreecommitdiffstats
path: root/chrome/browser/diagnostics
Commit message (Collapse)AuthorAgeFilesLines
* Rename CommandLine::ARGUMENTS_ONLY to NO_PROGRAM.mattm@chromium.org2010-10-211-1/+1
| | | | | | | | | | | ARGUMENTS_ONLY was misleading since CommandLine has methods for handling "switches" and "arguments", but that constructor still allows both. BUG=none TEST=still builds Review URL: http://codereview.chromium.org/3935001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63325 0039d316-1c4b-4281-b951-d872f2087c98
* Move windows version-related stuff out of base/win_util and into ↵brettw@chromium.org2010-10-151-5/+5
| | | | | | | | | | | | | | base/win/windows_version. Many files now only need to include this instead of all of win_util. Remove a bunch of unused code from base/win_util. There was a surprising amount. Replace the AppUserModel property key with the one from the SDK now that we use the Win7 SDK. Move GetLogonSessionOnlyDACL from win_util to ipc since it's only used in that one place. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3823002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62694 0039d316-1c4b-4281-b951-d872f2087c98
* Move Stats, histograms, and field trial into a metrics subdirectory of base andbrettw@chromium.org2010-10-141-1/+1
| | | | | | | | | put them in the base namespace. TEST=it compiles BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62510 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Disable DiagnosticsModelTest.RunAll."agl@chromium.org2010-10-071-2/+1
| | | | | | This reverts commit r61844. nsylvain fixed the builder. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61846 0039d316-1c4b-4281-b951-d872f2087c98
* Disable DiagnosticsModelTest.RunAll.agl@chromium.org2010-10-071-1/+2
| | | | | | | | | It's crashing the builder but we don't know why. However, we don't want to be blinded by it in the mean time. TBR=jar git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61844 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor version-getting info into a chrome::VersionInfo object.evan@chromium.org2010-08-271-12/+8
| | | | | | | | | | | | | | | I was trying to replace wstring usage in base::FileVersionInfo, but that class is rather Windows-specific with strange fields like "private_build()" where the value and encoding aren't clear. 95% of the users of FileVersionInfo actually just care about the current Chrome version, so we can provide a much simpler interface for them. We still use FileVersionInfo for retrieving information from e.g. plugin DLLs, but in those cases the usage is clearer. Review URL: http://codereview.chromium.org/3135028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57725 0039d316-1c4b-4281-b951-d872f2087c98
* Convert FormatBytes to string16.evan@chromium.org2010-08-191-4/+2
| | | | | | | | | | I considered ASCII, but it's producing what is intended to be a human-readable string. For example, in theory the "bytes/s" annotation could be localized into a language where the "b" is non-ASCII. Review URL: http://codereview.chromium.org/3107021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56771 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up unnecessary #includes of base/{scoped_ptr.h,string16.h}.viettrungluu@chromium.org2010-08-191-2/+1
| | | | | | | | | | | | Also fix a bunch of #includes. Folks, scoped_refptr<> is defined in base/ref_counted.h, not in base/scoped_ptr.h. BUG=none TEST=builds Review URL: http://codereview.chromium.org/3132024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56712 0039d316-1c4b-4281-b951-d872f2087c98
* Move bundled copy of sqlite one level deeper to better separate itphajdan.jr@chromium.org2010-08-181-1/+1
| | | | | | | | | | | | | from our patches, READMEs, etc. Also, add a shim header so we can use it for building with system sqlite. TEST=compile BUG=22208 Review URL: http://codereview.chromium.org/3108030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56619 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Convert FormatBytes to string16."evan@chromium.org2010-08-181-2/+4
| | | | | | This reverts commit r56603. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56608 0039d316-1c4b-4281-b951-d872f2087c98
* Convert FormatBytes to string16.evan@chromium.org2010-08-181-4/+2
| | | | | | | | | | I considered ASCII, but it's producing what is intended to be a human-readable string. For example, in theory the "bytes/s" annotation could be localized into a language where the "b" is non-ASCII. Review URL: http://codereview.chromium.org/3107021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56603 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 1)thestig@chromium.org2010-08-051-4/+0
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3071012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55034 0039d316-1c4b-4281-b951-d872f2087c98
* Add #include utf_string_conversions.h to all files that use ASCIIToWide andbrettw@chromium.org2010-08-031-0/+1
| | | | | | | | | | | | | | | ASCIIToUTF16. I removed string_util includes from a few places where it obviously wasn't needed. In a separate pass, I'm going to remove ASCIITo* from string_util, then I'm going to do an even later pass to find the unnecessary string_util.h includes and remove them. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3058027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54746 0039d316-1c4b-4281-b951-d872f2087c98
* Move the number conversions from string_util to a new file.brettw@chromium.org2010-07-302-4/+5
| | | | | | | | | | | | | Use the base namespace in the new file. Update callers. I removed all wstring variants and also the string->number ones that ignore the return value. That encourages people to write code and forget about error handling. TEST=included unit tests BUG=none Review URL: http://codereview.chromium.org/3056029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54355 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unneeded headers from app/thestig@chromium.org2010-07-292-2/+0
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/2819063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54102 0039d316-1c4b-4281-b951-d872f2087c98
* `#pragma once` for app, base, chrome, gfx, ipc, net, skia, viewsthakis@chromium.org2010-07-265-0/+5
| | | | | | | | | BUG=50273 TEST=everything still builds, build is 10% faster on windows, same speed on mac/linux TBR: erg git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53716 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity issues 9318, 9319: Uninitialized members in diagnostics_main.ccfinnur@chromium.org2010-07-241-3/+5
| | | | | | | | | BUG=None (Coverity issue id listed above) TEST=None Review URL: http://codereview.chromium.org/3047013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53594 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Break another common->app dependency.thestig@chromium.org2010-07-201-3/+5
| | | | | | | | BUG=46666 TEST=none Review URL: http://codereview.chromium.org/3007008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53113 0039d316-1c4b-4281-b951-d872f2087c98
* Upping the diagnostic mode limits for file sizecpu@chromium.org2010-07-141-2/+2
| | | | | | | | | | | | - The local state and the overall size of the profile dir BUG=42561 Review URL: http://codereview.chromium.org/2934019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52400 0039d316-1c4b-4281-b951-d872f2087c98
* diagnostics: Implement PosixConsole::Read function.tfarina@chromium.org2010-06-181-2/+9
| | | | | | | | | | BUG=42894 TEST=enable the "Press [enter] to continue" on Posix, run out/Debug/chrome --diagnostics and see if it can read the line. Review URL: http://codereview.chromium.org/2820009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50208 0039d316-1c4b-4281-b951-d872f2087c98
* diagnostics: Add VersionTest to linux.tfarina@chromium.org2010-06-172-1/+4
| | | | | | | | | | | Now with the mac build fixed. BUG=42894 TEST=out/Debug/chrome --diagnostics Review URL: http://codereview.chromium.org/2821009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50071 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 49875 - diagnostics: Add VersionTest to linux.michaeln@google.com2010-06-162-2/+1
| | | | | | | | | | | | BUG=42894 TEST=out/Debug/chrome --diagnostics Review URL: http://codereview.chromium.org/2856004 TBR=tfarina@chromium.org Review URL: http://codereview.chromium.org/2827010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49881 0039d316-1c4b-4281-b951-d872f2087c98
* diagnostics: Add VersionTest to linux.tfarina@chromium.org2010-06-162-1/+2
| | | | | | | | | BUG=42894 TEST=out/Debug/chrome --diagnostics Review URL: http://codereview.chromium.org/2856004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49875 0039d316-1c4b-4281-b951-d872f2087c98
* Posix: implement diagnostics mode console output.mattm@chromium.org2010-05-151-37/+144
| | | | | | | | | BUG=42894,42345 TEST=chrome --diagnostics Review URL: http://codereview.chromium.org/2115001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47348 0039d316-1c4b-4281-b951-d872f2087c98
* Changing linux platform for chromeos. This involved moving platform_util ↵dhg@chromium.org2010-04-291-1/+1
| | | | | | | | | | and extension_error_reporter to browser, so that the UIs surrounding them can be in the browser. BUG=none TEST=none Review URL: http://codereview.chromium.org/1705009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45986 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor FileVersionInfo into an interface with platform implementations.tony@chromium.org2010-04-141-1/+2
| | | | | | | | | | | | | This allows us to move the chrome specific version informaton used by Linux into src/chrome. Add a GetChromeVersionInfo() for Linux in src/chrome/app/ and make sure to use this in src/chrome. In src/webkit/glue, add a new glue method for getting the product version. When compiling chrome, use an implementation in src/chrome/renderer (which uses GetChromeVersionInfo()) and a stub implementation for test_shell. Review URL: http://codereview.chromium.org/1560027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44435 0039d316-1c4b-4281-b951-d872f2087c98
* Add diagnostic check for the Preferences filecpu@chromium.org2010-04-145-4/+90
| | | | | | | | | | | - Parses the json file BUG=27885 TEST=test modified Review URL: http://codereview.chromium.org/1609022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44429 0039d316-1c4b-4281-b951-d872f2087c98
* Allow specifiying a different user-data-dir for diagnostic modecpu@chromium.org2010-04-096-27/+62
| | | | | | | | | | | - Polish the db tests a little bit, now there are critical and non-critical and it handles better is chrome is already running BUG=27885 TEST=none Review URL: http://codereview.chromium.org/1540027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44122 0039d316-1c4b-4281-b951-d872f2087c98
* Include the appcache and database tracker databases into the sql diagnostics ↵michaeln@chromium.org2010-04-024-71/+69
| | | | | | | | | | | | fold. TEST=manual BUG=none Review URL: http://codereview.chromium.org/1508016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43534 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land sqlite diagnosticscpu@chromium.org2010-03-314-6/+125
| | | | | | | | | | | | | | - Forgot to update a stupid number in the tests See http://codereview.chromium.org/1548005 BUG=27885 TEST=included Review URL: http://codereview.chromium.org/1543006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43169 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 43143 - Implement db corruption tests for diagnostic modearv@chromium.org2010-03-313-123/+4
| | | | | | | | | | | | | | | Uses sqlite's pragma integrity_check Tests the 5 main databases BUG=27885 TEST=none Review URL: http://codereview.chromium.org/1548005 TBR=cpu@chromium.org Review URL: http://codereview.chromium.org/1539006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43152 0039d316-1c4b-4281-b951-d872f2087c98
* Implement db corruption tests for diagnostic modecpu@chromium.org2010-03-303-4/+123
| | | | | | | | | | | | | - Uses sqlite's pragma integrity_check - Tests the 5 main databases BUG=27885 TEST=none Review URL: http://codereview.chromium.org/1548005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43143 0039d316-1c4b-4281-b951-d872f2087c98
* Diagnostic mode shows wrong size for certain pathscpu@chromium.org2010-03-241-8/+19
| | | | | | | | | | | | | - See the bug, it has a screenshot The problem is that we need to recourse over the directory. TEST=none BUG=39130 Review URL: http://codereview.chromium.org/1252002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42556 0039d316-1c4b-4281-b951-d872f2087c98
* BSD ifdefs, etc. necessary for chrome/pvalchev@google.com2010-03-091-6/+6
| | | | | | Review URL: http://codereview.chromium.org/656009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41067 0039d316-1c4b-4281-b951-d872f2087c98
* chrome 1: string_util.h -> utf_string_conversions.h fix.jhawkins@chromium.org2010-03-081-0/+1
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/672001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40937 0039d316-1c4b-4281-b951-d872f2087c98
* More work on diagnostics modecpu@chromium.org2010-02-054-43/+106
| | | | | | | | | | | | | | | | - Clean up a bit, reduce windows specific code - Add disk space test - Add Chrome version test - Make OS version readable, cross platform mostly This concludes the minimal sanity tests. BUG=27885 TEST= run chrome with --diagnostics , no test should fail. Review URL: http://codereview.chromium.org/561078 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38176 0039d316-1c4b-4281-b951-d872f2087c98
* Improvements to the diagnostic modecpu@chromium.org2010-01-142-11/+47
| | | | | | | | | | | | | | - bettet banner - init ICU - size checks on the critical paths BUG=none TEST=included Review URL: http://codereview.chromium.org/521050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36278 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup histogram classes mixing SetFlags into FactoryGet argumentsjar@chromium.org2009-12-281-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Generic cleanup of histogram class, renaming *FactoryGet to FactoryGet, along with reformatting. The macros were cleaned up to use common sub-macros rather than repeating code as much. Removed ThreadSafeHistogram (and associated ASSET_HISTOGRAM macros) since this class was not getting used. I introduced UMA_HISTOGRAM_ENUMERATION to support the common use of LinearHistograms to count various enumerated values. I added a Flags argument to all the FactoryGet routines to help avoid needing to call SetFlags each time a new sample is Add()ed. This also simplifies the code. This will all help prepare for a "don't histogram at all" macro setting so that I can test the impact of the histogram macro calls on performance (since there are now so many active histograms). BUG=31206 r=raman.tenneti Review URL: http://codereview.chromium.org/515033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35295 0039d316-1c4b-4281-b951-d872f2087c98
* Add some diagnostic test to diagnostic modecpu@chromium.org2009-12-205-48/+266
| | | | | | | | | | | | | | | - Move the one existing test to final location - Test more critical paths to be accessilbe and writable - Test the OS version - Test user or systemwide install type BUG=27885 TEST=unit test updated Review URL: http://codereview.chromium.org/504028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35057 0039d316-1c4b-4281-b951-d872f2087c98
* Use factory to create histograms, and refcounts to track lifetimesjar@chromium.org2009-12-061-3/+5
| | | | | | | | | | | | | | | This is CL patch 377028 by Raman Tenneti, with minor changes to make the try-bots happier. It is cleanup that better ensures lifetimes of histograms (making it harder for users to abuse them). bug=16495 (repairs leak induced by the first landing) bug=18840 (should make leaks less possible) tbr=raman.tenneti Review URL: http://codereview.chromium.org/462027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33933 0039d316-1c4b-4281-b951-d872f2087c98
* Basic console output for diagnostics mode windows-onlycpu@chromium.org2009-12-035-7/+244
| | | | | | | | | | | | Also Wires --diagnostics to the diagnostics entry point start chrome with --diagnostics and watch pretty output BUG=27885 TEST= unit tests included Review URL: http://codereview.chromium.org/454020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33663 0039d316-1c4b-4281-b951-d872f2087c98
* Introducing the diagnostic model classescpu@chromium.org2009-11-194-0/+380
| | | | | | | | | | | | | Chrome diagnostics wants to be a MVC, this is the classes that compose the model part. BUG=27885 TEST=included unit tests Review URL: http://codereview.chromium.org/385144 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32494 0039d316-1c4b-4281-b951-d872f2087c98
* Move the sqlite error handler to a single locationcpu@chromium.org2009-10-152-0/+79
| | | | | | | | | | | | | | - Eliminate code duplication - Cover other 3 databases - Still doing the same as before, sending UMA histograms BUG=11908 TEST=none Review URL: http://codereview.chromium.org/270101 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29073 0039d316-1c4b-4281-b951-d872f2087c98
* Empty files to kick off the diagnostic modecpu@chromium.org2009-10-134-0/+29
- Main entry point - sqlite error handler BUG=none TEST=none TBR=huanr Review URL: http://codereview.chromium.org/274033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28908 0039d316-1c4b-4281-b951-d872f2087c98