Changeset 108296 in webkit

Timestamp:

Feb 20, 2012, 8:17:26 PM (14 years ago)

Author:

commit-queue@webkit.org

Message:

[BlackBerry] Need to override active/inactive text search highlight color for RenderThemeBlackBerry
​https://bugs.webkit.org/show_bug.cgi?id=78920

Override two functions for text search highlight colors so that we can
provide platform specified colors.

Patch by Andy Chen <​andchen@rim.com> on 2012-02-17
Reviewed by Antonio Gomes.

No function change so no new tests.

• platform/blackberry/RenderThemeBlackBerry.cpp:

(WebCore::RenderThemeBlackBerry::platformActiveTextSearchHighlightColor):
(WebCore::RenderThemeBlackBerry::platformInactiveTextSearchHighlightColor):

• platform/blackberry/RenderThemeBlackBerry.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/blackberry/RenderThemeBlackBerry.cpp (modified) (1 diff)
• platform/blackberry/RenderThemeBlackBerry.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-02-20  Andy Chen  <andchen@rim.com>
+
+        2012-02-17  Andy Chen  <andchen@rim.com>
+
+        [BlackBerry] Need to override active/inactive text search highlight color for RenderThemeBlackBerry
+        https://bugs.webkit.org/show_bug.cgi?id=78920
+
+        Override two functions for text search highlight colors so that we can
+        provide platform specified colors.
+
+        Reviewed by Antonio Gomes.
+
+        No function change so no new tests.
+
+        * platform/blackberry/RenderThemeBlackBerry.cpp:
+        (WebCore::RenderThemeBlackBerry::platformActiveTextSearchHighlightColor):
+        (WebCore::RenderThemeBlackBerry::platformInactiveTextSearchHighlightColor):
+        * platform/blackberry/RenderThemeBlackBerry.h:
+
 2012-02-20  John Knottenbelt  <jknotten@chromium.org>
 

trunk/Source/WebCore/platform/blackberry/RenderThemeBlackBerry.cpp

@@ -966 +966 @@
 }
 
+Color RenderThemeBlackBerry::platformActiveTextSearchHighlightColor() const
+{
+    return Color(255, 150, 50); // Orange.
+}
+
+Color RenderThemeBlackBerry::platformInactiveTextSearchHighlightColor() const
+{
+    return Color(255, 255, 0); // Yellow.
+}
+
 } // namespace WebCore

trunk/Source/WebCore/platform/blackberry/RenderThemeBlackBerry.h

@@ -89 +89 @@
     virtual Color platformActiveSelectionBackgroundColor() const;
 
+    // Highlighting colors for TextMatches.
+    virtual Color platformActiveTextSearchHighlightColor() const;
+    virtual Color platformInactiveTextSearchHighlightColor() const;
+
 private:
     static const String& defaultGUIFont();