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:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
proposed patch	joepeck: review+

Alexey Proskuryakov

Reported 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?

Attachments
proposed patch (2.96 KB, patch) 2011-05-13 13:53 PDT, Alexey Proskuryakov	joepeck: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Alexey Proskuryakov

Comment 1 2011-05-13 13:53:38 PDT

Created attachment 93504 [details] proposed patch

Joseph Pecoraro

Comment 2 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.

Alexey Proskuryakov

Comment 3 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>.

Ademar Reis

Comment 4 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>

Note You need to log in before you can comment on or make changes to this bug.