Bug 68626 – Remove all uses of FindSafari

Bug 68626 - Remove all uses of FindSafari


Summary:	Remove all uses of FindSafari

Status:	RESOLVED FIXED

Product:	WebKit
Classification:	Unclassified
Component:	Tools / Tests
Version:	528+ (Nightly build)
Platform:	PC Windows 7

Importance:	P2 Normal
Assigned To:	Adam Roben (:aroben)

URL:
Keywords:	PlatformOnly

Depends on:
Blocks:	68628
	Show dependency tree / graph

Reported:	2011-09-22 09:10 PDT by Adam Roben (:aroben)
Modified:	2011-09-22 10:52 PDT (History)
CC List:	1 user (show)

See Also:

Attachments
Remove all uses of FindSafari (4.04 KB, patch) 2011-09-22 09:13 PDT, Adam Roben (:aroben)	darin: review+	Details \| Formatted Diff \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

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

Description Adam Roben (:aroben) 2011-09-22 09:10:26 PDT

FindSafari isn't really needed anymore. It originally had two jobs:

1) Print out commands that could be used to launch Safari with a nightly version of WebKit
2) Print out the path to Safari

(1) isn't needed anymore; we have WebKit.exe for launching Safari with nightly builds. And (2) isn't needed, either; we can just use regtool to pull the values out of the registry much more easily.

Comment 1 Adam Roben (:aroben) 2011-09-22 09:13:48 PDT

Created attachment 108343 [details]
Remove all uses of FindSafari

Comment 2 Adam Roben (:aroben) 2011-09-22 10:52:07 PDT

Committed r95733: <http://trac.webkit.org/changeset/95733>