Bug 26312 - Multiple copies of Safari opened with WebKit nightly and Safari 3.2.3
Summary: Multiple copies of Safari opened with WebKit nightly and Safari 3.2.3
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-06-11 01:13 PDT by Gerald Cornish
Modified: 2009-06-15 00:38 PDT (History)
2 users (show)

See Also:


Attachments
Patch (2.60 KB, patch)
2009-06-15 00:09 PDT, Mark Rowe (bdash)
zwarich: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gerald Cornish 2009-06-11 01:13:59 PDT
Webkit r44552 & r44591. When webkit or webarchive or url opened in Finder I get a new version of Safari each time. Each version has only the url or archive clicked on, or has whatever pages I opened within that version. In other words each version is distinct from all the others. The menu entry reads Safari for each invocation and the dock shows multiple versions of "Safari" - previously the dock would show ONE entry for WEBKIT and there would be only one copy of Safari!
Comment 1 Mark Rowe (bdash) 2009-06-11 01:24:34 PDT
I know what's going on here.  I made some tweaks to the WebKit nightly launcher recently to improve the behavior with Safari 4, and they result in this behavior if you're still using Safari 3.  You can trivially avoid this problem by updating to Safari 4.  I'll attempt to fix this next week when I have some free time.
Comment 2 Mark Rowe (bdash) 2009-06-15 00:09:25 PDT
Created attachment 31282 [details]
Patch
Comment 3 Cameron Zwarich (cpst) 2009-06-15 00:16:39 PDT
Comment on attachment 31282 [details]
Patch

"and having WebKitNightlyEnabler assume that it did"

What is the 'it' in the comment here? The trampoline detection tricks? Then it should be a plural.

Other than that, r=me.
Comment 4 Mark Rowe (bdash) 2009-06-15 00:38:08 PDT
Fixed with r44680.  I'll build a new nightly containing the fix soon.