|Summary:||Hide appcache status bar items|
|Product:||WebKit||Reporter:||Alexey Proskuryakov <ap>|
|Component:||Web Inspector (Deprecated)||Assignee:||Alexey Proskuryakov <ap>|
|Severity:||Normal||CC:||ademar, ddkilzer, joepeck, michaeln, pfeldman, yurys|
|Version:||528+ (Nightly build)|
Description Alexey Proskuryakov 2011-05-13 13:50:43 PDT
Data that this displayed in Web Inspector's Resources->Application Cache status bar is very misleading. I think that it is more likely to confuse developers than help them. See: http://webkit.org/b/41637 Web Inspector: Give Semantics to "Refresh" and "Delete" Buttons in ApplicationCache DataGrid http://webkit.org/b/60590 Application cache status always displayed as UNCACHED at first http://webkit.org/b/60793 Application cache status indicator gets stuck at DOWNLOADING after a failure Fixing these is not trivial, so I suggest just hiding the items for now. It seems that removing essentially dead code would not be helpful in this case. Thoughts?
Comment 1 Alexey Proskuryakov 2011-05-13 13:53:38 PDT
Created attachment 93504 [details] proposed patch
Comment 2 Joseph Pecoraro 2011-05-13 13:55:50 PDT
Comment on attachment 93504 [details] proposed patch Sorry, I originally added this and never had time to implement more of it. I am fine with hiding it.
Comment 3 Alexey Proskuryakov 2011-05-13 14:01:30 PDT
Thanks! We'll need to actually remove the code if it begins to rot before we can fix it. Committed <http://trac.webkit.org/changeset/86460>.
Comment 4 Ademar Reis 2011-05-17 11:11:40 PDT
Revision r86460 cherry-picked into qtwebkit-2.2 with commit fcd06b7 <http://gitorious.org/webkit/qtwebkit/commit/fcd06b7>