[Suggestions] Move ImageManager to the component

components/suggestions/image_fetcher.h

+// 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.
+
+#ifndef COMPONENTS_SUGGESTIONS_IMAGE_FETCHER_H_
+#define COMPONENTS_SUGGESTIONS_IMAGE_FETCHER_H_
+
+#include "base/callback.h"
+#include "components/suggestions/image_fetcher_delegate.h"
+#include "url/gurl.h"
+
+class SkBitmap;
+
+namespace suggestions {
+
+// A class used to fetch server images.
+class ImageFetcher {
+ public:
+  ImageFetcher() {}
+  virtual ~ImageFetcher() {}
+
+  virtual void SetImageFetcherDelegate(ImageFetcherDelegate* delegate) = 0;

[huangs, 2014/09/24 15:46:57]

Can forward-declare ImageFetcherDelegate (and also URL) and remove #include.

[Mathieu, 2014/09/24 19:31:30]

Turns out I can't do this because of the mock in image_manager_unittest

+
+  virtual void StartOrQueueNetworkRequest(
+      const GURL& url, const GURL& image_url,
+      base::Callback<void(const GURL&, const SkBitmap*)> callback) = 0;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ImageFetcher);
+};
+
+}  // namespace suggestions
+
+#endif  // COMPONENTS_SUGGESTIONS_IMAGE_FETCHER_H_