Reviewed by Adam Roben.

        Fix drawSelectionRect to actually draw the selection rect.

        * DumpRenderTree/cg/PixelDumpSupportCG.cpp:
        (drawSelectionRect):

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

diff --git a/WebKitTools/ChangeLog b/WebKitTools/ChangeLog
index da7458537cabf2049da6cdd9b10761d4446e05f4..741c30df7b787630d7dba6c22dd4b11525f21080 100644 (file)
--- a/WebKitTools/ChangeLog
+++ b/WebKitTools/ChangeLog
@@ -1,3 +1,12 @@
+2007-11-30  Sam Weinig  <sam@webkit.org>
+
+        Reviewed by Adam Roben.
+
+        Fix drawSelectionRect to actually draw the selection rect.
+
+        * DumpRenderTree/cg/PixelDumpSupportCG.cpp:
+        (drawSelectionRect):
+
 2007-11-30  Darin Adler  <darin@apple.com>
 
         * Scripts/do-webcore-rename: WildFox already did the TextStyle -> FontStyle one.

diff --git a/WebKitTools/DumpRenderTree/cg/PixelDumpSupportCG.cpp b/WebKitTools/DumpRenderTree/cg/PixelDumpSupportCG.cpp
index 6e3db58ce10fa408d87146eecdbd014dc37ac01c..e2e790c2575882c5f7555d2ba66067a292567fec 100644 (file)
--- a/WebKitTools/DumpRenderTree/cg/PixelDumpSupportCG.cpp
+++ b/WebKitTools/DumpRenderTree/cg/PixelDumpSupportCG.cpp
@@ -87,10 +87,8 @@ static void getMD5HashStringForBitmap(CGContextRef bitmap, char string[33])
 
 void drawSelectionRect(CGContextRef context, const CGRect& rect)
 {
-    const CGFloat redColor[4] = { 1.0, 0.0, 0.0, 0.0 };
-
     CGContextSaveGState(context);
-    CGContextSetStrokeColor(context, redColor);
+    CGContextSetRGBStrokeColor(context, 1.0, 0.0, 0.0, 1.0);
     CGContextStrokeRect(context, rect);
     CGContextRestoreGState(context);
 }