[Webkit-unassigned] [Bug 43924] Web Inspector: -[WebInspector attach] and detach should work

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=43924





--- Comment #12 from Pavel Feldman <pfeldman at chromium.org>  2010-08-15 00:04:39 PST ---
> > I am not sure we should distinguish between backend and front-end on
> > the WebKit embedder level. 
> 
> I agree, but that I figured distinction already exists because of InspectorClient
> and InspectorFrontendClient.
> 

My point was that we should distinguish between those on the WebKit level, but not pass distinction further to the embedder.

> My rationale for the extra class was that the extra class is an Obj-C
> wrapper around the C++ class. I thought this was a common thing to do, or
> at least I had seen the pattern so I felt I should emulate it. I'm fine with
> either approach. WebKit clients should still only see and use the
> "WebInspector" class. They don't have access to the WebInspectorFrontend.

It was unclear from the code: header is there, it does not look too much different from WebInspector itself. As a result, embedder might get confused. Bug given that we maintain close relationship with the embedder, I guess it is fine.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.