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

Unified Diff: runtime/vm/coverage_test.cc

Issue 351373002: Coverage API revamp (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: handle scripts via url instead of object Created 6 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
Index: runtime/vm/coverage_test.cc
diff --git a/runtime/vm/coverage_test.cc b/runtime/vm/coverage_test.cc
index 7d8e44701be7750bf1cad41f2139467dd56a3d16..6219212f8c74ab80dd6d9c10107b1dc775438fd9 100644
--- a/runtime/vm/coverage_test.cc
+++ b/runtime/vm/coverage_test.cc
@@ -28,7 +28,7 @@ TEST_CASE(Coverage_Empty) {
ExecuteScript(kScript);
JSONStream js;
- CodeCoverage::PrintJSON(isolate, &js);
+ CodeCoverage::PrintJSON(isolate, &js, NULL);
EXPECT_SUBSTRING(
"{\"source\":\"test-lib\",\"script\":{"
@@ -57,7 +57,7 @@ TEST_CASE(Coverage_MainWithClass) {
ExecuteScript(kScript);
JSONStream js;
- CodeCoverage::PrintJSON(isolate, &js);
+ CodeCoverage::PrintJSON(isolate, &js, NULL);
// Coverage data is printed per class, i.e., there should be two sections
// for test-lib in the JSON data.

Powered by Google App Engine
This is Rietveld 408576698