Commit 0f2bffa1 authored by jochen's avatar jochen Committed by Commit bot

Mark deprecated debugger APIs as such

We should consider deprecating the message based API as well in the
future.

BUG=none
R=yangguo@chromium.org
LOG=y

Review URL: https://codereview.chromium.org/1485333007

Cr-Commit-Position: refs/heads/master@{#32621}
parent 7d1263db
...@@ -157,10 +157,9 @@ class V8_EXPORT Debug { ...@@ -157,10 +157,9 @@ class V8_EXPORT Debug {
static bool SetDebugEventListener(Isolate* isolate, EventCallback that, static bool SetDebugEventListener(Isolate* isolate, EventCallback that,
Local<Value> data = Local<Value>()); Local<Value> data = Local<Value>());
V8_DEPRECATE_SOON( V8_DEPRECATED("Use version with an Isolate",
"Use version with an Isolate", static bool SetDebugEventListener(
static bool SetDebugEventListener(EventCallback that, EventCallback that, Local<Value> data = Local<Value>()));
Local<Value> data = Local<Value>()));
// Schedule a debugger break to happen when JavaScript code is run // Schedule a debugger break to happen when JavaScript code is run
// in the given isolate. // in the given isolate.
...@@ -175,8 +174,8 @@ class V8_EXPORT Debug { ...@@ -175,8 +174,8 @@ class V8_EXPORT Debug {
// Message based interface. The message protocol is JSON. // Message based interface. The message protocol is JSON.
static void SetMessageHandler(Isolate* isolate, MessageHandler handler); static void SetMessageHandler(Isolate* isolate, MessageHandler handler);
V8_DEPRECATE_SOON("Use version with an Isolate", V8_DEPRECATED("Use version with an Isolate",
static void SetMessageHandler(MessageHandler handler)); static void SetMessageHandler(MessageHandler handler));
static void SendCommand(Isolate* isolate, static void SendCommand(Isolate* isolate,
const uint16_t* command, int length, const uint16_t* command, int length,
...@@ -200,10 +199,9 @@ class V8_EXPORT Debug { ...@@ -200,10 +199,9 @@ class V8_EXPORT Debug {
* } * }
* \endcode * \endcode
*/ */
static V8_DEPRECATE_SOON( static V8_DEPRECATED("Use maybe version",
"Use maybe version", Local<Value> Call(v8::Local<v8::Function> fun,
Local<Value> Call(v8::Local<v8::Function> fun, Local<Value> data = Local<Value>()));
Local<Value> data = Local<Value>()));
// TODO(dcarney): data arg should be a MaybeLocal // TODO(dcarney): data arg should be a MaybeLocal
static MaybeLocal<Value> Call(Local<Context> context, static MaybeLocal<Value> Call(Local<Context> context,
v8::Local<v8::Function> fun, v8::Local<v8::Function> fun,
...@@ -212,8 +210,8 @@ class V8_EXPORT Debug { ...@@ -212,8 +210,8 @@ class V8_EXPORT Debug {
/** /**
* Returns a mirror object for the given object. * Returns a mirror object for the given object.
*/ */
static V8_DEPRECATE_SOON("Use maybe version", static V8_DEPRECATED("Use maybe version",
Local<Value> GetMirror(v8::Local<v8::Value> obj)); Local<Value> GetMirror(v8::Local<v8::Value> obj));
static MaybeLocal<Value> GetMirror(Local<Context> context, static MaybeLocal<Value> GetMirror(Local<Context> context,
v8::Local<v8::Value> obj); v8::Local<v8::Value> obj);
...@@ -249,8 +247,8 @@ class V8_EXPORT Debug { ...@@ -249,8 +247,8 @@ class V8_EXPORT Debug {
* of this method. * of this method.
*/ */
static void ProcessDebugMessages(Isolate* isolate); static void ProcessDebugMessages(Isolate* isolate);
V8_DEPRECATE_SOON("Use version with an Isolate", V8_DEPRECATED("Use version with an Isolate",
static void ProcessDebugMessages()); static void ProcessDebugMessages());
/** /**
* Debugger is running in its own context which is entered while debugger * Debugger is running in its own context which is entered while debugger
...@@ -260,8 +258,8 @@ class V8_EXPORT Debug { ...@@ -260,8 +258,8 @@ class V8_EXPORT Debug {
* least one DebugEventListener or MessageHandler is set. * least one DebugEventListener or MessageHandler is set.
*/ */
static Local<Context> GetDebugContext(Isolate* isolate); static Local<Context> GetDebugContext(Isolate* isolate);
V8_DEPRECATE_SOON("Use version with an Isolate", V8_DEPRECATED("Use version with an Isolate",
static Local<Context> GetDebugContext()); static Local<Context> GetDebugContext());
/** /**
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment