Bug 82362

Summary: Web Inspector: Provide private -[DOMNode inspect]
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, sam, timothy, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
[PATCH] Proposed Patch
[PATCH] Proposed Patch none

Description Joseph Pecoraro 2012-03-27 11:52:41 PDT
Add -[DOMNode inspect] as a way to open a particular node in the inspector.
Comment 1 Joseph Pecoraro 2012-03-27 11:55:42 PDT
Created attachment 134111 [details]
[PATCH] Proposed Patch
Comment 2 Joseph Pecoraro 2012-03-27 11:56:11 PDT
Comment on attachment 134111 [details]
[PATCH] Proposed Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=134111&action=review

> Source/WebCore/bindings/objc/DOM.mm:49
> +#import "Page.h"
> +#import "InspectorController.h"

Oops, ordering.
Comment 3 Joseph Pecoraro 2012-03-27 11:57:17 PDT
Created attachment 134112 [details]
[PATCH] Proposed Patch
Comment 4 WebKit Review Bot 2012-03-27 13:08:23 PDT
Comment on attachment 134112 [details]
[PATCH] Proposed Patch

Clearing flags on attachment: 134112

Committed r112305: <http://trac.webkit.org/changeset/112305>
Comment 5 WebKit Review Bot 2012-03-27 13:08:28 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Sam Weinig 2012-03-27 15:53:33 PDT
Can this be done using IDL?
Comment 7 Joseph Pecoraro 2012-03-27 16:04:20 PDT
(In reply to comment #6)
> Can this be done using IDL?

Probably. Is there a way to specify that this IDL is for ObjC only, and not
meant to actually add something to the JS* interfaces? I did a brief search
through IDL and didn't find a keyword for this, but I'm pretty new to IDL.
If so, this could be added to Node.idl and we could implement Node::inspect.
Comment 8 Timothy Hatcher 2012-03-27 16:10:12 PDT
You can use #ifdefs to make it ObjC only.
Comment 9 Joseph Pecoraro 2012-03-27 16:11:10 PDT
Ahh, I think I found it. There is a bunch of this in other IDL files:

Comment 10 Joseph Pecoraro 2012-03-27 19:49:37 PDT
(In reply to comment #9)
> Ahh, I think I found it. There is a bunch of this in other IDL files:

Moving this to a new bug:
<http://webkit.org/b/82419> Web Inspector: Create -[DOMNode inspect] from IDL