Summary: | Cannot remove/customize the "Inspect Element" contextual menu item | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ian Cely <ian> | ||||||||||||
Component: | Web Inspector (Deprecated) | Assignee: | Adam Roben (:aroben) <aroben> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | ||||||||||||||
Priority: | P2 | ||||||||||||||
Version: | 523.x (Safari 3) | ||||||||||||||
Hardware: | Mac | ||||||||||||||
OS: | OS X 10.4 | ||||||||||||||
Attachments: |
|
Description
Ian Cely
2007-06-22 17:34:41 PDT
We should have a way to let the client change this. But we also want the control to force it in there so debugging client apps can still be done, event if the UI delegate removes all menu items blindly. (In reply to comment #1) > We should have a way to let the client change this. But we also want the > control to force it in there so debugging client apps can still be done, event > if the UI delegate removes all menu items blindly. Perhaps this is as simple as adding the item to the menu before it is passed up to the UI delegate in Release builds, and adding it afterwards in Debug builds? Created attachment 15425 [details]
Patch 1/3
Adds Settings::developerMenuExtrasEnabled
Created attachment 15426 [details]
Patch 2/3
Remove ContextMenuClient::shouldIncludeInspectElementItem
Created attachment 15427 [details]
Patch 3/3
Pass the Inspect Element item up to the UI delegate.
I decided to fix a FIXME in ContextMenuController while I was at it (patches 1 and 2). Patch 3 actually fixes the bug. I suspect these patches cause Safari on Windows to not show the Inspect Element context menu item, though I haven't gotten to test this theory yet. Please don't land these patches yet. Created attachment 15553 [details]
Patch 1.5/3
Here's a pretty important patch that somehow got left out.
Created attachment 15555 [details]
new Patch 3/3
This patch fixes up Windows so that the Inspect Element item will still show up.
|