0b3e8e184c
DevTools may process another protocol message during API interrupt this API may lead to createInjectedScript reentrance and will fail. Let's postpone interrupts. Bug: chromium:846099 Cq-Include-Trybots: luci.chromium.try:linux_chromium_headless_rel;luci.chromium.try:linux_chromium_rel_ng;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ia06e034a6287087e4674559d8911d2f4a0b1b459 Reviewed-on: https://chromium-review.googlesource.com/1086372 Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> Reviewed-by: Yang Guo <yangguo@chromium.org> Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#53531}
66 lines
2.2 KiB
C++
66 lines
2.2 KiB
C++
// Copyright 2018 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 <memory>
|
|
|
|
#include "test/cctest/cctest.h"
|
|
|
|
#include "include/v8-inspector.h"
|
|
#include "include/v8.h"
|
|
#include "src/inspector/protocol/Runtime.h"
|
|
|
|
using v8_inspector::StringBuffer;
|
|
using v8_inspector::StringView;
|
|
using v8_inspector::V8ContextInfo;
|
|
using v8_inspector::V8Inspector;
|
|
using v8_inspector::V8InspectorSession;
|
|
|
|
namespace {
|
|
|
|
class NoopChannel : public V8Inspector::Channel {
|
|
public:
|
|
virtual ~NoopChannel() {}
|
|
void sendResponse(int callId,
|
|
std::unique_ptr<StringBuffer> message) override {}
|
|
void sendNotification(std::unique_ptr<StringBuffer> message) override {}
|
|
void flushProtocolNotifications() override {}
|
|
};
|
|
|
|
void WrapOnInterrupt(v8::Isolate* isolate, void* data) {
|
|
const char* object_group = "";
|
|
StringView object_group_view(reinterpret_cast<const uint8_t*>(object_group),
|
|
strlen(object_group));
|
|
reinterpret_cast<V8InspectorSession*>(data)->wrapObject(
|
|
isolate->GetCurrentContext(), v8::Null(isolate), object_group_view,
|
|
false);
|
|
}
|
|
|
|
} // namespace
|
|
|
|
TEST(WrapInsideWrapOnInterrupt) {
|
|
LocalContext env;
|
|
v8::Isolate* isolate = env->GetIsolate();
|
|
v8::HandleScope handle_scope(isolate);
|
|
|
|
v8_inspector::V8InspectorClient default_client;
|
|
std::unique_ptr<V8Inspector> inspector =
|
|
V8Inspector::create(isolate, &default_client);
|
|
const char* name = "";
|
|
StringView name_view(reinterpret_cast<const uint8_t*>(name), strlen(name));
|
|
V8ContextInfo context_info(env.local(), 1, name_view);
|
|
inspector->contextCreated(context_info);
|
|
|
|
NoopChannel channel;
|
|
const char* state = "{}";
|
|
StringView state_view(reinterpret_cast<const uint8_t*>(state), strlen(state));
|
|
std::unique_ptr<V8InspectorSession> session =
|
|
inspector->connect(1, &channel, state_view);
|
|
|
|
const char* object_group = "";
|
|
StringView object_group_view(reinterpret_cast<const uint8_t*>(object_group),
|
|
strlen(object_group));
|
|
isolate->RequestInterrupt(&WrapOnInterrupt, session.get());
|
|
session->wrapObject(env.local(), v8::Null(isolate), object_group_view, false);
|
|
}
|