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

Unified Diff: components/favicon.gypi

Issue 1064823002: Componentize FaviconTabHelper (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@favicon-notification
Patch Set: Fix BUILD.gn Created 5 years, 8 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
« no previous file with comments | « chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc ('k') | components/favicon/DEPS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/favicon.gypi
diff --git a/components/favicon.gypi b/components/favicon.gypi
index f2c9cf54b7459b8b634d8f0349a052b64698f796..d150316eb8eb77d08090911ee90330dbf3a2514e 100644
--- a/components/favicon.gypi
+++ b/components/favicon.gypi
@@ -9,7 +9,9 @@
'target_name': 'favicon_core',
'type': 'static_library',
'dependencies': [
+ '../base/base.gyp:base',
'../skia/skia.gyp:skia',
+ '../ui/base/ui_base.gyp:ui_base',
'../ui/gfx/gfx.gyp:gfx',
'../url/url.gyp:url_lib',
'bookmarks_browser',
@@ -23,7 +25,10 @@
'favicon/core/fallback_icon_service.cc',
'favicon/core/fallback_icon_service.h',
'favicon/core/favicon_client.h',
+ 'favicon/core/favicon_driver.cc',
'favicon/core/favicon_driver.h',
+ 'favicon/core/favicon_driver_impl.cc',
+ 'favicon/core/favicon_driver_impl.h',
'favicon/core/favicon_driver_observer.h',
'favicon/core/favicon_handler.cc',
'favicon/core/favicon_handler.h',
@@ -45,13 +50,17 @@
'target_name': 'favicon_content',
'type': 'static_library',
'dependencies': [
+ '../base/base.gyp:base',
'../content/content.gyp:content_browser',
'../content/content.gyp:content_common',
+ '../ui/gfx/gfx.gyp:gfx',
'favicon_base',
'favicon_core',
],
'sources': [
# Note: sources list duplicated in GN build.
+ 'favicon/content/content_favicon_driver.cc',
+ 'favicon/content/content_favicon_driver.h',
'favicon/content/favicon_url_util.cc',
'favicon/content/favicon_url_util.h',
],
@@ -67,13 +76,17 @@
'target_name': 'favicon_ios',
'type': 'static_library',
'dependencies': [
+ '../base/base.gyp:base',
'../ios/web/ios_web.gyp:ios_web',
+ '../ui/gfx/gfx.gyp:gfx',
'favicon_base',
'favicon_core',
],
'sources': [
- 'favicon/ios/favicon_url_util.h',
'favicon/ios/favicon_url_util.cc',
+ 'favicon/ios/favicon_url_util.h',
+ 'favicon/ios/web_favicon_driver.cc',
+ 'favicon/ios/web_favicon_driver.h',
],
'include_dirs': [
'..',
« no previous file with comments | « chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc ('k') | components/favicon/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698