Bug 14964 - "Safari Extensions Detected" Warning needs to be clearer as to where plugins are installed
Summary: "Safari Extensions Detected" Warning needs to be clearer as to where plugins ...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.4
: P4 Enhancement
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-14 06:50 PDT by Dan Wood
Modified: 2008-10-23 11:11 PDT (History)
3 users (show)

See Also:


Attachments
screenshot of alert (33.33 KB, image/png)
2007-08-14 06:51 PDT, Dan Wood
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Wood 2007-08-14 06:50:28 PDT
When launching the nightly webkit build when some sort of Safari extensions are in place, there is a warning with the title "Safari Extensions Detected" that indicates that there may be crashes or incorrect behavior.

It would be very helpful to actually  list the extensions, by name or file path, in this alert.  Personally, I have had trouble hunting down these extensions (since they are installed in a variety of places).

I'm not sure if this alert is actually a function of Safari rather than WebKit, but since the text mentions nightly builds, I'm guessing that the text is at least generated by WebKit.
Comment 1 Dan Wood 2007-08-14 06:51:15 PDT
Created attachment 15959 [details]
screenshot of alert
Comment 2 Mark Rowe (bdash) 2007-08-14 07:33:40 PDT
The FAQ at <http://trac.webkit.org/projects/webkit/wiki/FAQ> lists the paths that contain extensions.  I'm not really sure how the dialog could be clearer without being made more confusing.  Suggestions are welcome.
Comment 3 Dan Wood 2007-08-14 07:41:49 PDT
Perhaps if the following text were added:

For more information, please see the FAQ at http://trac.webkit.org/projects/webkit/wiki/FAQ



(Perhaps a smaller URL could be made to redirect there, since the user will have to type in that URL by hand)
Comment 4 Brad 2007-11-17 12:16:49 PST
"Suggestions are welcome."

OK, please have a scrolling list of the possible offenders, and then open up their containing folder when you double click on them. I keep getting that message even though I am fairly certain I have removed all of my Safari extensions.
Comment 5 Mark Rowe (bdash) 2007-11-17 12:28:33 PST
That would require adding custom UI which is not at all desirable given the sensitive time that the dialog is displayed.  As mentioned in comment #2, http://trac.webkit.org/projects/webkit/wiki/FAQ lists the paths that are flagged as extension-carrying.  Any bundles detected in the process from those paths will trigger the dialog after a crash.
Comment 6 Brad 2007-11-17 13:14:51 PST
Any bundles at all? Not just "Safari Extensions" (those extensions designed specifically to extend Safari)? I don't have any of what I would call Safari Extensions, but perhaps it is triggered by "Smart Crash Reports" or "Detour.ape"?
Comment 7 Mark Rowe (bdash) 2007-11-17 13:19:58 PST
Yes, any code loaded from those locations.  Arbitrary code running inside Safari can do arbitrary things.  When running a development build of software that the extension developer is unlikely to have tested, the most common thing that arbitrary third-party code running inside Safari will do is crash.
Comment 8 Brad 2007-11-18 00:37:19 PST
Ah. Then I suggest you don't call them "Safari Extensions" in the alert box then. It is misleading. That is my new suggestion, since you didn't like the other one.
Comment 9 Mark Rowe (bdash) 2007-11-18 00:38:40 PST
Third-party code that is in Safari's address space is by definition extending Safari.
Comment 10 Stephen 2008-05-09 10:28:49 PDT
(In reply to comment #9)
> Third-party code that is in Safari's address space is by definition extending
> Safari.
> 

You seem set on replying in technical ways, not in ways that assist users in understanding what's actually going on in their computers using Webkit.  Whether something is technically "extending Safari" is not the issue to me. The issue is how Webkit can be happily and efficiently used by a non-technical user such as myself and apparently the other people who posted here.   
Comment 11 Harold 2008-07-01 22:20:31 PDT
OK, I went to all the 'library' files you indicated. Turned off the extensions. STILL GET THE SAME WARNING!
What next?
Comment 12 Mark Rowe (bdash) 2008-07-01 22:24:30 PDT
Open Console.app and look for the "Extension detected" messages.
Comment 13 Eric Welch 2008-10-23 07:19:47 PDT
How about letting us determine which extension does not crash Webkit and selecting it to not be turned off next time Webkit decides it doesn't like its presence, even though it's working perfectly? Mine (I only have one add-on) is 1Password. It works just fine with Webkit, except for when a new build is installed. It inevitably locks up until it crashes, and then I reinstall the plug-in and it works fine until a new build objects to its presence and locks up.
Comment 14 Mark Rowe (bdash) 2008-10-23 11:11:02 PDT
(In reply to comment #13)
> How about letting us determine which extension does not crash Webkit and
> selecting it to not be turned off next time Webkit decides it doesn't like its
> presence, even though it's working perfectly? Mine (I only have one add-on) is
> 1Password. It works just fine with Webkit, except for when a new build is
> installed. It inevitably locks up until it crashes, and then I reinstall the
> plug-in and it works fine until a new build objects to its presence and locks
> up.

WebKit doesn't turn off any extensions.  It merely raises them as a potential problem point if Safari crashes rather than exiting cleanly.