Refactoring starting a resource load

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

Index: third_party/WebKit/Source/core/fetch/ImageResource.cpp
diff --git a/third_party/WebKit/Source/core/fetch/ImageResource.cpp b/third_party/WebKit/Source/core/fetch/ImageResource.cpp
index e8462ed803e19d9d404dc00a3218161ea093dc92..6cfc9d0f581330d8118958a7072abab403543edc 100644
--- a/third_party/WebKit/Source/core/fetch/ImageResource.cpp
+++ b/third_party/WebKit/Source/core/fetch/ImageResource.cpp
@@ -503,7 +503,7 @@ void ImageResource::reloadIfLoFi(ResourceFetcher* fetcher)
     m_resourceRequest.setLoFiState(WebURLRequest::LoFiOff);
     error(Resource::LoadError);
     setStatus(NotStarted);
-    load(fetcher);
+    fetcher->startLoad(this);

[Yoav Weiss, 2016/04/20 08:52:08]

Can you DCHECK that fetcher is non-null here?

[Nate Chapin, 2016/05/18 23:56:04]

This is only called by ResourceFetcher at the moment, FWIW. I'm not opposed to
the DCHECK, but given that none of the FooResource::fetch() methods null-check
their ResourceFethcer*, I'm not sure how important it is.

 }
 
 void ImageResource::changedInRect(const blink::Image* image, const IntRect& rect)