Search fields render placeholder text improperly.
authorwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Jul 2015 18:47:12 +0000 (18:47 +0000)
committerwenson_hsieh@apple.com <wenson_hsieh@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Jul 2015 18:47:12 +0000 (18:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=147192
<rdar://problem/21901076>

Reviewed by Alexey Proskuryakov.

Due to changes in the way AppKit renders search inputs, we must now explicitly
set the placeholder text of a search input rendered using the Mac theme to be
an empty string when rendering the search input box (not including the actual
placeholder text).

* rendering/RenderThemeMac.mm:
(WebCore::RenderThemeMac::setSearchCellState): Force the placeholder text of
    the NSSearchFieldCell for the Mac theme to be an empty string.

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderThemeMac.mm

index c27f765..4b453f7 100644 (file)
@@ -1,3 +1,20 @@
+2015-07-22  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        Search fields render placeholder text improperly.
+        https://bugs.webkit.org/show_bug.cgi?id=147192
+        <rdar://problem/21901076>
+
+        Reviewed by Alexey Proskuryakov.
+
+        Due to changes in the way AppKit renders search inputs, we must now explicitly
+        set the placeholder text of a search input rendered using the Mac theme to be
+        an empty string when rendering the search input box (not including the actual
+        placeholder text).
+
+        * rendering/RenderThemeMac.mm:
+        (WebCore::RenderThemeMac::setSearchCellState): Force the placeholder text of
+            the NSSearchFieldCell for the Mac theme to be an empty string.
+
 2015-07-22  Matthew Daiter  <mdaiter@apple.com>
 
         Need the ability to give only best source UIDs to UserMedia request
index 4795ac8..d2af123 100644 (file)
@@ -1631,6 +1631,7 @@ void RenderThemeMac::setSearchCellState(const RenderObject& o, const IntRect&)
 {
     NSSearchFieldCell* search = this->search();
 
+    [search setPlaceholderString:@""];
     [search setControlSize:controlSizeForFont(o.style())];
 
     // Update the various states we respond to.