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

[bugzilla-daemon at webkit.org]

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





--- Comment #71 from Joseph Pecoraro <joepeck at webkit.org>  2010-07-07 19:33:39 PST ---
Created an attachment (id=60825)
 --> (https://bugs.webkit.org/attachment.cgi?id=60825)
[PATCH] Start of a Chromium Patch - Move Code Back to ApplicationCache

(In reply to comment #70)
> From what I read so far, there's a disconnect in agreeing about the way to proceed.
> Joe, needing patches for chromium to integrate inspector and appcache is not the
> issue. Just that the way you moved things from ACH to agent makes it harder for us
> to do it (because the way ACH is the only abstraction between chrome and webkit's
> appcache impl).

The move was recommended by a reviewer. I happen to agree with it. But it seems
there is more to this than meets the eye. I've mentioned this is an okay short term
solution, so putting a patch up to revert this decision can be done by anyone.


> Moving methods to ACH is a better approach than forking the agent.
> But I am ok with either of the solution.

I've attached a starting patch to get the Chromium team started doing this.
But if you take this approach, please file a bug about unforking or better
abstracting the different platform application caches.

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