Changeset 88193 in webkit


Ignore:
Timestamp:
Jun 6, 2011 3:00:10 PM (13 years ago)
Author:
leviw@chromium.org
Message:

2011-06-06 Levi Weintraub <leviw@chromium.org>

Reviewed by Eric Seidel.

Switch paintScrollbar to use IntPoint
https://bugs.webkit.org/show_bug.cgi?id=62151

Switching paintScrollbar to use IntPoint instead of a pair of ints.

No new tests since this is simple refactoring.

  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::paintObject): (WebCore::RenderListBox::paintScrollbar):
  • rendering/RenderListBox.h:
Location:
trunk/Source/WebCore
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r88192 r88193  
     12011-06-06  Levi Weintraub  <leviw@chromium.org>
     2
     3        Reviewed by Eric Seidel.
     4
     5        Switch paintScrollbar to use IntPoint
     6        https://bugs.webkit.org/show_bug.cgi?id=62151
     7
     8        Switching paintScrollbar to use IntPoint instead of a pair of ints.
     9
     10        No new tests since this is simple refactoring.
     11
     12        * rendering/RenderListBox.cpp:
     13        (WebCore::RenderListBox::paintObject):
     14        (WebCore::RenderListBox::paintScrollbar):
     15        * rendering/RenderListBox.h:
     16
    1172011-06-06  Daniel Cheng  <dcheng@chromium.org>
    218
  • trunk/Source/WebCore/rendering/RenderListBox.cpp

    r88087 r88193  
    280280    case PaintPhaseForeground:
    281281        if (m_vBar->isOverlayScrollbar())
    282             paintScrollbar(paintInfo, paintOffset.x(), paintOffset.y());
     282            paintScrollbar(paintInfo, paintOffset);
    283283        break;
    284284    case PaintPhaseBlockBackground:
    285285        if (!m_vBar->isOverlayScrollbar())
    286             paintScrollbar(paintInfo, paintOffset.x(), paintOffset.y());
     286            paintScrollbar(paintInfo, paintOffset);
    287287        break;
    288288    case PaintPhaseChildBlockBackground:
     
    326326}
    327327
    328 void RenderListBox::paintScrollbar(PaintInfo& paintInfo, int tx, int ty)
     328void RenderListBox::paintScrollbar(PaintInfo& paintInfo, const IntPoint& paintOffset)
    329329{
    330330    if (m_vBar) {
    331         IntRect scrollRect(tx + width() - borderRight() - m_vBar->width(),
    332                            ty + borderTop(),
     331        IntRect scrollRect(paintOffset.x() + width() - borderRight() - m_vBar->width(),
     332                           paintOffset.y() + borderTop(),
    333333                           m_vBar->width(),
    334334                           height() - (borderTop() + borderBottom()));
  • trunk/Source/WebCore/rendering/RenderListBox.h

    r88178 r88193  
    133133    int numItems() const;
    134134    int listHeight() const;
    135     void paintScrollbar(PaintInfo&, int tx, int ty);
     135    void paintScrollbar(PaintInfo&, const IntPoint&);
    136136    void paintItemForeground(PaintInfo&, const IntPoint&, int listIndex);
    137137    void paintItemBackground(PaintInfo&, const IntPoint&, int listIndex);
Note: See TracChangeset for help on using the changeset viewer.