Revision: 15735
Author:   [email protected]
Date:     Thu Jul 18 00:34:19 2013
Log:      Revert "Deprecate some debugger methods"

This reverts commit 69957c36edaebe0b9b614099eb1b067fb0208671. Blink roll this patch depends on was reverted for some reason.

BUG=None
[email protected]

Review URL: https://codereview.chromium.org/19483003
http://code.google.com/p/v8/source/detail?r=15735

Modified:
 /branches/bleeding_edge/include/v8-debug.h
 /branches/bleeding_edge/src/d8-debug.cc
 /branches/bleeding_edge/src/d8-debug.h
 /branches/bleeding_edge/src/d8.cc
 /branches/bleeding_edge/test/cctest/test-debug.cc

=======================================
--- /branches/bleeding_edge/include/v8-debug.h  Wed Jul 17 23:51:03 2013
+++ /branches/bleeding_edge/include/v8-debug.h  Thu Jul 18 00:34:19 2013
@@ -245,9 +245,8 @@
   typedef void (*DebugMessageDispatchHandler)();

   // Set a C debug event listener.
-  V8_DEPRECATED(static bool SetDebugEventListener(
-      EventCallback that,
-      Handle<Value> data = Handle<Value>()));
+  static bool SetDebugEventListener(EventCallback that,
+                                    Handle<Value> data = Handle<Value>());
   static bool SetDebugEventListener2(EventCallback2 that,
                                      Handle<Value> data = Handle<Value>());

@@ -275,9 +274,8 @@

// Message based interface. The message protocol is JSON. NOTE the message
   // handler thread is not supported any more parameter must be false.
-  V8_DEPRECATED(static void SetMessageHandler(
-      MessageHandler handler,
-      bool message_handler_thread = false));
+  static void SetMessageHandler(MessageHandler handler,
+                                bool message_handler_thread = false);
   static void SetMessageHandler2(MessageHandler2 handler);

   // If no isolate is provided the default isolate is
=======================================
--- /branches/bleeding_edge/src/d8-debug.cc     Wed Jul 17 23:51:03 2013
+++ /branches/bleeding_edge/src/d8-debug.cc     Thu Jul 18 00:34:19 2013
@@ -50,12 +50,14 @@
 }


-void HandleDebugEvent(const Debug::EventDetails& event_details) {
+void HandleDebugEvent(DebugEvent event,
+                      Handle<Object> exec_state,
+                      Handle<Object> event_data,
+                      Handle<Value> data) {
// TODO(svenpanne) There should be a way to retrieve this in the callback.
   Isolate* isolate = Isolate::GetCurrent();
   HandleScope scope(isolate);

-  DebugEvent event = event_details.GetEvent();
   // Check for handled event.
   if (event != Break && event != Exception && event != AfterCompile) {
     return;
@@ -65,7 +67,6 @@

   // Get the toJSONProtocol function on the event and get the JSON format.
   Local<String> to_json_fun_name = String::New("toJSONProtocol");
-  Handle<Object> event_data = event_details.GetEventData();
   Local<Function> to_json_fun =
       Local<Function>::Cast(event_data->Get(to_json_fun_name));
   Local<Value> event_json = to_json_fun->Call(event_data, 0, NULL);
@@ -90,7 +91,6 @@

   // Get the debug command processor.
   Local<String> fun_name = String::New("debugCommandProcessor");
-  Handle<Object> exec_state = event_details.GetExecutionState();
   Local<Function> fun = Local<Function>::Cast(exec_state->Get(fun_name));
   Local<Object> cmd_processor =
       Local<Object>::Cast(fun->Call(exec_state, 0, NULL));
=======================================
--- /branches/bleeding_edge/src/d8-debug.h      Wed Jul 17 23:51:03 2013
+++ /branches/bleeding_edge/src/d8-debug.h      Thu Jul 18 00:34:19 2013
@@ -36,7 +36,10 @@
 namespace v8 {


-void HandleDebugEvent(const Debug::EventDetails& event_details);
+void HandleDebugEvent(DebugEvent event,
+                      Handle<Object> exec_state,
+                      Handle<Object> event_data,
+                      Handle<Value> data);

// Start the remove debugger connecting to a V8 debugger agent on the specified
 // port.
=======================================
--- /branches/bleeding_edge/src/d8.cc   Wed Jul 17 23:51:03 2013
+++ /branches/bleeding_edge/src/d8.cc   Thu Jul 18 00:34:19 2013
@@ -810,7 +810,7 @@
 #ifdef ENABLE_DEBUGGER_SUPPORT
   // Start the in-process debugger if requested.
   if (i::FLAG_debugger && !i::FLAG_debugger_agent) {
-    v8::Debug::SetDebugEventListener2(HandleDebugEvent);
+    v8::Debug::SetDebugEventListener(HandleDebugEvent);
   }
 #endif  // ENABLE_DEBUGGER_SUPPORT
 }
=======================================
--- /branches/bleeding_edge/test/cctest/test-debug.cc Wed Jul 17 23:51:03 2013 +++ /branches/bleeding_edge/test/cctest/test-debug.cc Thu Jul 18 00:34:19 2013
@@ -29,7 +29,6 @@

 #include <stdlib.h>

-#define V8_DISABLE_DEPRECATIONS 1
 #include "v8.h"

 #include "api.h"
@@ -40,7 +39,6 @@
 #include "platform.h"
 #include "stub-cache.h"
 #include "utils.h"
-#undef V8_DISABLE_DEPRECATIONS


 using ::v8::internal::EmbeddedVector;

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to