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 | ||
Nikita Vasilyev
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.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Timothy Hatcher
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.
Timothy Hatcher
Basically only developers need to do this, and showing it in Debug and Release builds is the right answer.
Nikita Vasilyev
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.
Timothy Hatcher
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
Nikita Vasilyev
It works, thanks! I didn’t know there is such a difference between debug and release builds.