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

Unified Diff: chrome/common/extensions/api/schemas.gni

Issue 489153003: Split bundle generation steps so that API registration is generated in browser, not common. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: extra targets Created 6 years, 4 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: chrome/common/extensions/api/schemas.gni
diff --git a/chrome/common/extensions/api/schemas.gni b/chrome/common/extensions/api/schemas.gni
new file mode 100644
index 0000000000000000000000000000000000000000..1e4973a680683efb837eff328cc2c6bde44361a7
--- /dev/null
+++ b/chrome/common/extensions/api/schemas.gni
@@ -0,0 +1,49 @@
+# Copyright 2014 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.
+
+import("//build/config/features.gni")
+
+gypi_values = exec_script(
+ "//build/gypi_to_gn.py",
+ [ rebase_path("schemas.gypi") ],
+ "scope",
+ [ "schemas.gypi" ])
+
+# Common sources that are both bundled and compiled.
+if (is_android) {
+ # Should be eliminated. See crbug.com/305852.
+ sources = gypi_values.android_schema_files
+} else {
+ sources = gypi_values.main_schema_files
+ if (is_chromeos) {
+ sources += gypi_values.chromeos_schema_files
+ if (is_chrome_branded) {
+ sources += gypi_values.chromeos_branded_schema_files
+ }
+ }
+ if (enable_webrtc) {
+ sources += gypi_values.webrtc_schema_files
+ }
+}
+
+if (!is_android) {
+ uncompiled_sources = gypi_values.main_non_compiled_schema_files
+}
+
+# impl_dir = "//chrome/browser/extensions/api"
+root_namespace = "extensions::api::%(namespace)s"
+
+deps = [
+ # Different APIs include some headers from chrome/common that in turn
+ # include generated headers from these targets.
+ # TODO(brettw) this should be made unnecessary if possible.
+ "//components/copresence/proto",
+ "//components/metrics/proto",
+ "//content/public/browser",
+ "//skia",
+ "//sync",
+]
+if (is_chromeos) {
+ # deps += [ "<(DEPTH)/chrome/chrome.gyp:drive_proto" ] TODO)GYP)
scottmg 2014/08/25 17:09:01 (GYP)
+}

Powered by Google App Engine
This is Rietveld 408576698