Changeset 91118 in webkit


Ignore:
Timestamp:
Jul 15, 2011 3:13:24 PM (13 years ago)
Author:
tony@chromium.org
Message:

[chromium] land linux selection color layout test
​https://bugs.webkit.org/show_bug.cgi?id=64631

Reviewed by Ojan Vafai.

Tools:

I had removed this code in r76620 because I thought it wasn't used.
Turns out there was a layout test in the chromium tree that never
got migrated that used it. Re-add the code and land the layout
test (I will remove it from the chromium tree soon).

  • DumpRenderTree/chromium/LayoutTestController.cpp:

(LayoutTestController::LayoutTestController):
(LayoutTestController::forceRedSelectionColors):

  • DumpRenderTree/chromium/LayoutTestController.h:

LayoutTests:

  • platform/chromium-linux/editing/selection/linux_selection_color.html: Added.
Location:
trunk
Files:
1 added
4 edited
2 moved

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r91117 r91118  
     12011-07-15  Tony Chang  <tony@chromium.org>
     2
     3        [chromium] land linux selection color layout test
     4        https://bugs.webkit.org/show_bug.cgi?id=64631
     5
     6        Reviewed by Ojan Vafai.
     7
     8        * platform/chromium-linux/editing/selection/linux_selection_color.html: Added.
     9
    1102011-07-15  Vincent Scheib  <scheib@chromium.org>
    211

  • trunk/Tools/ChangeLog

    r91106 r91118  
     12011-07-15  Tony Chang  <tony@chromium.org>
     2
     3        [chromium] land linux selection color layout test
     4        https://bugs.webkit.org/show_bug.cgi?id=64631
     5
     6        Reviewed by Ojan Vafai.
     7
     8        I had removed this code in r76620 because I thought it wasn't used.
     9        Turns out there was a layout test in the chromium tree that never
     10        got migrated that used it.  Re-add the code and land the layout
     11        test (I will remove it from the chromium tree soon).
     12
     13        * DumpRenderTree/chromium/LayoutTestController.cpp:
     14        (LayoutTestController::LayoutTestController):
     15        (LayoutTestController::forceRedSelectionColors):
     16        * DumpRenderTree/chromium/LayoutTestController.h:
     17
    1182011-07-15  Dimitri Glazkov  <dglazkov@chromium.org>
    219

  • trunk/Tools/DumpRenderTree/chromium/LayoutTestController.cpp

    r90137 r91118  
    113113    bindMethod("setIsolatedWorldSecurityOrigin", &LayoutTestController::setIsolatedWorldSecurityOrigin);
    114114    bindMethod("execCommand", &LayoutTestController::execCommand);
     115    bindMethod("forceRedSelectionColors", &LayoutTestController::forceRedSelectionColors);
    115116    bindMethod("grantDesktopNotificationPermission", &LayoutTestController::grantDesktopNotificationPermission);
    116117    bindMethod("hasSpellingMarker", &LayoutTestController::hasSpellingMarker);
    … …  
    16001601}
    16011602
     1603void LayoutTestController::forceRedSelectionColors(const CppArgumentList& arguments, CppVariant* result)
     1604{
     1605    result->setNull();
     1606    m_shell->webView()->setSelectionColors(0xffee0000, 0xff00ee00, 0xff000000, 0xffc0c0c0);
     1607}
     1608
    16021609void LayoutTestController::addUserScript(const CppArgumentList& arguments, CppVariant* result)
    16031610{

  • trunk/Tools/DumpRenderTree/chromium/LayoutTestController.h

    r90137 r91118  
    334334    // Allows layout tests to exec scripts at WebInspector side.
    335335    void evaluateInWebInspector(const CppArgumentList&, CppVariant*);
     336
     337    // Forces the selection colors for testing under Linux.
     338    void forceRedSelectionColors(const CppArgumentList&, CppVariant*);
    336339
    337340    // Adds a user script or user style sheet to be injected into new documents.
Note: See TracChangeset for help on using the changeset viewer.