From fee69b4b384e94f03cd7d4dadf3140dd91bb9029 Mon Sep 17 00:00:00 2001 From: sdefresne Date: Mon, 16 Mar 2015 09:56:03 -0700 Subject: Componentize FaviconService and FaviconHandler Move FaviconService and FaviconHandler into //components/favicon/core/browser as they have no problematic dependencies anymore. Update DEPS and dependencies of favicon component. BUG=359514,359513 Review URL: https://codereview.chromium.org/983043003 Cr-Commit-Position: refs/heads/master@{#320738} --- components/favicon.gypi | 30 ++++++++++++++++++++++++++++-- 1 file changed, 28 insertions(+), 2 deletions(-) (limited to 'components/favicon.gypi') diff --git a/components/favicon.gypi b/components/favicon.gypi index 262bd2b4..09a78f2 100644 --- a/components/favicon.gypi +++ b/components/favicon.gypi @@ -10,12 +10,11 @@ 'type': 'static_library', 'dependencies': [ '../ui/gfx/gfx.gyp:gfx_geometry', + '../url/url.gyp:url_lib', 'favicon_base', - 'keyed_service_core', ], 'sources': [ # Note: sources list duplicated in GN build. - 'favicon/core/browser/favicon_client.h', 'favicon/core/favicon_driver.h', 'favicon/core/favicon_url.cc', 'favicon/core/favicon_url.h', @@ -24,5 +23,32 @@ '..', ], }, + { + # GN version: //components/favicon/core/browser + 'target_name': 'favicon_core_browser', + 'type': 'static_library', + 'dependencies': [ + '../skia/skia.gyp:skia', + '../ui/gfx/gfx.gyp:gfx', + '../url/url.gyp:url_lib', + 'bookmarks_browser', + 'favicon_base', + 'favicon_core', + 'history_core_browser', + 'keyed_service_core', + ], + 'sources': [ + # Note: sources list duplicated in GN build. + 'favicon/core/browser/favicon_client.h', + 'favicon/core/browser/favicon_handler.cc', + 'favicon/core/browser/favicon_handler.h', + 'favicon/core/browser/favicon_service.cc', + 'favicon/core/browser/favicon_service.h', + 'favicon/core/browser/favicon_tab_helper_observer.h', + ], + 'include_dirs': [ + '..', + ], + }, ], } -- cgit v1.1