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

Unified Diff: ios/chrome/browser/rlz/BUILD.gn

Issue 2443373002: Split //ios/chrome/browser in multiple targets. (Closed)
Patch Set: Rebase Created 4 years, 2 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: ios/chrome/browser/rlz/BUILD.gn
diff --git a/ios/chrome/browser/rlz/BUILD.gn b/ios/chrome/browser/rlz/BUILD.gn
new file mode 100644
index 0000000000000000000000000000000000000000..602e9177d2411ffa63c600b2eadf322b96a77f5a
--- /dev/null
+++ b/ios/chrome/browser/rlz/BUILD.gn
@@ -0,0 +1,21 @@
+# Copyright 2016 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+source_set("rlz") {
rohitrao (ping after 24h) 2016/10/25 19:48:53 Should this whole target be behind enable_rlz?
sdefresne 2016/10/25 21:10:23 No, in general we avoid putting targets behind if(
+ sources = [
+ "rlz_tracker_delegate_impl.cc",
+ "rlz_tracker_delegate_impl.h",
+ ]
+ deps = [
+ "//base",
+ "//components/omnibox/browser",
+ "//components/rlz",
+ "//components/search_engines",
+ "//ios/chrome/browser:browser_no_public_deps",
+ "//ios/chrome/browser/browser_state",
+ "//ios/chrome/browser/google",
+ "//ios/chrome/browser/search_engines",
+ "//ios/web",
+ ]
+}

Powered by Google App Engine
This is Rietveld 408576698