Bug 55227

Summary:

old-run-webkit-tests falsely claims to be saving crash logs on Windows Vista/7

Product:

WebKit

Reporter:

Adam Roben (:aroben) <aroben>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ddkilzer, sfalken

Priority:

Keywords:

PlatformOnly

Version:

528+ (Nightly build)

Hardware:

OS:

Windows 7

Attachments:

Description	Flags
Double-check that modifying the registry worked	darin: review+

Adam Roben (:aroben)

Reported 2011-02-25 09:48:31 PST

old-run-webkit-tests modifies the registry to configure a post-mortem debugger to save crash logs. This doesn't work on Windows Vista/7 because you need administrator privileges to modify the registry. But old-run-webkit-tests thinks it successfully modified the registry and thus claims to be saving crash logs. Actually making crash log saving work is covered by bug 44135. But in the meantime we should make old-run-webkit-tests stop lying.

Attachments
Double-check that modifying the registry worked (1.89 KB, patch) 2011-02-25 09:53 PST, Adam Roben (:aroben)	darin: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Adam Roben (:aroben)

Comment 1 2011-02-25 09:53:11 PST

Created attachment 83827 [details] Double-check that modifying the registry worked

Adam Roben (:aroben)

Comment 2 2011-02-25 10:50:31 PST

Committed r79698: <http://trac.webkit.org/changeset/79698>

Note You need to log in before you can comment on or make changes to this bug.