Bug 70815

Summary:

Web Inspector: redesign workers sidebar pane

Product:

WebKit

Reporter:

Yury Semikhatsky <yurys>

Component:

Web Inspector (Deprecated)

Assignee:

Yury Semikhatsky <yurys>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

apavlov, bweinstein, dglazkov, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

58541

Attachments:

Description	Flags
Patch	none
Screenshot with the patch applied	none
Patch	pfeldman: review+
Screenshot with the patch applied	none
Patch for landing(layout tests fixed)	none

Yury Semikhatsky

Reported 2011-10-25 06:24:08 PDT

There should be a checkbox that allows starting workers paused, link to a page with a list of all shared workers and a link for each dedicated worker created by inspected page. Clicking on link should open/bring to front a dedicated worker inspector.

Attachments
Patch (15.46 KB, patch) 2011-10-25 06:39 PDT, Yury Semikhatsky	no flags	Details Formatted Diff Diff
Screenshot with the patch applied (9.68 KB, image/png) 2011-10-25 06:40 PDT, Yury Semikhatsky	no flags	Details
Patch (15.50 KB, patch) 2011-10-25 07:47 PDT, Yury Semikhatsky	pfeldman: review+	Details Formatted Diff Diff
Screenshot with the patch applied (9.06 KB, image/png) 2011-10-25 07:48 PDT, Yury Semikhatsky	no flags	Details
Patch for landing(layout tests fixed) (16.20 KB, patch) 2011-10-25 08:29 PDT, Yury Semikhatsky	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Yury Semikhatsky

Comment 1 2011-10-25 06:39:01 PDT

Created attachment 112330 [details] Patch

Yury Semikhatsky

Comment 2 2011-10-25 06:40:03 PDT

Created attachment 112331 [details] Screenshot with the patch applied

Pavel Feldman

Comment 3 2011-10-25 06:54:20 PDT

I'd call it "Discover shared workers" and "Inspect http:// ..."

WebKit Review Bot

Comment 4 2011-10-25 07:02:00 PDT

Comment on attachment 112330 [details] Patch Attachment 112330 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10214152 New failing tests: inspector/styles/styles-add-new-rule.html inspector/styles/styles-add-blank-property.html inspector/styles/styles-source-lines.html inspector/styles/parse-utf8-bom.html inspector/styles/styles-computed-trace.html inspector/styles/commit-selector.html inspector/elements/elements-panel-styles.html inspector/styles/styles-commit-editing.html inspector/styles/styles-disable-inherited.html inspector/styles/styles-iframe.html inspector/styles/styles-source-lines-inline.html inspector/styles/styles-add-invalid-property.html inspector/styles/show-user-agent-styles.html inspector/elements/elements-delete-inline-style.html inspector/styles/styles-cancel-editing.html

Yury Semikhatsky

Comment 5 2011-10-25 07:47:10 PDT

Created attachment 112335 [details] Patch

Yury Semikhatsky

Comment 6 2011-10-25 07:48:18 PDT

Created attachment 112336 [details] Screenshot with the patch applied

Yury Semikhatsky

Comment 7 2011-10-25 07:48:53 PDT

(In reply to comment #3) > I'd call it "Discover shared workers" and Done. > "Inspect http:// ..." Changed section separator title to "Dedicated worker inspectors"

Yury Semikhatsky

Comment 8 2011-10-25 08:29:46 PDT

Created attachment 112340 [details] Patch for landing(layout tests fixed)

Yury Semikhatsky

Comment 9 2011-10-25 08:35:18 PDT

Committed r98348: <http://trac.webkit.org/changeset/98348>

Note You need to log in before you can comment on or make changes to this bug.