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

Unified Diff: chrome/chrome_watcher/BUILD.gn

Issue 2086403002: Remove the Kasko based out of process browser hang instrumentation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge Created 4 years, 6 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
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | chrome/chrome_watcher/chrome_watcher.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_watcher/BUILD.gn
diff --git a/chrome/chrome_watcher/BUILD.gn b/chrome/chrome_watcher/BUILD.gn
index 428616443ddfc9b511a19923fc3487f3b513361c..8a3e48a931d97228f6f061bc1a5b90a3c9a7dd3e 100644
--- a/chrome/chrome_watcher/BUILD.gn
+++ b/chrome/chrome_watcher/BUILD.gn
@@ -4,7 +4,6 @@
import("//chrome/version.gni")
import("//testing/test.gni")
-import("//third_party/kasko/kasko.gni")
source_set("client") {
sources = [
@@ -16,42 +15,36 @@ source_set("client") {
]
}
-if (enable_kasko) {
- source_set("kasko_util") {
- sources = [
- "kasko_util.cc",
- "kasko_util.h",
- "system_load_estimator.cc",
- "system_load_estimator.h",
- ]
- deps = [
- "//base",
- "//components/crash/content/app",
- "//components/memory_pressure",
- "//third_party/kasko",
- ]
- libs = [ "pdh.lib" ] # Used by system_load_estimator.h.
- ldflags = [ "/DELAYLOAD:pdh.dll" ] # Only used on report capture.
- }
+source_set("hang_util") {
+ sources = [
+ "system_load_estimator.cc",
+ "system_load_estimator.h",
+ ]
+ deps = [
+ "//base",
+ "//components/memory_pressure",
+ ]
+ libs = [ "pdh.lib" ] # Used by system_load_estimator.h.
+ ldflags = [ "/DELAYLOAD:pdh.dll" ] # Only used on report capture.
+}
+
+test("system_load_estimator_unittests") {
+ sources = [
+ "system_load_estimator_unittest.cc",
+ ]
+ deps = [
+ ":client",
+ ":hang_util",
+ "//base/test:run_all_unittests",
+ "//base/test:test_support",
+ "//testing/gtest",
+ ]
+}
- test("system_load_estimator_unittests") {
- sources = [
- "system_load_estimator_unittest.cc",
- ]
- deps = [
- ":client",
- ":kasko_util",
- "//base/test:run_all_unittests",
- "//base/test:test_support",
- "//testing/gtest",
- ]
- }
-} else {
- group("kasko_util") {
- public_deps = [
- "//third_party/kasko:kasko_features",
- ]
- }
+group("kasko_util") {
+ public_deps = [
+ "//third_party/kasko:kasko_features",
+ ]
}
process_version("chrome_watcher_resources") {
« no previous file with comments | « chrome/chrome_browser.gypi ('k') | chrome/chrome_watcher/chrome_watcher.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698