Bug 51184

Summary: Web Inspector: remove unnecessary dependencies from InspectorBackendStub
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Ilya Tikhonovsky <loislo>
Severity: Normal CC: apavlov, bweinstein, commit-queue, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
[patch] first iteration. none

Description Ilya Tikhonovsky 2010-12-16 05:58:43 PST
InspectorBackendStub is a transport-only thing but it has dependencies with Callback.js and WebInspector namespace.
Callback can be integrated into InspectorBackendStub because it is the only client.

Without these dependencies InspectorBackendStub can be used as an API wrapper for Inspector Protocol.
Comment 1 Ilya Tikhonovsky 2010-12-16 06:04:13 PST
Created attachment 76763 [details]
[patch] first iteration.
Comment 2 Ilya Tikhonovsky 2010-12-17 00:30:06 PST
Comment on attachment 76763 [details]
[patch] first iteration.

Clearing flags on attachment: 76763

Committed r74236: <http://trac.webkit.org/changeset/74236>
Comment 3 Ilya Tikhonovsky 2010-12-17 00:30:18 PST
All reviewed patches have been landed.  Closing bug.
Comment 4 WebKit Commit Bot 2010-12-17 01:24:22 PST
The commit-queue encountered the following flaky tests while processing attachment 76763 [details]:

http/tests/misc/redirect.php bug 51238 (author: mjs@apple.com)
inspector/styles-new-API.html bug 50889 (authors: apavlov@chromium.org and pfeldman@chromium.org)
The commit-queue is continuing to process your patch.