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

Unified Diff: testing/coverage_util_ios.h

Issue 2789433004: Add tools for code coverage support in iOS. (Closed)
Patch Set: Created 3 years, 9 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: testing/coverage_util_ios.h
diff --git a/testing/coverage_util_ios.h b/testing/coverage_util_ios.h
index 702811aa7b2e93b27389428888bc336918a4895a..fe993c8a56887418e229d5d487c9edf637c0278a 100644
--- a/testing/coverage_util_ios.h
+++ b/testing/coverage_util_ios.h
@@ -7,10 +7,9 @@
namespace coverage_util {
-// Flushes .gcda coverage files if ENABLE_TEST_CODE_COVERAGE is defined. iOS 7
-// does not call any code at the "end" of an app so flushing should be
-// performed manually.
-void FlushCoverageDataIfNecessary();
+// In debug builds, if ENABLE_TEST_CODE_COVERAGE is defined, sets the filename
lpromero 2017/03/31 11:22:06 Sylvain, you told me GN was against such env varia
+// of the coverage file. Otherwise, it does nothing.
+void SetupIfNecessary();
Eugene But (OOO till 7-30) 2017/03/31 18:27:51 Should this be SetUpForDebugBuild()
lpromero 2017/03/31 20:35:20 Well, there would probably be another condition: E
Eugene But (OOO till 7-30) 2017/03/31 20:42:12 Nope, I guess IfNecessary is fine. Sorry my previo
} // namespace coverage_util

Powered by Google App Engine
This is Rietveld 408576698