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

Unified Diff: runtime/vm/thread_test.cc

Issue 2562243003: Fixed tests breaking build. (Closed)
Patch Set: Created 4 years 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 | « no previous file | runtime/vm/zone_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/thread_test.cc
diff --git a/runtime/vm/thread_test.cc b/runtime/vm/thread_test.cc
index 99302caf787a7c6b2d557ae032f868ef5995d43a..9651dff66c8bd04b75b34172c9129b3e82848c34 100644
--- a/runtime/vm/thread_test.cc
+++ b/runtime/vm/thread_test.cc
@@ -335,18 +335,20 @@ TEST_CASE(ManySimpleTasksWithZones) {
char* zone_info_buf =
OS::SCreate(current_zone,
"\"type\":\"_Zone\","
- "\"capacity\":%ld,"
- "\"used\":%ld",
+ "\"capacity\":%" Pd
+ ","
+ "\"used\":%" Pd "",
top_zone->SizeInBytes(), top_zone->UsedSizeInBytes());
EXPECT_SUBSTRING(zone_info_buf, json);
top_zone = top_zone->previous();
}
// Check the thread exists and is the correct size.
- char* thread_info_buf = OS::SCreate(current_zone,
- "\"type\":\"_Thread\","
- "\"id\":\"threads\\/%" Pd64 "",
- thread->os_thread()->trace_id());
+ char* thread_info_buf = OS::SCreate(
+ current_zone,
+ "\"type\":\"_Thread\","
+ "\"id\":\"threads\\/%" Pd "",
+ OSThread::ThreadIdToIntPtr(thread->os_thread()->trace_id()));
// Ensure the isolate for each thread is valid.
« no previous file with comments | « no previous file | runtime/vm/zone_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698