Refactor HistoryService to not send NOTIFICATION_FAVICON_CHANGED

chrome/browser/history/DEPS

 include_rules = [
   # History is being made into a Browser Component, so we have these
   # basic rules followed by temporary exceptions.  Please don't
   # add to the list of exceptions!
   "-chrome/browser",
   "-components/search_engines",
   "+chrome/browser/chrome_notification_types.h",
   "+chrome/browser/common",
   "+chrome/browser/favicon",
   "+chrome/browser/history",
   "+chrome/browser/network_time",
   "+chrome/tools/profiles",  # For history unit tests.
   "+components/visitedlink/browser",
 
   # TODO(erikwright): Bring this list to zero.
   #
   # Do not add to the list of temporarily-allowed dependencies below,
   # and please do not introduce more #includes of these files.
   # http://crbug.com/370850
   "!chrome/browser/autocomplete/history_url_provider.h",
   "!chrome/browser/bookmarks/bookmark_model_factory.h",
+  "!chrome/browser/bookmarks/chrome_bookmark_client.h",
+  "!chrome/browser/bookmarks/chrome_bookmark_client_factory.h",
   "!chrome/browser/browser_process.h",
   "!chrome/browser/chromeos/login/existing_user_controller.h",
   "!chrome/browser/content_settings/cookie_settings.h",
   "!chrome/browser/diagnostics/sqlite_diagnostics.h",
   "!chrome/browser/omnibox/omnibox_field_trial.h",
   "!chrome/browser/prefs/scoped_user_pref_update.h",
   "!chrome/browser/prerender/prerender_contents.h",
   "!chrome/browser/prerender/prerender_manager.h",
   "!chrome/browser/prerender/prerender_manager_factory.h",
   "!chrome/browser/profiles/incognito_helpers.h",
@@ skipping 26 matching lines @@
   ],
   # TODO(sdefresne): Bring this list to zero. http://crbug.com/370850
   #
   # Do not add to the list of temporarily-allowed dependencies below,
   # and please do not introduce more #includes of these files.
   '.*_[a-z]*test\.cc': [
     "!components/bookmarks/test/bookmark_test_helpers.h",
     "!components/bookmarks/test/test_bookmark_client.h",
   ]
 }