[Fallback Icons] Refactor FallbackIconService to be a BrowserContext-level singleton

components/favicon/core/fallback_icon_client.h

+// Copyright 2015 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_FAVICON_CORE_FALLBACK_ICON_CLIENT_H_
+#define COMPONENTS_FAVICON_CORE_FALLBACK_ICON_CLIENT_H_
+
+#include <string>
+#include <vector>
+
+#include "base/strings/string16.h"
+#include "components/keyed_service/core/keyed_service.h"
+
+class GURL;
+
+namespace favicon_base {
+class FallbackIconService;

[pkotwicz, 2015/03/27 03:52:09]

Can you remove the forward declaration?

[huangs, 2015/03/27 17:33:07]

Done.

+}
+
+// This class abstracts operations that depend on the embedder's environment,
+// e.g. Chrome.
+class FallbackIconClient : public KeyedService {
+ public:
+  // Returns a list of font names for fallback icon rendering.
+  virtual const std::vector<std::string>& GetFontNameList() const = 0;
+
+  // Returns the text to render in fallback icon for |url|.
+  virtual base::string16 GetFallbackIconText(const GURL& url) const = 0;
+
+ protected:
+  ~FallbackIconClient() override {}
+};
+
+#endif  // COMPONENTS_FAVICON_CORE_FALLBACK_ICON_CLIENT_H_