AppCache viewer in DevTools broken in M70 |
||||
Issue descriptionWhat steps will reproduce the problem? (1) Go to https://archive.jonathanstark.com/labs/app-cache-2b/ or any other https site that uses AppCache (2) Open DevTools and go to the Application tab (3) Click "Application Cache" on the left sidebar, under "Cache" What is the expected result? On M69, there is a dropdrown next to "Application Cache". It shows the manifest used and cached resources. What happens instead? On M70 and M71, there is no dropdown, and clicking "Application Cache" does nothing. Note that clicking on "Clear Storage" shows quota usage, so DevTools is aware of the quota change due to AppCache use. Please use labels and text to provide additional information. Setting to P2 because we recommend modern applications to use Service Worker instead of AppCache. If the code has bit-rotted and there aren't resources to fix it, I'd be fine with removing the AppCache view from DevTools altogether. That seems better than the current state, which suggests no AppCache is active.
,
Jan 2
,
Jan 4
I did a bisect that pointed to these changes, I will continue to investigate: https://chromium.googlesource.com/chromium/src/+log/304c2bc1d38d2332be13e488bda066c375b665ce..fd8eed831aaa5f64bca745b43539d5032029ae1e
,
Jan 4
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/82cbd9682a79279d3399ace1859c1abd55802ee5 commit 82cbd9682a79279d3399ace1859c1abd55802ee5 Author: Joey Arhar <jarhar@chromium.org> Date: Fri Jan 04 23:23:38 2019 [DevTools] Fix AppCache viewer window.applicationCache isn't available within DevTools after http://crrev.com/c/1160644/ so we can't use it for status constants. Bug: 892562 Change-Id: Icbd33c345003595f090558fdb2ce7312f54ba93a Reviewed-on: https://chromium-review.googlesource.com/c/1396615 Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Joey Arhar <jarhar@chromium.org> Cr-Commit-Position: refs/heads/master@{#620101} [modify] https://crrev.com/82cbd9682a79279d3399ace1859c1abd55802ee5/third_party/blink/renderer/devtools/front_end/resources/ApplicationCacheItemsView.js [modify] https://crrev.com/82cbd9682a79279d3399ace1859c1abd55802ee5/third_party/blink/renderer/devtools/front_end/resources/ApplicationCacheModel.js
,
Jan 4
,
Jan 7
Able to reproduce this issue on Mac OS 10.14 on the build without Fix 71.0.3573.0 and the issue is fixed on the latest M-73 build 73.0.3664.0. 1. Launched Chrome 2. Opened given url "https://archive.jonathanstark.com/labs/app-cache-2b/ " 3. Opened Devtools > Application Tab > Cache Observed there is a drop down next to the "Application Cache" Attached is the screen shot for reference. Hence adding TE verified labels. Thanks..! |
||||
►
Sign in to add a comment |
||||
Comment 1 by pwnall@chromium.org
, Jan 2