[webkit-reviews] review requested: [Bug 180292] [Web App Manifest] Support fetching the app manifest : [Attachment 328176] WIP Patch

[bugzilla-daemon at webkit.org]

David Quesada <david_quesada at apple.com> has asked  for review:
Bug 180292: [Web App Manifest] Support fetching the app manifest
https://bugs.webkit.org/show_bug.cgi?id=180292

Attachment 328176: WIP Patch

https://bugs.webkit.org/attachment.cgi?id=328176&action=review




--- Comment #1 from David Quesada <david_quesada at apple.com> ---
Created attachment 328176

  --> https://bugs.webkit.org/attachment.cgi?id=328176&action=review

WIP Patch

My WIP patch for review. This builds on 177973, so it doesn't apply right now,
but I thought I'd post the patch for feedback. Particularly with the
architecture of representing outstanding requests for the app manifest as
ApplicationManifestLoaders attached to the DocumentLoader. I didn't see any
precedent for a way to load subresources of a document on demand in a way they
could be exposed to an embedder, other than IconLoader, so I used that as
inspiration.