chrome/browser/google/DEPS - Issue 808253006: Remove the GoogleURLTracker infobar functionality entirely.

Unified Diff: chrome/browser/google/DEPS

Issue 808253006: Remove the GoogleURLTracker infobar functionality entirely. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Fix size_t -> int conversion (since I turned off the warning disable for it) Created 6 years 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/chromeos/customization/customization_wallpaper_downloader_browsertest.cc ('k') | chrome/browser/google/chrome_google_url_tracker_client.h » ('j') | chrome/browser/ui/navigation_correction_tab_observer.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/google/DEPS

diff --git a/chrome/browser/google/DEPS b/chrome/browser/google/DEPS

index f40a6cae47d9d12960b1c92f6463c96794f66bec..d1f1275030932d21402b04b945155042aae7a06c 100644

--- a/chrome/browser/google/DEPS

+++ b/chrome/browser/google/DEPS

@@ -1,7 +1,5 @@

specific_include_rules = {

# GoogleURLTracker can't use WebContents, since it is used by iOS.

- # If WebContents functionality is needed, add it to

- # GoogleURLTrackerNavigationHelperImpl.

'google_url_tracker\.': [

"-content/public/browser/web_contents.h",

Ilya Sherman 2015/01/06 00:36:28 Is any of this still needed now that the code has

Peter Kasting 2015/01/06 00:54:26 Can you clarify what the "now that the code has be

Ilya Sherman 2015/01/06 01:19:35 I guess to be clearer, this rule seems to now only

Peter Kasting 2015/01/06 01:27:51 Never mind, I was misunderstanding some things (an