Bug 60799 - Hide appcache status bar items
Summary: Hide appcache status bar items
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alexey Proskuryakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-13 13:50 PDT by Alexey Proskuryakov
Modified: 2011-05-17 11:11 PDT (History)
6 users (show)

See Also:


Attachments
proposed patch (2.96 KB, patch)
2011-05-13 13:53 PDT, Alexey Proskuryakov
joepeck: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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>