Bug 28087 – [Chromium] Mods for a chrome specific impl of ApplicationCacheHost

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Michael Nordman]

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.

[Michael Nordman]

Created attachment 34348 [details]
use PLATFORM(CHROMIUM) to segregate the private section

[Michael Nordman]

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.

[Jeremy Orlow]

Will land by hand in a sec.

[Jeremy Orlow]

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