Collapse images disallowed by the Safe Browsing Subresource Filter.

third_party/WebKit/Source/core/fetch/ImageResource.cpp

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
     Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
@@ skipping 69 matching lines @@
       WebURLRequest::RequestContextUnspecified) {
     request.mutableResourceRequest().setRequestContext(
         WebURLRequest::RequestContextImage);
   }
   if (fetcher->context().pageDismissalEventBeingDispatched()) {
     KURL requestURL = request.resourceRequest().url();
     if (requestURL.isValid() &&
         fetcher->context().canRequest(Resource::Image,
                                       request.resourceRequest(), requestURL,
                                       request.options(), request.forPreload(),
-                                      request.getOriginRestriction()))
+                                      request.getOriginRestriction()) ==
+            ResourceRequestBlockedReason::None) {

[Mike West, 2016/11/30 15:57:35]

Nit: this is pretty ugly, but I'm not sure I have a better solution.

[engedy, 2016/11/30 17:37:39]

Introduced a helper variable, not sure it helped much.

       fetcher->context().sendImagePing(requestURL);
+    }
     return nullptr;
   }
 
   ImageResource* resource = toImageResource(
       fetcher->requestResource(request, ImageResourceFactory(request)));
   if (resource &&
       request.placeholderImageRequestType() != FetchRequest::AllowPlaceholder &&
       resource->m_isPlaceholder) {
     // If the image is a placeholder, but this fetch doesn't allow a
     // placeholder, then load the original image. Note that the cache is not
@@ skipping 616 matching lines @@
            WebServiceWorkerResponseTypeOpaque;
   }
   if (!getImage()->currentFrameHasSingleSecurityOrigin())
     return false;
   if (passesAccessControlCheck(securityOrigin))
     return true;
   return !securityOrigin->taintsCanvas(response().url());
 }
 
 }  // namespace blink