Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(374)

Unified Diff: components/ntp_tiles/webui/ntp_tiles_internals_message_handler.h

Issue 2936793002: ntp_tiles: Extend chrome://ntp-tiles-internals with favicon data (Closed)
Patch Set: Avoid using std::pair with constexpr. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/ntp_tiles/webui/ntp_tiles_internals_message_handler.h
diff --git a/components/ntp_tiles/webui/ntp_tiles_internals_message_handler.h b/components/ntp_tiles/webui/ntp_tiles_internals_message_handler.h
index 03b58c40f5240dd86bc45ccdb9a5409af061be5b..4306f8326924d0be8e59c6f5b76c045d2211d086 100644
--- a/components/ntp_tiles/webui/ntp_tiles_internals_message_handler.h
+++ b/components/ntp_tiles/webui/ntp_tiles_internals_message_handler.h
@@ -5,18 +5,25 @@
#ifndef COMPONENTS_NTP_TILES_WEBUI_NTP_TILES_INTERNALS_MESSAGE_HANDLER_H_
#define COMPONENTS_NTP_TILES_WEBUI_NTP_TILES_INTERNALS_MESSAGE_HANDLER_H_
+#include <map>
#include <memory>
#include <string>
#include "base/macros.h"
#include "base/memory/weak_ptr.h"
#include "base/optional.h"
+#include "base/task/cancelable_task_tracker.h"
+#include "components/favicon_base/favicon_types.h"
#include "components/ntp_tiles/most_visited_sites.h"
namespace base {
class ListValue;
} // namespace base
+namespace favicon {
+class FaviconService;
+} // namespace favicon
+
namespace ntp_tiles {
class MostVisitedSites;
@@ -29,7 +36,9 @@ class NTPTilesInternalsMessageHandlerClient;
// to the embedder's API. It cannot itself implement either API directly.
class NTPTilesInternalsMessageHandler : public MostVisitedSites::Observer {
public:
- NTPTilesInternalsMessageHandler();
+ // |favicon_service| must not be null and must outlive this object.
+ explicit NTPTilesInternalsMessageHandler(
+ favicon::FaviconService* favicon_service);
~NTPTilesInternalsMessageHandler() override;
// Called when the WebUI page's JavaScript has loaded and it is ready to
@@ -37,6 +46,9 @@ class NTPTilesInternalsMessageHandler : public MostVisitedSites::Observer {
void RegisterMessages(NTPTilesInternalsMessageHandlerClient* client);
private:
+ using FaviconResultMap = std::map<std::pair<GURL, favicon_base::IconType>,
+ favicon_base::FaviconRawBitmapResult>;
+
// Callbacks registered in RegisterMessages().
void HandleRegisterForEvents(const base::ListValue* args);
void HandleUpdate(const base::ListValue* args);
@@ -44,12 +56,21 @@ class NTPTilesInternalsMessageHandler : public MostVisitedSites::Observer {
void HandleViewPopularSitesJson(const base::ListValue* args);
void SendSourceInfo();
- void SendTiles(const NTPTilesVector& tiles);
+ void SendTiles(const NTPTilesVector& tiles,
+ const FaviconResultMap& result_map);
// MostVisitedSites::Observer.
void OnMostVisitedURLsAvailable(const NTPTilesVector& tiles) override;
void OnIconMadeAvailable(const GURL& site_url) override;
+ void OnFaviconLookupDone(const NTPTilesVector& tiles,
+ FaviconResultMap* result_map,
+ size_t* num_pending_lookups,
+ const GURL& page_url,
+ const favicon_base::FaviconRawBitmapResult& result);
+
+ favicon::FaviconService* favicon_service_;
+
// Bridge to embedder's API.
NTPTilesInternalsMessageHandlerClient* client_;
@@ -59,6 +80,7 @@ class NTPTilesInternalsMessageHandler : public MostVisitedSites::Observer {
std::string suggestions_status_;
std::string popular_sites_json_;
+ base::CancelableTaskTracker cancelable_task_tracker_;
base::WeakPtrFactory<NTPTilesInternalsMessageHandler> weak_ptr_factory_;
DISALLOW_COPY_AND_ASSIGN(NTPTilesInternalsMessageHandler);
« no previous file with comments | « chrome/browser/ui/webui/ntp_tiles_internals_ui.cc ('k') | components/ntp_tiles/webui/ntp_tiles_internals_message_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698