ec06bb6ce5
This is a reland of d1b27019d3
Fixes include:
Adding missing file to bazel build
Forward-declaring classing before friend-classing them to fix win/gcc
Add missing v8-isolate.h include for vtune builds
Original change's description:
> [include] Split out v8.h
>
> This moves every single class/function out of include/v8.h into a
> separate header in include/, which v8.h then includes so that
> externally nothing appears to have changed.
>
> Every include of v8.h from inside v8 has been changed to a more
> fine-grained include.
>
> Previously inline functions defined at the bottom of v8.h would call
> private non-inline functions in the V8 class. Since that class is now
> in v8-initialization.h and is rarely included (as that would create
> dependency cycles), this is not possible and so those methods have been
> moved out of the V8 class into the namespace v8::api_internal.
>
> None of the previous files in include/ now #include v8.h, which means
> if embedders were relying on this transitive dependency then it will
> give compile failures.
>
> v8-inspector.h does depend on v8-scripts.h for the time being to ensure
> that Chrome continue to compile but that change will be reverted once
> those transitive #includes in chrome are changed to include it directly.
>
> Full design:
> https://docs.google.com/document/d/1rTD--I8hCAr-Rho1WTumZzFKaDpEp0IJ8ejZtk4nJdA/edit?usp=sharing
>
> Bug: v8:11965
> Change-Id: I53b84b29581632710edc80eb11f819c2097a2877
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3097448
> Reviewed-by: Yang Guo <yangguo@chromium.org>
> Reviewed-by: Camillo Bruni <cbruni@chromium.org>
> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
> Reviewed-by: Leszek Swirski <leszeks@chromium.org>
> Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
> Commit-Queue: Dan Elphick <delphick@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#76424}
Cq-Include-Trybots: luci.v8.try:v8_linux_vtunejit
Bug: v8:11965
Change-Id: I99f5d3a73bf8fe25b650adfaf9567dc4e44a09e6
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3113629
Reviewed-by: Leszek Swirski <leszeks@chromium.org>
Reviewed-by: Camillo Bruni <cbruni@chromium.org>
Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
Reviewed-by: Simon Zünd <szuend@chromium.org>
Commit-Queue: Dan Elphick <delphick@chromium.org>
Cr-Commit-Position: refs/heads/main@{#76460}
139 lines
4.8 KiB
C++
139 lines
4.8 KiB
C++
// Copyright 2017 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/execution/isolate.h"
|
|
|
|
#include "include/libplatform/libplatform.h"
|
|
#include "include/v8-platform.h"
|
|
#include "include/v8-template.h"
|
|
#include "src/base/macros.h"
|
|
#include "src/base/platform/semaphore.h"
|
|
#include "src/execution/execution.h"
|
|
#include "src/init/v8.h"
|
|
#include "test/unittests/test-utils.h"
|
|
#include "testing/gtest/include/gtest/gtest.h"
|
|
|
|
namespace v8 {
|
|
|
|
using IsolateTest = TestWithIsolate;
|
|
|
|
namespace {
|
|
|
|
class MemoryPressureTask : public v8::Task {
|
|
public:
|
|
MemoryPressureTask(Isolate* isolate, base::Semaphore* semaphore)
|
|
: isolate_(isolate), semaphore_(semaphore) {}
|
|
~MemoryPressureTask() override = default;
|
|
MemoryPressureTask(const MemoryPressureTask&) = delete;
|
|
MemoryPressureTask& operator=(const MemoryPressureTask&) = delete;
|
|
|
|
// v8::Task implementation.
|
|
void Run() override {
|
|
isolate_->MemoryPressureNotification(MemoryPressureLevel::kCritical);
|
|
semaphore_->Signal();
|
|
}
|
|
|
|
private:
|
|
Isolate* isolate_;
|
|
base::Semaphore* semaphore_;
|
|
};
|
|
|
|
} // namespace
|
|
|
|
// Check that triggering a memory pressure notification on the isolate thread
|
|
// doesn't request a GC interrupt.
|
|
TEST_F(IsolateTest, MemoryPressureNotificationForeground) {
|
|
internal::Isolate* i_isolate =
|
|
reinterpret_cast<internal::Isolate*>(isolate());
|
|
|
|
ASSERT_FALSE(i_isolate->stack_guard()->CheckGC());
|
|
isolate()->MemoryPressureNotification(MemoryPressureLevel::kCritical);
|
|
ASSERT_FALSE(i_isolate->stack_guard()->CheckGC());
|
|
}
|
|
|
|
// Check that triggering a memory pressure notification on an background thread
|
|
// requests a GC interrupt.
|
|
TEST_F(IsolateTest, MemoryPressureNotificationBackground) {
|
|
internal::Isolate* i_isolate =
|
|
reinterpret_cast<internal::Isolate*>(isolate());
|
|
|
|
base::Semaphore semaphore(0);
|
|
|
|
internal::V8::GetCurrentPlatform()->CallOnWorkerThread(
|
|
std::make_unique<MemoryPressureTask>(isolate(), &semaphore));
|
|
|
|
semaphore.Wait();
|
|
|
|
ASSERT_TRUE(i_isolate->stack_guard()->CheckGC());
|
|
v8::platform::PumpMessageLoop(internal::V8::GetCurrentPlatform(), isolate());
|
|
}
|
|
|
|
using IncumbentContextTest = TestWithIsolate;
|
|
|
|
// Check that Isolate::GetIncumbentContext() returns the correct one in basic
|
|
// scenarios.
|
|
TEST_F(IncumbentContextTest, Basic) {
|
|
auto Str = [&](const char* s) {
|
|
return String::NewFromUtf8(isolate(), s).ToLocalChecked();
|
|
};
|
|
auto Run = [&](Local<Context> context, const char* script) {
|
|
Context::Scope scope(context);
|
|
return Script::Compile(context, Str(script))
|
|
.ToLocalChecked()
|
|
->Run(context)
|
|
.ToLocalChecked();
|
|
};
|
|
|
|
// Set up the test environment; three contexts with getIncumbentGlobal()
|
|
// function.
|
|
Local<FunctionTemplate> get_incumbent_global = FunctionTemplate::New(
|
|
isolate(), [](const FunctionCallbackInfo<Value>& info) {
|
|
Local<Context> incumbent_context =
|
|
info.GetIsolate()->GetIncumbentContext();
|
|
info.GetReturnValue().Set(incumbent_context->Global());
|
|
});
|
|
Local<ObjectTemplate> global_template = ObjectTemplate::New(isolate());
|
|
global_template->Set(isolate(), "getIncumbentGlobal", get_incumbent_global);
|
|
|
|
Local<Context> context_a = Context::New(isolate(), nullptr, global_template);
|
|
Local<Context> context_b = Context::New(isolate(), nullptr, global_template);
|
|
Local<Context> context_c = Context::New(isolate(), nullptr, global_template);
|
|
Local<Object> global_a = context_a->Global();
|
|
Local<Object> global_b = context_b->Global();
|
|
Local<Object> global_c = context_c->Global();
|
|
|
|
Local<String> security_token = Str("security_token");
|
|
context_a->SetSecurityToken(security_token);
|
|
context_b->SetSecurityToken(security_token);
|
|
context_c->SetSecurityToken(security_token);
|
|
|
|
global_a->Set(context_a, Str("b"), global_b).ToChecked();
|
|
global_b->Set(context_b, Str("c"), global_c).ToChecked();
|
|
|
|
// Test scenario 2: A -> B -> C, then the incumbent is C.
|
|
Run(context_a, "funcA = function() { return b.funcB(); }");
|
|
Run(context_b, "funcB = function() { return c.getIncumbentGlobal(); }");
|
|
// Without BackupIncumbentScope.
|
|
EXPECT_EQ(global_b, Run(context_a, "funcA()"));
|
|
{
|
|
// With BackupIncumbentScope.
|
|
Context::BackupIncumbentScope backup_incumbent(context_a);
|
|
EXPECT_EQ(global_b, Run(context_a, "funcA()"));
|
|
}
|
|
|
|
// Test scenario 2: A -> B -> C -> C, then the incumbent is C.
|
|
Run(context_a, "funcA = function() { return b.funcB(); }");
|
|
Run(context_b, "funcB = function() { return c.funcC(); }");
|
|
Run(context_c, "funcC = function() { return getIncumbentGlobal(); }");
|
|
// Without BackupIncumbentScope.
|
|
EXPECT_EQ(global_c, Run(context_a, "funcA()"));
|
|
{
|
|
// With BackupIncumbentScope.
|
|
Context::BackupIncumbentScope backup_incumbent(context_a);
|
|
EXPECT_EQ(global_c, Run(context_a, "funcA()"));
|
|
}
|
|
}
|
|
|
|
} // namespace v8
|