Revision: 21590
Author:   yang...@chromium.org
Date:     Mon Jun  2 06:22:09 2014 UTC
Log:      Remove unused debug API.

R=yu...@chromium.org

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

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

=======================================
--- /branches/bleeding_edge/include/v8-debug.h  Wed May 14 16:28:46 2014 UTC
+++ /branches/bleeding_edge/include/v8-debug.h  Mon Jun  2 06:22:09 2014 UTC
@@ -139,9 +139,6 @@
    */
   typedef void (*EventCallback)(const EventDetails& event_details);

-  // TODO(yangguo) Deprecate this.
-  typedef EventCallback EventCallback2;
-
   /**
    * Debug message callback function.
    *
@@ -152,9 +149,6 @@
    */
   typedef void (*MessageHandler)(const Message& message);

-  // TODO(yangguo) Deprecate this.
-  typedef MessageHandler MessageHandler2;
-
   /**
    * Callback function for the host to ensure debug messages are processed.
    */
@@ -162,12 +156,6 @@

   static bool SetDebugEventListener(EventCallback that,
                                     Handle<Value> data = Handle<Value>());
-
-  // TODO(yangguo) Deprecate this.
-  static bool SetDebugEventListener2(EventCallback2 that,
- Handle<Value> data = Handle<Value>()) {
-    return SetDebugEventListener(that, data);
-  }

   // Schedule a debugger break to happen when JavaScript code is run
   // in the given isolate.
@@ -191,11 +179,6 @@

   // Message based interface. The message protocol is JSON.
   static void SetMessageHandler(MessageHandler handler);
-
-  // TODO(yangguo) Deprecate this.
-  static void SetMessageHandler2(MessageHandler2 handler) {
-    SetMessageHandler(handler);
-  }

   static void SendCommand(Isolate* isolate,
                           const uint16_t* command, int length,
=======================================
--- /branches/bleeding_edge/src/debug.cc        Wed May 28 10:41:13 2014 UTC
+++ /branches/bleeding_edge/src/debug.cc        Mon Jun  2 06:22:09 2014 UTC
@@ -2864,8 +2864,8 @@
                                Handle<Object> event_data,
                                v8::Debug::ClientData* client_data) {
   Handle<Foreign> callback_obj(Handle<Foreign>::cast(event_listener_));
-  v8::Debug::EventCallback2 callback =
-      FUNCTION_CAST<v8::Debug::EventCallback2>(
+  v8::Debug::EventCallback callback =
+      FUNCTION_CAST<v8::Debug::EventCallback>(
           callback_obj->foreign_address());
   EventDetailsImpl event_details(
       event,
=======================================
--- /branches/bleeding_edge/test/cctest/test-debug.cc Tue May 27 13:43:29 2014 UTC +++ /branches/bleeding_edge/test/cctest/test-debug.cc Mon Jun 2 06:22:09 2014 UTC
@@ -6647,7 +6647,7 @@

   counting_message_handler_counter = 0;

-  v8::Debug::SetMessageHandler2(CountingMessageHandler);
+  v8::Debug::SetMessageHandler(CountingMessageHandler);
   send_command_thread_ = new SendCommandThread(isolate);
   send_command_thread_->Start();

@@ -6659,7 +6659,7 @@

   CHECK_EQ(100, counting_message_handler_counter);

-  v8::Debug::SetMessageHandler2(NULL);
+  v8::Debug::SetMessageHandler(NULL);
   CheckDebuggerUnloaded();
 }

--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to