2009-01-09 Adele Peterson <adele@apple.com>
authoradele@apple.com <adele@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 10 Jan 2009 01:12:04 +0000 (01:12 +0000)
committeradele@apple.com <adele@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 10 Jan 2009 01:12:04 +0000 (01:12 +0000)
        Reviewed by Darin Adler.

        Fix infinite recursion in clientPaddingLeft and clientPaddingRight.  This changes the code back to
        how it was before RenderTextControlSingleLine was split out from RenderTextControl.

        No test added since this code is only exercised when trying to place a search field's recent searches popup menu.

        * rendering/RenderTextControlSingleLine.cpp:
        (WebCore::RenderTextControlSingleLine::clientPaddingLeft):
        (WebCore::RenderTextControlSingleLine::clientPaddingRight):

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

WebCore/ChangeLog
WebCore/rendering/RenderTextControlSingleLine.cpp

index 7b9fa7f..e2a7697 100644 (file)
@@ -1,3 +1,16 @@
+2009-01-09  Adele Peterson  <adele@apple.com>
+
+        Reviewed by Darin Adler.
+
+        Fix infinite recursion in clientPaddingLeft and clientPaddingRight.  This changes the code back to
+        how it was before RenderTextControlSingleLine was split out from RenderTextControl.
+        
+        No test added since this code is only exercised when trying to place a search field's recent searches popup menu.
+
+        * rendering/RenderTextControlSingleLine.cpp:
+        (WebCore::RenderTextControlSingleLine::clientPaddingLeft):
+        (WebCore::RenderTextControlSingleLine::clientPaddingRight):
+
 2009-01-09  Darin Adler  <darin@apple.com>
 
         Reviewed by Jon Honeycutt.
index 4f52527..04c9701 100644 (file)
@@ -682,7 +682,7 @@ int RenderTextControlSingleLine::clientInsetRight() const
 
 int RenderTextControlSingleLine::clientPaddingLeft() const
 {
-    int padding = clientPaddingLeft();
+    int padding = paddingLeft();
 
     if (RenderObject* resultsRenderer = m_resultsButton ? m_resultsButton->renderer() : 0)
         padding += resultsRenderer->width();
@@ -692,7 +692,7 @@ int RenderTextControlSingleLine::clientPaddingLeft() const
 
 int RenderTextControlSingleLine::clientPaddingRight() const
 {
-    int padding = clientPaddingRight();
+    int padding = paddingRight();
 
     if (RenderObject* cancelRenderer = m_cancelButton ? m_cancelButton->renderer() : 0)
         padding += cancelRenderer->width();