Remove some unneeded debugger code.
authormark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Jan 2014 23:29:30 +0000 (23:29 +0000)
committermark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Jan 2014 23:29:30 +0000 (23:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=127805.

Reviewed by Oliver Hunt.

JSC will now always support the debugger. Hence, the #if ENABLE(JAVASCRIPT_DEBUGGER)
checks can be removed.

DebuggerCallFrame::callFrame() is also unused and will be removed.

* debugger/Breakpoint.h:
* debugger/Debugger.cpp:
* debugger/DebuggerCallFrame.h:
* inspector/InjectedScript.cpp:
(Inspector::InjectedScript::wrapCallFrames):
* inspector/InjectedScript.h:
* inspector/JSGlobalObjectScriptDebugServer.cpp:
* inspector/JSGlobalObjectScriptDebugServer.h:
* inspector/JSJavaScriptCallFrame.cpp:
* inspector/JSJavaScriptCallFrame.h:
* inspector/JSJavaScriptCallFramePrototype.cpp:
* inspector/JSJavaScriptCallFramePrototype.h:
* inspector/JavaScriptCallFrame.cpp:
* inspector/JavaScriptCallFrame.h:
* inspector/ScriptDebugListener.h:
* inspector/ScriptDebugServer.cpp:
* inspector/ScriptDebugServer.h:
* inspector/agents/InspectorDebuggerAgent.cpp:
* inspector/agents/InspectorDebuggerAgent.h:
* inspector/agents/InspectorRuntimeAgent.cpp:
(Inspector::InspectorRuntimeAgent::InspectorRuntimeAgent):
(Inspector::setPauseOnExceptionsState):
(Inspector::InspectorRuntimeAgent::evaluate):
(Inspector::InspectorRuntimeAgent::callFunctionOn):
(Inspector::InspectorRuntimeAgent::getProperties):
* inspector/agents/InspectorRuntimeAgent.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@162970 268f45cc-cd09-0410-ab3c-d52691b4dbfc

21 files changed:
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/debugger/Breakpoint.h
Source/JavaScriptCore/debugger/Debugger.cpp
Source/JavaScriptCore/debugger/DebuggerCallFrame.h
Source/JavaScriptCore/inspector/InjectedScript.cpp
Source/JavaScriptCore/inspector/InjectedScript.h
Source/JavaScriptCore/inspector/JSGlobalObjectScriptDebugServer.cpp
Source/JavaScriptCore/inspector/JSGlobalObjectScriptDebugServer.h
Source/JavaScriptCore/inspector/JSJavaScriptCallFrame.cpp
Source/JavaScriptCore/inspector/JSJavaScriptCallFrame.h
Source/JavaScriptCore/inspector/JSJavaScriptCallFramePrototype.cpp
Source/JavaScriptCore/inspector/JSJavaScriptCallFramePrototype.h
Source/JavaScriptCore/inspector/JavaScriptCallFrame.cpp
Source/JavaScriptCore/inspector/JavaScriptCallFrame.h
Source/JavaScriptCore/inspector/ScriptDebugListener.h
Source/JavaScriptCore/inspector/ScriptDebugServer.cpp
Source/JavaScriptCore/inspector/ScriptDebugServer.h
Source/JavaScriptCore/inspector/agents/InspectorDebuggerAgent.cpp
Source/JavaScriptCore/inspector/agents/InspectorDebuggerAgent.h
Source/JavaScriptCore/inspector/agents/InspectorRuntimeAgent.cpp
Source/JavaScriptCore/inspector/agents/InspectorRuntimeAgent.h

index 44c5484518fd831b13b4b74f768fa72d1d3015b6..37f6a37dc8e44e97ebe94e08ea0fdf54f1cae0d5 100644 (file)
@@ -1,3 +1,42 @@
+2014-01-28  Mark Lam  <mark.lam@apple.com>
+
+        Remove some unneeded debugger code.
+        https://bugs.webkit.org/show_bug.cgi?id=127805.
+
+        Reviewed by Oliver Hunt.
+
+        JSC will now always support the debugger. Hence, the #if ENABLE(JAVASCRIPT_DEBUGGER)
+        checks can be removed.
+
+        DebuggerCallFrame::callFrame() is also unused and will be removed.
+
+        * debugger/Breakpoint.h:
+        * debugger/Debugger.cpp:
+        * debugger/DebuggerCallFrame.h:
+        * inspector/InjectedScript.cpp:
+        (Inspector::InjectedScript::wrapCallFrames):
+        * inspector/InjectedScript.h:
+        * inspector/JSGlobalObjectScriptDebugServer.cpp:
+        * inspector/JSGlobalObjectScriptDebugServer.h:
+        * inspector/JSJavaScriptCallFrame.cpp:
+        * inspector/JSJavaScriptCallFrame.h:
+        * inspector/JSJavaScriptCallFramePrototype.cpp:
+        * inspector/JSJavaScriptCallFramePrototype.h:
+        * inspector/JavaScriptCallFrame.cpp:
+        * inspector/JavaScriptCallFrame.h:
+        * inspector/ScriptDebugListener.h:
+        * inspector/ScriptDebugServer.cpp:
+        * inspector/ScriptDebugServer.h:
+        * inspector/agents/InspectorDebuggerAgent.cpp:
+        * inspector/agents/InspectorDebuggerAgent.h:
+        * inspector/agents/InspectorRuntimeAgent.cpp:
+        (Inspector::InspectorRuntimeAgent::InspectorRuntimeAgent):
+        (Inspector::setPauseOnExceptionsState):
+        (Inspector::InspectorRuntimeAgent::evaluate):
+        (Inspector::InspectorRuntimeAgent::callFunctionOn):
+        (Inspector::InspectorRuntimeAgent::getProperties):
+        * inspector/agents/InspectorRuntimeAgent.h:
+
 2014-01-28  Geoffrey Garen  <ggaren@apple.com>
 
         REGRESSION: JavascriptCore crash during OS Installation (due to
index 136db4dc8bea927464b7b13e6bac1fd4c2217857..95b92881d4707655f61ef8c3438f8c614e887860 100644 (file)
@@ -26,8 +26,6 @@
 #ifndef Breakpoint_h
 #define Breakpoint_h
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
-
 #include "DebuggerPrimitives.h"
 #include <wtf/text/WTFString.h>
 
@@ -65,6 +63,4 @@ struct Breakpoint {
 
 } // namespace JSC
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
-
 #endif // Breakpoint_h
index b27c9dd1cf320e1061fc6021f1a5e57e32f07f03..afa7546c8065b8f3b170df81a3372a8f069adb41 100644 (file)
@@ -20,8 +20,6 @@
  */
 
 #include "config.h"
-#if ENABLE(JAVASCRIPT_DEBUGGER)
-
 #include "Debugger.h"
 
 #include "CodeBlock.h"
@@ -761,5 +759,3 @@ DebuggerCallFrame* Debugger::currentDebuggerCallFrame() const
 }
 
 } // namespace JSC
-
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
index c58dc68cc0f3b9c65f59676e48ad6fcab85d893c..09c3fb9d81113105bbd79e062dd374c99a1aafb2 100644 (file)
@@ -48,7 +48,6 @@ public:
 
     JS_EXPORT_PRIVATE explicit DebuggerCallFrame(CallFrame*);
 
-    CallFrame* callFrame() const { return m_callFrame; }
     JS_EXPORT_PRIVATE PassRefPtr<DebuggerCallFrame> callerFrame();
     ExecState* exec() const { return m_callFrame; }
     JS_EXPORT_PRIVATE SourceID sourceID() const;
index 38fac56297258c93a137aab1e12d1d2cecfaf443..a02f85fb4d14c94eaef93deb41d613ce2b16a817 100644 (file)
@@ -141,7 +141,6 @@ void InjectedScript::getInternalProperties(ErrorString* errorString, const Strin
         *properties = array;
 }
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 PassRefPtr<Array<Inspector::TypeBuilder::Debugger::CallFrame>> InjectedScript::wrapCallFrames(const Deprecated::ScriptValue& callFrames)
 {
     ASSERT(!hasNoValue());
@@ -157,7 +156,6 @@ PassRefPtr<Array<Inspector::TypeBuilder::Debugger::CallFrame>> InjectedScript::w
 
     return Array<Inspector::TypeBuilder::Debugger::CallFrame>::create();
 }
-#endif
 
 PassRefPtr<Inspector::TypeBuilder::Runtime::RemoteObject> InjectedScript::wrapObject(const Deprecated::ScriptValue& value, const String& groupName, bool generatePreview) const
 {
index ae6712654940595edd9a3afbf9d631bcbe34c0d3..4c58b19e51f7f7109bdf2ad7400b0f9f7b27ab6e 100644 (file)
@@ -63,9 +63,7 @@ public:
     void getProperties(ErrorString*, const String& objectId, bool ownProperties, RefPtr<TypeBuilder::Array<TypeBuilder::Runtime::PropertyDescriptor>>* result);
     void getInternalProperties(ErrorString*, const String& objectId, RefPtr<TypeBuilder::Array<TypeBuilder::Runtime::InternalPropertyDescriptor>>* result);
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
     PassRefPtr<TypeBuilder::Array<TypeBuilder::Debugger::CallFrame>> wrapCallFrames(const Deprecated::ScriptValue&);
-#endif
     PassRefPtr<TypeBuilder::Runtime::RemoteObject> wrapObject(const Deprecated::ScriptValue&, const String& groupName, bool generatePreview = false) const;
     PassRefPtr<TypeBuilder::Runtime::RemoteObject> wrapTable(const Deprecated::ScriptValue& table, const Deprecated::ScriptValue& columns) const;
 
index 254a2c7d84a324e7e97e1d65c03554f041657929..29d1950fc257ff1c196fed465ef4d0aeb633db3a 100644 (file)
@@ -27,7 +27,6 @@
 #include "JSGlobalObjectScriptDebugServer.h"
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "EventLoop.h"
 
@@ -85,5 +84,4 @@ void JSGlobalObjectScriptDebugServer::runEventLoopWhilePaused()
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
index 2f51aa09fbf9ca7e02d3ad86d246f3662b44149b..ca0c2479f98fc5bfd6287ff81eb6fadd9e11136a 100644 (file)
@@ -27,7 +27,6 @@
 #define JSGlobalObjectScriptDebugServer_h
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "ScriptDebugServer.h"
 #include <wtf/Forward.h>
@@ -63,7 +62,6 @@ private:
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
 
 #endif // JSGlobalObjectScriptDebugServer_h
index 9f8a5f723b74312dc4e1980d076f269d99934223..08fd618a804d516dd415db673a524d702cc3e6be 100644 (file)
@@ -27,7 +27,6 @@
 #include "JSJavaScriptCallFrame.h"
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "Error.h"
 #include "JSCJSValue.h"
@@ -211,5 +210,4 @@ JSJavaScriptCallFrame* toJSJavaScriptCallFrame(JSValue value)
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
index 225edbab0bfb23fd7e331922729e29820be67c7e..dbbb93fa2dc18b3002005d0ffe8b15bda0bf18fb 100644 (file)
@@ -27,7 +27,6 @@
 #define JSJavaScriptCallFrame_h
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "JSDestructibleObject.h"
 #include "JavaScriptCallFrame.h"
@@ -96,7 +95,6 @@ JSJavaScriptCallFrame* toJSJavaScriptCallFrame(JSC::JSValue);
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
 
 #endif // !defined(JSJavaScriptCallFrame_h)
index acb2b061d9fb35c8f224e7669f79ae022c10e814..d74126cb24cad46bd75f2750ddd80584dbacc092 100644 (file)
@@ -27,7 +27,6 @@
 #include "JSJavaScriptCallFramePrototype.h"
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "Error.h"
 #include "GetterSetter.h"
@@ -234,5 +233,4 @@ EncodedJSValue JSC_HOST_CALL jsJavaScriptCallFrameConstantCATCH_SCOPE(ExecState*
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
index faa74a9ba9062a26fc189801f70f7cfd1dc08717..b868c53e253f7483e5f5dff64e56024692c51824 100644 (file)
@@ -27,7 +27,6 @@
 #define JSJavaScriptCallFramePrototype_h
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "JSObject.h"
 
@@ -64,7 +63,6 @@ private:
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
 
 #endif // !defined(JSJavaScriptCallFramePrototype_h)
index 3bc2da29df8f1579a06b9af9da1fcc3eef7acdea..f735a8aa61298973de3916d7b61df35154cca97b 100644 (file)
@@ -27,7 +27,6 @@
 #include "JavaScriptCallFrame.h"
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 using namespace JSC;
 
@@ -53,5 +52,4 @@ JavaScriptCallFrame* JavaScriptCallFrame::caller()
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
index 1e1168ff42cf99bc4e492659f1909b1ad23b97b1..b584067a2746313812d99831a0680693df727dd6 100644 (file)
@@ -27,7 +27,6 @@
 #define JavaScriptCallFrame_h
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "JSCJSValueInlines.h"
 #include "debugger/DebuggerCallFrame.h"
@@ -69,7 +68,6 @@ private:
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
 
 #endif // JavaScriptCallFrame_h
index d1fda11492be5e7d83d8fc0ac92973ab99f44143..bee3bc047fd037bc8f4d4d25bbc0f9aabb16ad9e 100644 (file)
@@ -30,8 +30,6 @@
 #ifndef ScriptDebugListener_h
 #define ScriptDebugListener_h
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
-
 #include "debugger/Debugger.h"
 #include <wtf/Forward.h>
 #include <wtf/text/WTFString.h>
@@ -80,6 +78,4 @@ public:
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
-
 #endif // ScriptDebugListener_h
index d7e86ccce251196051a89ef38b28c2fda362c35c..b1151cf7f920d8357c7f96b12a54460950fcb4fd 100644 (file)
@@ -31,8 +31,6 @@
 #include "config.h"
 #include "ScriptDebugServer.h"
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
-
 #include "DebuggerCallFrame.h"
 #include "JSJavaScriptCallFrame.h"
 #include "JSLock.h"
@@ -338,5 +336,3 @@ const Vector<ScriptBreakpointAction>& ScriptDebugServer::getActionsForBreakpoint
 }
 
 } // namespace Inspector
-
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
index c67dd5d064425b9c678d76999b7f5da58ed00579..6d55de24182eabb1f976a3b0330bad86ada7af1f 100644 (file)
@@ -30,8 +30,6 @@
 #ifndef ScriptDebugServer_h
 #define ScriptDebugServer_h
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
-
 #include "ScriptBreakpoint.h"
 #include "ScriptDebugListener.h"
 #include "bindings/ScriptObject.h"
@@ -115,6 +113,4 @@ private:
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
-
 #endif // ScriptDebugServer_h
index 6c4e3f8f275d0512fbe3167389b54542f388fa3e..06b759568e5d135fea870053f32c7dcf9e07a651 100644 (file)
@@ -31,7 +31,6 @@
 #include "InspectorDebuggerAgent.h"
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "ContentSearchUtilities.h"
 #include "InjectedScript.h"
@@ -733,5 +732,4 @@ void InspectorDebuggerAgent::didClearGlobalObject()
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
index ab5e19763c17c3826270d6fb38834c8243874489..0e652499d6a9251367350170590a3c33ce89e0fa 100644 (file)
@@ -31,7 +31,6 @@
 #define InspectorDebuggerAgent_h
 
 #if ENABLE(INSPECTOR)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 
 #include "InspectorJSBackendDispatchers.h"
 #include "InspectorJSFrontendDispatchers.h"
@@ -169,7 +168,6 @@ private:
 
 } // namespace Inspector
 
-#endif // ENABLE(JAVASCRIPT_DEBUGGER)
 #endif // ENABLE(INSPECTOR)
 
 #endif // !defined(InspectorDebuggerAgent_h)
index 956445140e7dca2863e744171892776d5dbdfa9f..76a47fb3afaa7eff70fa3ea517cd0aba3afe25f8 100644 (file)
 #include "InspectorValues.h"
 #include "JSLock.h"
 #include "ParserError.h"
+#include "ScriptDebugServer.h"
 #include "SourceCode.h"
 #include <wtf/PassRefPtr.h>
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
-#include "ScriptDebugServer.h"
-#endif
-
 using namespace JSC;
 
 namespace Inspector {
@@ -59,9 +56,7 @@ static bool asBool(const bool* const b)
 InspectorRuntimeAgent::InspectorRuntimeAgent(InjectedScriptManager* injectedScriptManager)
     : InspectorAgentBase(ASCIILiteral("Runtime"))
     , m_injectedScriptManager(injectedScriptManager)
-#if ENABLE(JAVASCRIPT_DEBUGGER)
     , m_scriptDebugServer(nullptr)
-#endif
     , m_enabled(false)
 {
 }
@@ -70,7 +65,6 @@ InspectorRuntimeAgent::~InspectorRuntimeAgent()
 {
 }
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
 static ScriptDebugServer::PauseOnExceptionsState setPauseOnExceptionsState(ScriptDebugServer* scriptDebugServer, ScriptDebugServer::PauseOnExceptionsState newState)
 {
     ASSERT(scriptDebugServer);
@@ -79,7 +73,6 @@ static ScriptDebugServer::PauseOnExceptionsState setPauseOnExceptionsState(Scrip
         scriptDebugServer->setPauseOnExceptionsState(newState);
     return presentState;
 }
-#endif
 
 static PassRefPtr<Inspector::TypeBuilder::Runtime::ErrorRange> buildErrorRangeObject(const JSTokenLocation& tokenLocation)
 {
@@ -124,11 +117,9 @@ void InspectorRuntimeAgent::evaluate(ErrorString* errorString, const String& exp
     if (injectedScript.hasNoValue())
         return;
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
     ScriptDebugServer::PauseOnExceptionsState previousPauseOnExceptionsState = ScriptDebugServer::DontPauseOnExceptions;
     if (asBool(doNotPauseOnExceptionsAndMuteConsole))
         previousPauseOnExceptionsState = setPauseOnExceptionsState(m_scriptDebugServer, ScriptDebugServer::DontPauseOnExceptions);
-#endif
     if (asBool(doNotPauseOnExceptionsAndMuteConsole))
         muteConsole();
 
@@ -136,9 +127,7 @@ void InspectorRuntimeAgent::evaluate(ErrorString* errorString, const String& exp
 
     if (asBool(doNotPauseOnExceptionsAndMuteConsole)) {
         unmuteConsole();
-#if ENABLE(JAVASCRIPT_DEBUGGER)
         setPauseOnExceptionsState(m_scriptDebugServer, previousPauseOnExceptionsState);
-#endif
     }
 }
 
@@ -154,11 +143,9 @@ void InspectorRuntimeAgent::callFunctionOn(ErrorString* errorString, const Strin
     if (optionalArguments)
         arguments = (*optionalArguments)->toJSONString();
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
     ScriptDebugServer::PauseOnExceptionsState previousPauseOnExceptionsState = ScriptDebugServer::DontPauseOnExceptions;
     if (asBool(doNotPauseOnExceptionsAndMuteConsole))
         previousPauseOnExceptionsState = setPauseOnExceptionsState(m_scriptDebugServer, ScriptDebugServer::DontPauseOnExceptions);
-#endif
     if (asBool(doNotPauseOnExceptionsAndMuteConsole))
         muteConsole();
 
@@ -166,9 +153,7 @@ void InspectorRuntimeAgent::callFunctionOn(ErrorString* errorString, const Strin
 
     if (asBool(doNotPauseOnExceptionsAndMuteConsole)) {
         unmuteConsole();
-#if ENABLE(JAVASCRIPT_DEBUGGER)
         setPauseOnExceptionsState(m_scriptDebugServer, previousPauseOnExceptionsState);
-#endif
     }
 }
 
@@ -180,18 +165,14 @@ void InspectorRuntimeAgent::getProperties(ErrorString* errorString, const String
         return;
     }
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
     ScriptDebugServer::PauseOnExceptionsState previousPauseOnExceptionsState = setPauseOnExceptionsState(m_scriptDebugServer, ScriptDebugServer::DontPauseOnExceptions);
-#endif
     muteConsole();
 
     injectedScript.getProperties(errorString, objectId, ownProperties ? *ownProperties : false, &result);
     injectedScript.getInternalProperties(errorString, objectId, &internalProperties);
 
     unmuteConsole();
-#if ENABLE(JAVASCRIPT_DEBUGGER)
     setPauseOnExceptionsState(m_scriptDebugServer, previousPauseOnExceptionsState);
-#endif
 }
 
 void InspectorRuntimeAgent::releaseObject(ErrorString*, const String& objectId)
index 847e2a6696803b1f9efd7251cd8f5f89534af000..2bc6e2994a9d2da186e2ea8bad0721370bd6ea18 100644 (file)
@@ -67,9 +67,7 @@ public:
     virtual void releaseObjectGroup(ErrorString*, const String& objectGroup) override final;
     virtual void run(ErrorString*) override;
 
-#if ENABLE(JAVASCRIPT_DEBUGGER)
     void setScriptDebugServer(ScriptDebugServer* scriptDebugServer) { m_scriptDebugServer = scriptDebugServer; }
-#endif
 
     bool enabled() const { return m_enabled; }
 
@@ -86,9 +84,7 @@ protected:
 
 private:
     InjectedScriptManager* m_injectedScriptManager;
-#if ENABLE(JAVASCRIPT_DEBUGGER)
     ScriptDebugServer* m_scriptDebugServer;
-#endif
     bool m_enabled;
 };