Start same-origin URLs with / in cache storage dev tools |
|||||
Issue descriptionIt'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.
,
Sep 1 2017
Fiiiine I'll give it a go
,
Sep 1 2017
(I'm actually pretty excited at giving it a go)
,
Oct 6 2017
@Jake: still excited ? :)
,
Dec 14 2017
,
Dec 15 2017
Ah sorry, didn't find the time for this 😞
,
Jul 21
,
Jul 21
,
Jul 28
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8ffd6173380d5327c0764a87ee548d4ef3603dd7 commit 8ffd6173380d5327c0764a87ee548d4ef3603dd7 Author: Xunran Ding <xunran.ding@samsung.com> Date: Sat Jul 28 01:54:34 2018 Start same-origin URLs with slash in cache storage dev tools Bug: 760102 Change-Id: I3811976622dcc06845ac970de7436d7f1fb6896e Reviewed-on: https://chromium-review.googlesource.com/1146122 Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Xunran Ding <xunran.ding@samsung.com> Cr-Commit-Position: refs/heads/master@{#578893} [modify] https://crrev.com/8ffd6173380d5327c0764a87ee548d4ef3603dd7/third_party/WebKit/LayoutTests/http/tests/devtools/cache-storage/cache-data-expected.txt [modify] https://crrev.com/8ffd6173380d5327c0764a87ee548d4ef3603dd7/third_party/WebKit/LayoutTests/http/tests/devtools/cache-storage/cache-deletion-expected.txt [modify] https://crrev.com/8ffd6173380d5327c0764a87ee548d4ef3603dd7/third_party/WebKit/LayoutTests/http/tests/devtools/cache-storage/cache-entry-deletion-expected.txt [modify] https://crrev.com/8ffd6173380d5327c0764a87ee548d4ef3603dd7/third_party/WebKit/LayoutTests/http/tests/devtools/cache-storage/cache-live-update-cache-content-expected.txt [modify] https://crrev.com/8ffd6173380d5327c0764a87ee548d4ef3603dd7/third_party/blink/renderer/devtools/front_end/resources/ServiceWorkerCacheViews.js
,
Sep 3
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by pfeldman@chromium.org
, Aug 30 2017Status: Assigned (was: Untriaged)