summaryrefslogtreecommitdiffstats
path: root/chrome_frame/crash_reporting
Commit message (Collapse)AuthorAgeFilesLines
* Cleanup: Add a bunch of missing includes.thestig@chromium.org2010-05-101-3/+5
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/2018001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46850 0039d316-1c4b-4281-b951-d872f2087c98
* Avoid reporting crashes for exceptions that hit our SEH from calls to the ↵robertshield@chromium.org2010-05-012-33/+48
| | | | | | | | | | | | original implementation of BindToStorage() when we do not wrap the bind status callback. BUG=42660 TEST=Induce exception in code called under original IMoniker::BindToStorage implementation when we don't wrap the callback and notice that no crash is reported. Review URL: http://codereview.chromium.org/1748016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46176 0039d316-1c4b-4281-b951-d872f2087c98
* Some cleanup regarding Siggi's comments on ↵robertshield@chromium.org2010-04-282-3/+4
| | | | | | | | | | | | http://codereview.chromium.org/1733021/show BUG=42660 TEST=none Review URL: http://codereview.chromium.org/1703015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45859 0039d316-1c4b-4281-b951-d872f2087c98
* Add an ExceptionBarrier around outbound calls to patched methods in IE. In ↵robertshield@chromium.org2010-04-282-12/+39
| | | | | | | | | | | | so doing, we have an SEH present in the SEH chain and so the VEH won't erroneously report crashes that occur in other modules when we happen to be on the stack. BUG=42660 TEST=Less false positives in the crash reports. Review URL: http://codereview.chromium.org/1733021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45764 0039d316-1c4b-4281-b951-d872f2087c98
* Heuristically avoid reporting crashes during DLL loading.siggi@chromium.org2010-03-125-18/+117
| | | | | | | | | | | Cleanup white space as requested by Robert in CL 882001. BUG=31980 TEST=Unittests in this change. Review URL: http://codereview.chromium.org/872004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41367 0039d316-1c4b-4281-b951-d872f2087c98
* Utility functions to interact with the NT loader's data structures and ↵siggi@chromium.org2010-03-116-0/+587
| | | | | | | | | | | | | associated tests. This is in preparation for squelching false positive crash reports during DLL load. BUG=31980 TEST=Unittests in this change. Review URL: http://codereview.chromium.org/882001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41289 0039d316-1c4b-4281-b951-d872f2087c98
* Make minidump test deterministic by explicitly loading and unloading a ↵siggi@chromium.org2010-02-261-5/+13
| | | | | | | | | | | | | | specific DLL, instead of relying on CoInitialize/CoUninitialize to load and unload at least one DLL. Add minidump_test as a dependency to make sure it's always built. BUG=none TEST=none Review URL: http://codereview.chromium.org/661081 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40142 0039d316-1c4b-4281-b951-d872f2087c98
* Add a test executable to allow verifying which minidump flag configurations ↵siggi@chromium.org2010-02-052-0/+444
| | | | | | | | | | | work on which OSen. BUG=32441 TEST=none Review URL: http://codereview.chromium.org/578003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38238 0039d316-1c4b-4281-b951-d872f2087c98
* Tweak the crash dump flags to make sure we get PEB/TEB and the unloaded ↵siggi@chromium.org2010-02-051-1/+20
| | | | | | module list where available. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38224 0039d316-1c4b-4281-b951-d872f2087c98
* Committing for stoyan since we need this now...mad@chromium.org2010-02-031-3/+3
| | | | | | | | | http://codereview.chromium.org/569017 BUG=0 TEST=Run the test for both Debug and Release builds git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37972 0039d316-1c4b-4281-b951-d872f2087c98
* Omit FPO so ::RtlCaptureStackBackTrace is more reliable in Release build for ↵stoyan@chromium.org2010-02-022-5/+8
| | | | | | | | the unittests. Review URL: http://codereview.chromium.org/566010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37868 0039d316-1c4b-4281-b951-d872f2087c98
* Prevent excessive crash reporting due stack overflow (due exception in SEH ↵stoyan@chromium.org2010-01-318-358/+645
| | | | | | | | | filter). BUG=32441 Review URL: http://codereview.chromium.org/557021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37673 0039d316-1c4b-4281-b951-d872f2087c98
* Suppress crash dump generation due to IsBadXXX functions.amit@chromium.org2010-01-156-44/+127
| | | | | | | | | | | | | IsBad[Read/Write]Ptr etc functions cause access violations which we catch and take crash dump if we are on stack. Adding code to suppress these false positives. BUG=none TEST=added new tests cases to ChromeFrame.ExceptionReport Review URL: http://codereview.chromium.org/536073 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36403 0039d316-1c4b-4281-b951-d872f2087c98
* The ChromeFrame crash reporting code should not depend on chrome frame as it ↵ananta@chromium.org2009-12-092-12/+26
| | | | | | | | | | | | | is also linked in by toolbar. This CL gets rid of this dependency and also ensures that full crash dumps are gathered during chrome frame reliability test runs. Review URL: http://codereview.chromium.org/481003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34180 0039d316-1c4b-4281-b951-d872f2087c98
* Added support for running reliability tests for ChromeFrame on similar lines ↵ananta@chromium.org2009-12-091-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | as Chrome. We only run these tests for IE at this point. The reliability test code for Chrome has been copied and modified accordingly. Other related changes in this CL include the following:- 1. If ChromeFrame is running in headless mode determined by a registry value in HKCU\Software\Google\ChromeFrame we initialize ChromeFrame crash reporting and connect to the Chrome crash server. This would enable us to gather crash dumps from the reliability test runs and report the same. 2. The LowIntegrity fixes for the WebBrowser which Stoyan had done a while back are only needed for IE7 on Vista. For this CL though we just do the requisite hacks if the OS is Vista. For Windows7 the returned IWebBrowser interface pointer works fine. 3. I moved the WebBrowserEventSink to chrome_frame_test_utils as this class is now shared. Fixes portions of http://code.google.com/p/chromium/issues/detail?id=29451 Bug=29451 Review URL: http://codereview.chromium.org/465074 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34119 0039d316-1c4b-4281-b951-d872f2087c98
* Comment fixed.stoyan@google.com2009-10-231-1/+1
| | | | | | | TBR=amit Review URL: http://codereview.chromium.org/335013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29924 0039d316-1c4b-4281-b951-d872f2087c98
* Do not create dump on custom exception codes.stoyan@google.com2009-10-231-0/+7
| | | | | | | | BUG=23845 TEST=none Review URL: http://codereview.chromium.org/337002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29920 0039d316-1c4b-4281-b951-d872f2087c98
* For Steven Truong (stevet@)mad@chromium.org2009-10-202-0/+206
| | | | | | | | Reviewed in: http://codereview.chromium.org/276068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29543 0039d316-1c4b-4281-b951-d872f2087c98
* Committing for Steven who got LGTM from here:mad@chromium.org2009-10-155-0/+357
http://codereview.chromium.org/264068 BUG=none TEST=none Review URL: http://codereview.chromium.org/276050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29196 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-02 08:17:44 +0000