Changeset 76620 in webkit

Timestamp:

Jan 25, 2011 11:39:58 AM (13 years ago)

Author:

tony@chromium.org

Message:

2011-01-25 Tony Chang <​tony@chromium.org>

Reviewed by Dimitri Glazkov.

[chromium] remove unused layoutTestController callback
​https://bugs.webkit.org/show_bug.cgi?id=53103

This method is used by a test_shell_test, but not needed by DRT.

• DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::LayoutTestController):
• DumpRenderTree/chromium/LayoutTestController.h:

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/chromium/LayoutTestController.cpp (modified) (2 diffs)
• DumpRenderTree/chromium/LayoutTestController.h (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2011-01-25  Tony Chang  <tony@chromium.org>
+
+        Reviewed by Dimitri Glazkov.
+
+        [chromium] remove unused layoutTestController callback
+        https://bugs.webkit.org/show_bug.cgi?id=53103
+
+        This method is used by a test_shell_test, but not needed by DRT.
+
+        * DumpRenderTree/chromium/LayoutTestController.cpp:
+        (LayoutTestController::LayoutTestController):
+        * DumpRenderTree/chromium/LayoutTestController.h:
+
 2011-01-25  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
 

trunk/Tools/DumpRenderTree/chromium/LayoutTestController.cpp

@@ -107 +107 @@
     bindMethod("evaluateScriptInIsolatedWorld", &LayoutTestController::evaluateScriptInIsolatedWorld);
     bindMethod("execCommand", &LayoutTestController::execCommand);
-    bindMethod("forceRedSelectionColors", &LayoutTestController::forceRedSelectionColors);
     bindMethod("grantDesktopNotificationPermission", &LayoutTestController::grantDesktopNotificationPermission);
     bindMethod("isCommandEnabled", &LayoutTestController::isCommandEnabled);
@@ -1465 +1464 @@
 }
 
-void LayoutTestController::forceRedSelectionColors(const CppArgumentList& arguments, CppVariant* result)
-{
-    result->setNull();
-    m_shell->webView()->setSelectionColors(0xffee0000, 0xff00ee00, 0xff000000, 0xffc0c0c0);
-}
-
 void LayoutTestController::addUserScript(const CppArgumentList& arguments, CppVariant* result)
 {

trunk/Tools/DumpRenderTree/chromium/LayoutTestController.h

@@ -312 +312 @@
     void evaluateInWebInspector(const CppArgumentList&, CppVariant*);
 
-    // Forces the selection colors for testing under Linux.
-    void forceRedSelectionColors(const CppArgumentList&, CppVariant*);
-
     // Adds a user script or user style sheet to be injected into new documents.
     void addUserScript(const CppArgumentList&, CppVariant*);