Move favicon callbacks to favicon_base

Move FaviconImageCallback, FaviconResultsCallback and FaviconRawCallback
to favicon_base, those types are shared between favicon and history
components (methods are declared in FaviconService, but it delegates the
work to HistoryService) and only uses types from base:: and
favicon_base:: namespaces.

Also BookmarkClient::FaviconImageCallback is a copy of
FaviconService::FaviconImageCallback, replace it with favicon_base::
version.

BUG=372321
TBR=thakis for //chrome outside of //chrome/browser/history,
//chrome/browser/favicon

Review URL: https://codereview.chromium.org/291643002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272065 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/extensions/extension_web_ui.h b/chrome/browser/extensions/extension_web_ui.h
index 4bd2cc1..110ef10 100644
--- a/chrome/browser/extensions/extension_web_ui.h
+++ b/chrome/browser/extensions/extension_web_ui.h
@@ -8,8 +8,8 @@
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
-#include "chrome/browser/favicon/favicon_service.h"
 #include "chrome/common/extensions/manifest_url_handler.h"
+#include "components/favicon_base/favicon_callback.h"
 #include "content/public/browser/web_ui_controller.h"
 
 class Profile;
@@ -67,7 +67,7 @@
   static void GetFaviconForURL(
       Profile* profile,
       const GURL& page_url,
-      const FaviconService::FaviconResultsCallback& callback);
+      const favicon_base::FaviconResultsCallback& callback);
 
  private:
   // Unregister the specified override, and if it's the currently active one,