The private section of class ApplicationCacheHost currently contains members that aren't applicable to chrome's impl, and is lacking other members that are needed. We need to carve out some space in this class to share the public part of this interface and not step on each other in the private section.
Created attachment 34348 [details] use PLATFORM(CHROMIUM) to segregate the private section
Fyi: I've intentionally chosen to use the PLATFORM flag here to help clarify what parts of the appcache system are actually in the CHROMIUM port. The majority of the appcache directory is not, a few files are... * ApplicationCacheHost.h * DOMApplicationCache.idl .h .cpp Using the PLATFORM() flag in this .h file is reflective of that, and while it may not make it crystal clear what classes/files are relevant to CHROMIUM, it does provide big clues that ApplicationCacheGroup and familiy don't apply. My thinking is that having this reflected in the .h file helps with maintainability. I'd be willing to shuffle some more deck chairs around to create a ApplicationCacheHostInternal class that inter-operates with ApplicationCacheGroup and family (and removes the PLATFORM flag), but for the reason stated above, I don't think that would be an improvement.
Will land by hand in a sec.
Sending WebCore/ChangeLog Sending WebCore/loader/appcache/ApplicationCacheHost.cpp Sending WebCore/loader/appcache/ApplicationCacheHost.h Sending WebCore/page/DOMWindow.cpp Transmitting file data .... Committed revision 47018. http://trac.webkit.org/changeset/47018