Bug 60799

Summary: Hide appcache status bar items
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: Web Inspector (Deprecated)Assignee: Alexey Proskuryakov <ap>
Status: RESOLVED FIXED    
Severity: Normal CC: ademar, ddkilzer, joepeck, michaeln, pfeldman, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
proposed patch joepeck: review+

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>