Bug 80585

Summary:	Web Inspector: Unable to inspect the inspector in Safari
Product:	WebKit	Reporter:	Nikita Vasilyev <me>
Component:	Web Inspector (Deprecated)	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED WONTFIX
Severity:	Normal	CC:	apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority:	P2
Version:	528+ (Nightly build)
Hardware:	All
OS:	All

Description Nikita Vasilyev 2012-03-08 02:54:56 PST

There was a context menu with a "Inspect Element" item http://trac.webkit.org/attachment/wiki/HackingWebInspector/inspect.png
There is no context menu any more.

Alt+Cmd+I works in Chromium, but not in Safari.

Comment 1 Timothy Hatcher 2012-03-08 07:44:45 PST

We only show the context menu item in Debug and Release builds — not shipping Safari versions. It confuses users when it is there. And soon Command-Option-I will toggle the Inspector in Safari, so that isn't suitable.

Comment 2 Timothy Hatcher 2012-03-08 07:45:21 PST

Basically only developers need to do this, and showing it in Debug and Release builds is the right answer.

Comment 3 Nikita Vasilyev 2012-03-08 11:57:44 PST

Although I agree the context menu item shouldn’t be in shipping Safari version, I don’t see it in neither WebKit nightly nor ToT build that I run via Tools/Scripts/run-safari.

Comment 4 Timothy Hatcher 2012-03-08 12:11:34 PST

Ah, I was wrong. It is only allowed in Debug builds. Here is the code:

http://trac.webkit.org/browser/trunk/Source/WebKit2/UIProcess/WebInspectorProxy.cpp#L53

To enable it locally, do this in Terminal and relaunch Safari:

defaults write com.apple.Safari "__WebInspectorPageGroup__.WebKit2DeveloperExtrasEnabled" -bool true

Comment 5 Nikita Vasilyev 2012-03-08 12:34:36 PST

It works, thanks! I didn’t know there is such a difference between debug and release builds.