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

Unified Diff: test/cctest/heap/test-array-buffer-tracker.cc

Issue 2006183003: Revert of [heap] Fine-grained JSArrayBuffer tracking (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 7 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 | « test/cctest/heap/heap-utils.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/heap/test-array-buffer-tracker.cc
diff --git a/test/cctest/heap/test-array-buffer-tracker.cc b/test/cctest/heap/test-array-buffer-tracker.cc
deleted file mode 100644
index faa8a9f8e4367404b081efdf31ddbf3aed5991ad..0000000000000000000000000000000000000000
--- a/test/cctest/heap/test-array-buffer-tracker.cc
+++ /dev/null
@@ -1,145 +0,0 @@
-// Copyright 2016 the V8 project authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "src/heap/array-buffer-tracker-inl.h"
-#include "src/heap/array-buffer-tracker.h"
-#include "test/cctest/cctest.h"
-#include "test/cctest/heap/heap-utils.h"
-
-namespace {
-
-typedef i::LocalArrayBufferTracker LocalTracker;
-
-void VerifyTrackedInNewSpace(i::JSArrayBuffer* buf) {
- CHECK(i::Page::FromAddress(buf->address())->InNewSpace());
- CHECK(i::Page::FromAddress(buf->address())->local_tracker()->IsTracked(buf));
-}
-
-void VerifyTrackedInOldSpace(i::JSArrayBuffer* buf) {
- CHECK(!i::Page::FromAddress(buf->address())->InNewSpace());
- CHECK(i::Page::FromAddress(buf->address())->local_tracker()->IsTracked(buf));
-}
-
-void VerifyUntracked(i::JSArrayBuffer* buf) {
- CHECK(!i::Page::FromAddress(buf->address())->local_tracker()->IsTracked(buf));
-}
-
-} // namespace
-
-namespace v8 {
-namespace internal {
-
-// The following tests make sure that JSArrayBuffer tracking works expected when
-// moving the objects through various spaces during GC phases.
-
-TEST(ArrayBuffer_OnlyMC) {
- CcTest::InitializeVM();
- LocalContext env;
- v8::Isolate* isolate = env->GetIsolate();
- Heap* heap = reinterpret_cast<Isolate*>(isolate)->heap();
-
- JSArrayBuffer* raw_ab = nullptr;
- {
- v8::HandleScope handle_scope(isolate);
- Local<v8::ArrayBuffer> ab = v8::ArrayBuffer::New(isolate, 100);
- Handle<JSArrayBuffer> buf = v8::Utils::OpenHandle(*ab);
- VerifyTrackedInNewSpace(*buf);
- heap::GcAndSweep(heap, OLD_SPACE);
- VerifyTrackedInNewSpace(*buf);
- heap::GcAndSweep(heap, OLD_SPACE);
- VerifyTrackedInOldSpace(*buf);
- raw_ab = *buf;
- }
- // 2 GCs are needed because we promote to old space as live, meaining that
- // we will survive one GC.
- heap::GcAndSweep(heap, OLD_SPACE);
- heap::GcAndSweep(heap, OLD_SPACE);
- VerifyUntracked(raw_ab);
-}
-
-TEST(ArrayBuffer_OnlyScavenge) {
- CcTest::InitializeVM();
- LocalContext env;
- v8::Isolate* isolate = env->GetIsolate();
- Heap* heap = reinterpret_cast<Isolate*>(isolate)->heap();
-
- JSArrayBuffer* raw_ab = nullptr;
- {
- v8::HandleScope handle_scope(isolate);
- Local<v8::ArrayBuffer> ab = v8::ArrayBuffer::New(isolate, 100);
- Handle<JSArrayBuffer> buf = v8::Utils::OpenHandle(*ab);
- VerifyTrackedInNewSpace(*buf);
- heap::GcAndSweep(heap, NEW_SPACE);
- VerifyTrackedInNewSpace(*buf);
- heap::GcAndSweep(heap, NEW_SPACE);
- VerifyTrackedInOldSpace(*buf);
- heap::GcAndSweep(heap, NEW_SPACE);
- VerifyTrackedInOldSpace(*buf);
- raw_ab = *buf;
- }
- // 2 GCs are needed because we promote to old space as live, meaning that
- // we will survive one GC.
- heap::GcAndSweep(heap, OLD_SPACE);
- heap::GcAndSweep(heap, OLD_SPACE);
- VerifyUntracked(raw_ab);
-}
-
-TEST(ArrayBuffer_ScavengeAndMC) {
- CcTest::InitializeVM();
- LocalContext env;
- v8::Isolate* isolate = env->GetIsolate();
- Heap* heap = reinterpret_cast<Isolate*>(isolate)->heap();
-
- JSArrayBuffer* raw_ab = nullptr;
- {
- v8::HandleScope handle_scope(isolate);
- Local<v8::ArrayBuffer> ab = v8::ArrayBuffer::New(isolate, 100);
- Handle<JSArrayBuffer> buf = v8::Utils::OpenHandle(*ab);
- VerifyTrackedInNewSpace(*buf);
- heap::GcAndSweep(heap, NEW_SPACE);
- VerifyTrackedInNewSpace(*buf);
- heap::GcAndSweep(heap, NEW_SPACE);
- VerifyTrackedInOldSpace(*buf);
- heap::GcAndSweep(heap, OLD_SPACE);
- VerifyTrackedInOldSpace(*buf);
- heap::GcAndSweep(heap, NEW_SPACE);
- VerifyTrackedInOldSpace(*buf);
- raw_ab = *buf;
- }
- // 2 GCs are needed because we promote to old space as live, meaning that
- // we will survive one GC.
- heap::GcAndSweep(heap, OLD_SPACE);
- heap::GcAndSweep(heap, OLD_SPACE);
- VerifyUntracked(raw_ab);
-}
-
-TEST(ArrayBuffer_Compaction) {
- FLAG_manual_evacuation_candidates_selection = true;
- CcTest::InitializeVM();
- LocalContext env;
- v8::Isolate* isolate = env->GetIsolate();
- Heap* heap = reinterpret_cast<Isolate*>(isolate)->heap();
- heap::AbandonCurrentlyFreeMemory(heap->old_space());
-
- v8::HandleScope handle_scope(isolate);
- Local<v8::ArrayBuffer> ab1 = v8::ArrayBuffer::New(isolate, 100);
- Handle<JSArrayBuffer> buf1 = v8::Utils::OpenHandle(*ab1);
- VerifyTrackedInNewSpace(*buf1);
- heap::GcAndSweep(heap, NEW_SPACE);
- heap::GcAndSweep(heap, NEW_SPACE);
-
- Page* page_before_gc = Page::FromAddress(buf1->address());
- page_before_gc->SetFlag(MemoryChunk::FORCE_EVACUATION_CANDIDATE_FOR_TESTING);
- VerifyTrackedInOldSpace(*buf1);
-
- heap->CollectAllGarbage();
-
- Page* page_after_gc = Page::FromAddress(buf1->address());
- VerifyTrackedInOldSpace(*buf1);
-
- CHECK_NE(page_before_gc, page_after_gc);
-}
-
-} // namespace internal
-} // namespace v8
« no previous file with comments | « test/cctest/heap/heap-utils.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698