[Webkit-unassigned] [Bug 24529] WebInspector: HTML5 Offline Web Applications Support (ApplicationCache)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 7 22:32:31 PDT 2010


https://bugs.webkit.org/show_bug.cgi?id=24529





--- Comment #74 from Pavel Feldman <pfeldman at chromium.org>  2010-07-07 22:32:30 PST ---
Reading through the thread, I can say few things:
- It is a shame ApplicationCache.h was used, but there was really no sign or comment preventing it from being used. I know it was a tough decision and there probably is a handful of people involved in that decision that know that the Host is the only touchable thing. There is no one to blame here.
- It is a shame that the patch was compiled from parts of Kavita work - we all like when the credit is shared fairly.
- However, I agree with Joe that we can't land when everything is ready. Implementation is incremental and as long as it does not break the build (is hidden), we are good with it.

So please, lets get back to constructive way of going further.

Here is what I would suggest:
1) For all the items Michael enumerated in #72, please file the bugs
2) Lets hide appcache from the UI unless it is ready and both parities are happy with it
3) Lets fix chromium fast so that we could iterate against working appcache inspection
4) Lets make sure ApplicationCacheHost.h has a nice interface that Inspector could talk to and move ApplicationCache.h (and rest of the implementation classes) to under WebCore/loader/appcache/internal.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list