New issue
Advanced search Search tips

Issue 608997 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

DevTools: new service workers pane design.

Project Member Reported by pfeldman@chromium.org, May 4 2016

Issue description

See image attached.
 
Screen Shot 2016-05-03 at 8.04.22 PM.png
244 KB View Download
Can you provide a screenshot with other worker states?
Project Member

Comment 3 by bugdroid1@chromium.org, May 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6c01d8dcde776efc5b0cd101aa80f8175153b02d

commit 6c01d8dcde776efc5b0cd101aa80f8175153b02d
Author: pfeldman <pfeldman@chromium.org>
Date: Thu May 05 05:24:31 2016

DevTools: extract the report view to reuse in app manifest and service worker pane.

BUG= 608997 

Review-Url: https://codereview.chromium.org/1952933002
Cr-Commit-Position: refs/heads/master@{#391766}

[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/devtools.gypi
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/components/Linkifier.js
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/resources/AppManifestView.js
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/resources/appManifestView.css
[add] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/ui/ReportView.js
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/ui/inspectorCommon.css
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/ui/inspectorSyntaxHighlight.css
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/ui/module.json
[add] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/ui/reportView.css
[modify] https://crrev.com/6c01d8dcde776efc5b0cd101aa80f8175153b02d/third_party/WebKit/Source/devtools/front_end/ui/toolbar.css

Project Member

Comment 4 by bugdroid1@chromium.org, May 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f60030d12c0738084bac0a563cfc6edab2bf5b40

commit f60030d12c0738084bac0a563cfc6edab2bf5b40
Author: pfeldman <pfeldman@chromium.org>
Date: Thu May 05 06:54:48 2016

DevTools: simplify Service Workers panel UI, remove non-actionable data.

BUG= 608997 

Review-Url: https://codereview.chromium.org/1940993002
Cr-Commit-Position: refs/heads/master@{#391774}

[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/LayoutTests/http/tests/inspector/service-workers/service-workers-test.js
[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/LayoutTests/http/tests/inspector/service-workers/service-workers-view-expected.txt
[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/LayoutTests/http/tests/inspector/service-workers/service-workers-view.html
[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/Source/devtools/front_end/resources/ResourcesPanel.js
[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkersView.js
[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/Source/devtools/front_end/resources/resourcesSidebar.css
[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/Source/devtools/front_end/resources/serviceWorkersView.css
[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/Source/devtools/front_end/sdk/ServiceWorkerManager.js
[modify] https://crrev.com/f60030d12c0738084bac0a563cfc6edab2bf5b40/third_party/WebKit/Source/devtools/front_end/security/originView.css

Project Member

Comment 5 by bugdroid1@chromium.org, May 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/19356078a3cf2a519b821746ae5d10466b2dae9c

commit 19356078a3cf2a519b821746ae5d10466b2dae9c
Author: pfeldman <pfeldman@chromium.org>
Date: Thu May 05 22:02:42 2016

DevTools: follow up to v391774 with small UI fixes.

TBR=caseq
BUG= 608997 

Review-Url: https://codereview.chromium.org/1946333005
Cr-Commit-Position: refs/heads/master@{#391921}

[modify] https://crrev.com/19356078a3cf2a519b821746ae5d10466b2dae9c/third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkersView.js

Status: Fixed (was: Assigned)

Sign in to add a comment