Chromium Code Reviews| Index: test/cctest/test-heap-profiler.cc |
| diff --git a/test/cctest/test-heap-profiler.cc b/test/cctest/test-heap-profiler.cc |
| index 4416f38973ae1123130fb1f395fd4d5e77d2c34f..de5af01043d6b30497cef6c7c09b1e7338f2503a 100644 |
| --- a/test/cctest/test-heap-profiler.cc |
| +++ b/test/cctest/test-heap-profiler.cc |
| @@ -95,10 +95,10 @@ class NamedEntriesDetector { |
| static const v8::HeapGraphNode* GetGlobalObject( |
| const v8::HeapSnapshot* snapshot) { |
| - CHECK_EQ(2, snapshot->GetRoot()->GetChildrenCount()); |
| + CHECK_EQ(3, snapshot->GetRoot()->GetChildrenCount()); |
| // The 0th-child is (GC Roots), 1st is the user root. |
|
Michael Starzinger
2015/07/10 13:26:30
nit: Comment is outdated.
danno
2015/07/13 09:43:37
Done
|
| const v8::HeapGraphNode* global_obj = |
| - snapshot->GetRoot()->GetChild(1)->GetToNode(); |
| + snapshot->GetRoot()->GetChild(2)->GetToNode(); |
| CHECK_EQ(0, strncmp("Object", const_cast<i::HeapEntry*>( |
| reinterpret_cast<const i::HeapEntry*>(global_obj))->name(), 6)); |
| return global_obj; |
| @@ -960,7 +960,7 @@ TEST(HeapSnapshotJSONSerialization) { |
| v8::Local<v8::String> ref_string = |
| CompileRun(STRING_LITERAL_FOR_TEST)->ToString(isolate); |
| #undef STRING_LITERAL_FOR_TEST |
| - CHECK_EQ(0, strcmp(*v8::String::Utf8Value(ref_string), |
| + CHECK_LT(0, strcmp(*v8::String::Utf8Value(ref_string), |
| *v8::String::Utf8Value(string))); |
| } |
| @@ -2082,6 +2082,7 @@ TEST(NoDebugObjectInSnapshot) { |
| CHECK(ValidateSnapshot(snapshot)); |
| const v8::HeapGraphNode* root = snapshot->GetRoot(); |
| int globals_count = 0; |
| + bool found = false; |
| for (int i = 0; i < root->GetChildrenCount(); ++i) { |
| const v8::HeapGraphEdge* edge = root->GetChild(i); |
| if (edge->GetType() == v8::HeapGraphEdge::kShortcut) { |
| @@ -2089,10 +2090,13 @@ TEST(NoDebugObjectInSnapshot) { |
| const v8::HeapGraphNode* global = edge->GetToNode(); |
| const v8::HeapGraphNode* foo = |
| GetProperty(global, v8::HeapGraphEdge::kProperty, "foo"); |
| - CHECK(foo); |
| + if (foo != nullptr) { |
| + found = true; |
| + } |
| } |
| } |
| - CHECK_EQ(1, globals_count); |
| + CHECK_EQ(2, globals_count); |
| + CHECK(found); |
| } |