[DevTools] Move opaque response explanatory link to 'clear storage' view
The original palce of that link is in service worker cache table, but
it makes more sense to have that link in the 'clear cache' view, since
the user is probably looking at the latter when they got an exception
about cache quota exceeding, which is probably caused by caching an
opaque response (UI meeting 11/07).
Bug: 847462
Change-Id: I99b039fc5b9617c1d0f4493dc3accce32d0385b3
Reviewed-on: https://chromium-review.googlesource.com/c/1325083
Reviewed-by: Erik Luo <[email protected]>
Commit-Queue: Haihong Li (Harley) <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#612425}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: f9b30cc6c37d8794d896e9f2260096628d1bb687
diff --git a/front_end/resources/ServiceWorkerCacheViews.js b/front_end/resources/ServiceWorkerCacheViews.js
index a478eb0..1b43324 100644
--- a/front_end/resources/ServiceWorkerCacheViews.js
+++ b/front_end/resources/ServiceWorkerCacheViews.js
@@ -358,19 +358,12 @@
if (columnId === 'path') {
value = this._path;
} else if (columnId === 'responseType') {
- if (this._responseType === 'opaqueResponse') {
- const opaque = UI.XLink.create(
- 'https://developers.google.com/web/tools/chrome-devtools/progressive-web-apps#opaque-responses',
- ls`opaque`);
- opaque.title = ls`As a security consideration, an opaque response potentially takes ` +
- ls`up far more cache space than its content length`;
- cell.appendChild(opaque);
- return cell;
- } else if (this._responseType === 'opaqueRedirect') {
+ if (this._responseType === 'opaqueResponse')
+ value = 'opaque';
+ else if (this._responseType === 'opaqueRedirect')
value = 'opaqueredirect';
- } else {
+ else
value = this._responseType;
- }
} else if (columnId === 'contentType') {
value = this._request.mimeType;
} else if (columnId === 'contentLength') {