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

Side by Side Diff: chrome/common/extensions/api/BUILD.gn

Issue 1226353004: Generate all extension schema namespaces as "api" and instead vary the generated bundle names. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 5 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 unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/json_schema_api.gni") 5 import("//build/json_schema_api.gni")
6 import("schemas.gni") 6 import("schemas.gni")
7 7
8 assert(enable_extensions) 8 assert(enable_extensions)
9 9
10 # GYP version: chrome/common/extensions/api/api.gyp:chrome_api 10 # GYP version: chrome/common/extensions/api/api.gyp:chrome_api
11 json_schema_api("api") { 11 json_schema_api("api") {
12 schemas = true 12 schemas = true
13 bundle = true 13 bundle = true
14 bundle_name = "Chrome"
14 15
15 deps = schema_dependencies 16 deps = schema_dependencies
16 } 17 }
17 18
18 # GYP version: chrome/browser/extensions/api/api.gyp:chrome_api_registration 19 # GYP version: chrome/browser/extensions/api/api.gyp:chrome_api_registration
19 json_schema_api("api_registration") { 20 json_schema_api("api_registration") {
20 impl_dir = "//chrome/browser/extensions/api" 21 impl_dir = "//chrome/browser/extensions/api"
21 bundle_registration = true 22 bundle_registration = true
23 bundle_name = "Chrome"
22 24
23 deps = [ 25 deps = [
24 # Different APIs include headers from these targets. 26 # Different APIs include headers from these targets.
25 "//content/public/browser", 27 "//content/public/browser",
26 "//extensions/browser", 28 "//extensions/browser",
27 29
28 # Different APIs include some headers from chrome/common that in turn 30 # Different APIs include some headers from chrome/common that in turn
29 # include generated headers from these targets. 31 # include generated headers from these targets.
30 # TODO(brettw) this should be made unnecessary if possible. 32 # TODO(brettw) this should be made unnecessary if possible.
31 ":api", 33 ":api",
32 "//components/copresence/proto", 34 "//components/copresence/proto",
33 "//components/metrics/proto", 35 "//components/metrics/proto",
34 "//skia", 36 "//skia",
35 "//sync", 37 "//sync",
36 "//ui/accessibility:ax_gen", 38 "//ui/accessibility:ax_gen",
37 ] 39 ]
38 if (is_chromeos) { 40 if (is_chromeos) {
39 # deps += [ "<(DEPTH)/chrome/chrome.gyp:drive_proto" ] TODO(GYP) 41 # deps += [ "<(DEPTH)/chrome/chrome.gyp:drive_proto" ] TODO(GYP)
40 } 42 }
41 deps += schema_dependencies 43 deps += schema_dependencies
42 44
43 # This must be a static library because the generated schemas have 45 # This must be a static library because the generated schemas have
44 # dependencies into both the browser and child processes, and this won't link 46 # dependencies into both the browser and child processes, and this won't link
45 # in the multi DLL build. In the child DLL, only some files are needed, and 47 # in the multi DLL build. In the child DLL, only some files are needed, and
46 # these have no dependencies into the browser, so a static library solves the 48 # these have no dependencies into the browser, so a static library solves the
47 # problem. 49 # problem.
48 generate_static_library = true 50 generate_static_library = true
49 } 51 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698