ResourceLoader is the owner of the resource data and we don't really want to transfer the ownership in resourceData().
Created attachment 203924 [details]
Comment on attachment 203924 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=203924&action=review
> + Change ResourceLoader::resourceData() to return a raw pointer
> + instead of a PassRefPtr since we don't want to transfer the
> + ownership of the resource data. Also pass a raw pointer to
Definitely a good change.
> + CachedResource::data() so that the cached resource can decide
> + whether to take a reference of the data or not, for example,
> + CachedRawResource only takes a reference when buffering policy is
> + BufferData.
Not sure this change is as good, but it’s OK.
Committed r151277: <http://trac.webkit.org/changeset/151277>