[REGRESSION][WTR] Build fail with werror after r120054
authorkbalazs@webkit.org <kbalazs@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Jun 2012 12:51:16 +0000 (12:51 +0000)
committerkbalazs@webkit.org <kbalazs@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Jun 2012 12:51:16 +0000 (12:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=88862

Reviewed by Csaba Osztrogonác.

Some static helpers become unused after counterValueForElementById
has moved to Internals. Let's remove them.

* WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
(WTR):

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

Tools/ChangeLog
Tools/WebKitTestRunner/InjectedBundle/LayoutTestController.cpp

index ee43069..ef744b7 100644 (file)
@@ -1,3 +1,16 @@
+2012-06-12  Balazs Kelemen  <kbalazs@webkit.org>
+
+        [REGRESSION][WTR] Build fail with werror after r120054
+        https://bugs.webkit.org/show_bug.cgi?id=88862
+
+        Reviewed by Csaba Osztrogonác.
+
+        Some static helpers become unused after counterValueForElementById
+        has moved to Internals. Let's remove them.
+
+        * WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
+        (WTR):
+
 2012-06-12  Csaba Osztrogonác  <ossy@webkit.org>
 
         [Qt] Slave losts cause build break on bots
index f34ce1d..df9dfa4 100644 (file)
@@ -52,40 +52,6 @@ namespace WTR {
 // Eventually it should be changed to match.
 const double LayoutTestController::waitToDumpWatchdogTimerInterval = 6;
 
-static JSValueRef propertyValue(JSContextRef context, JSObjectRef object, const char* propertyName)
-{
-    if (!object)
-        return 0;
-    JSRetainPtr<JSStringRef> propertyNameString(Adopt, JSStringCreateWithUTF8CString(propertyName));
-    JSValueRef exception;
-    return JSObjectGetProperty(context, object, propertyNameString.get(), &exception);
-}
-
-static JSObjectRef propertyObject(JSContextRef context, JSObjectRef object, const char* propertyName)
-{
-    JSValueRef value = propertyValue(context, object, propertyName);
-    if (!value || !JSValueIsObject(context, value))
-        return 0;
-    return const_cast<JSObjectRef>(value);
-}
-
-static JSObjectRef getElementById(WKBundleFrameRef frame, JSStringRef elementId)
-{
-    JSContextRef context = WKBundleFrameGetJavaScriptContext(frame);
-    JSObjectRef document = propertyObject(context, JSContextGetGlobalObject(context), "document");
-    if (!document)
-        return 0;
-    JSValueRef getElementById = propertyObject(context, document, "getElementById");
-    if (!getElementById || !JSValueIsObject(context, getElementById))
-        return 0;
-    JSValueRef elementIdValue = JSValueMakeString(context, elementId);
-    JSValueRef exception;
-    JSValueRef element = JSObjectCallAsFunction(context, const_cast<JSObjectRef>(getElementById), document, 1, &elementIdValue, &exception);
-    if (!element || !JSValueIsObject(context, element))
-        return 0;
-    return const_cast<JSObjectRef>(element);
-}
-
 PassRefPtr<LayoutTestController> LayoutTestController::create()
 {
     return adoptRef(new LayoutTestController);