[Resource Timing] Don't report resources with data: urls
Created attachment 179165 [details] Patch
Comment on attachment 179165 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=179165&action=review > Source/WebCore/loader/cache/CachedResourceLoader.cpp:709 > + if (resource && !resource->errorOccurred() && !resource->wasCanceled() && resource->response().isHTTP()) { Does this include HTTPS as well?
(In reply to comment #2) > (From update of attachment 179165 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=179165&action=review > > > Source/WebCore/loader/cache/CachedResourceLoader.cpp:709 > > + if (resource && !resource->errorOccurred() && !resource->wasCanceled() && resource->response().isHTTP()) { > > Does this include HTTPS as well? Yep.
Created attachment 179353 [details] Patch for landing
Comment on attachment 179353 [details] Patch for landing Clearing flags on attachment: 179353 Committed r137677: <http://trac.webkit.org/changeset/137677>
All reviewed patches have been landed. Closing bug.