Pass the ImageDecoder to the ImageFetcher at creation time.

components/image_fetcher/image_fetcher_impl.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chrome/browser/search/suggestions/image_fetcher_impl.h"
+#include "components/image_fetcher/image_fetcher_impl.h"
 
 #include <string>
 
 #include "base/bind.h"
-#include "chrome/browser/search/suggestions/image_decoder_impl.h"
-#include "content/public/browser/browser_thread.h"
 #include "net/base/load_flags.h"
 #include "net/url_request/url_request_context_getter.h"
-#include "ui/gfx/image/image.h"
 
-namespace suggestions {
+namespace image_fetcher {
 
 ImageFetcherImpl::ImageFetcherImpl(
+    std::unique_ptr<image_fetcher::ImageDecoder> image_decoder,
     net::URLRequestContextGetter* url_request_context)
     : delegate_(nullptr), url_request_context_(url_request_context),
-      image_decoder_(new suggestions::ImageDecoderImpl()),
+      image_decoder_(std::move(image_decoder)),
       image_data_fetcher_(
-          new image_fetcher::ImageDataFetcher(url_request_context_)) {
+          new image_fetcher::ImageDataFetcher(url_request_context_.get())) {

[Marc Treib, 2016/06/29 14:41:33]

nit: "image_fetcher::" isn't required anymore :)

[markusheintz_, 2016/06/29 14:56:24]

Done. Updated here and and in the rest of the file as well as in the header.

 }
 
 ImageFetcherImpl::~ImageFetcherImpl() {}
 
 ImageFetcherImpl::ImageRequest::ImageRequest() {}
 
 ImageFetcherImpl::ImageRequest::ImageRequest(const ImageRequest& other) =
     default;
 
 ImageFetcherImpl::ImageRequest::~ImageRequest() { }
@@ skipping 56 matching lines @@
 
   // Inform the ImageFetcherDelegate.
   if (delegate_) {
     delegate_->OnImageFetched(request->id, image);
   }
 
   // Erase the completed ImageRequest.
   pending_net_requests_.erase(image_iter);
 }
 
-}  // namespace suggestions
+}  // namespace image_fetcher