New issue
Advanced search Search tips

Issue 760102 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Start same-origin URLs with / in cache storage dev tools

Project Member Reported by jakearchibald@chromium.org, Aug 29 2017

Issue description

It's great that the origin is omitted for same-origin entries, but maybe they should start "/". This means they can be copy/pasted to caches.match() and you get the correct entry back.

Otherwise, it becomes a relative url and may not match.
 
Screen Shot 2017-08-29 at 14.36.22.png
373 KB View Download
Owner: jakearchibald@chromium.org
Status: Assigned (was: Untriaged)
That would be removing the line that currently trims it and updating the tests. Jake, are you up for the job? The line you would need to remove is currently being changed, but here is the pointer: https://chromium-review.googlesource.com/c/chromium/src/+/642452/4/third_party/WebKit/Source/devtools/front_end/resources/ServiceWorkerCacheViews.js#344
Fiiiine I'll give it a go
(I'm actually pretty excited at giving it a go)
@Jake: still excited ? :)
Owner: eostroukhov@chromium.org
Ah sorry, didn't find the time for this 😞
Owner: xunran.d...@samsung.com
Status: Started (was: Assigned)
Status: Fixed (was: Started)

Sign in to add a comment