Changeset 45624 in webkit


Ignore:
Timestamp:
Jul 8, 2009 6:55:55 AM (15 years ago)
Author:
ddkilzer@apple.com
Message:

2009-07-08 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by David Kilzer.

WebKit needs a style linting tool
​https://bugs.webkit.org/show_bug.cgi?id=25884

Fix bunch of style issues in WebCore/rendering.
This patch is created to demonstrate cpplint.py.

No testcase because it's just a style fixes.

  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::recalcColumn): (WebCore::AutoTableLayout::layout):
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::placeEllipsisBox):
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paintTextMatchMarker):
  • rendering/MediaControlElements.cpp: (WebCore::MediaControlTimelineElement::defaultEventHandler):
  • rendering/MediaControlElements.h:
  • rendering/RenderArena.cpp:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::startDelayUpdateScrollInfo): (WebCore::RenderBlock::finishDelayUpdateScrollInfo): (WebCore::RenderBlock::updateScrollInfoAfterLayout): (WebCore::RenderBlock::positionNewFloats): (WebCore::RenderBlock::newLine): (WebCore::RenderBlock::floatBottom): (WebCore::RenderBlock::leftBottom): (WebCore::RenderBlock::rightBottom):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcReplacedWidthUsing):
  • rendering/RenderFieldset.cpp: (WebCore::RenderFieldset::layoutLegend):
  • rendering/RenderFlexibleBox.cpp: (WebCore::FlexBoxIterator::FlexBoxIterator): (WebCore::FlexBoxIterator::reset): (WebCore::FlexBoxIterator::first): (WebCore::FlexBoxIterator::next): (WebCore::RenderFlexibleBox::layoutVerticalBox):
  • rendering/RenderFrameSet.cpp: (WebCore::borderStartEdgeColor):
  • rendering/RenderFrameSet.h:
  • rendering/RenderImage.cpp:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateVisibilityStatus): (WebCore::RenderLayer::calculateClipRects): (WebCore::RenderLayer::calculateRects):
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::panScroll):
  • rendering/RenderMarquee.cpp: (WebCore::RenderMarquee::updateMarqueeStyle):
  • rendering/RenderMedia.cpp: (WebCore::RenderMedia::updateControls):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::drawLineForBoxSide): (WebCore::RenderObject::localCaretRect):
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::adjustRectsForAspectRatio):
  • rendering/RenderSlider.h:
  • rendering/RenderTable.cpp: (WebCore::RenderTable::outerBorderBottom):
  • rendering/RenderTableCol.h:
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::itemIsSeparator):
  • rendering/RenderThemeChromiumSkia.cpp: (WebCore::RenderThemeChromiumSkia::supportsHover):
  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::supportsFocusRing):
  • rendering/SVGCharacterLayoutInfo.cpp: (WebCore::SVGCharacterLayoutInfo::addStackContent):
  • rendering/SVGCharacterLayoutInfo.h:
  • rendering/TextControlInnerElements.h:
  • rendering/bidi.cpp: (WebCore::RenderBlock::computeHorizontalPositionsForLine):
Location:
trunk/WebCore
Files:
30 edited

Legend:

Unmodified
Added
Removed

  • trunk/WebCore/ChangeLog

    r45620 r45624  
     12009-07-08  Shinichiro Hamaji  <hamaji@chromium.org>
     2
     3        Reviewed by David Kilzer.
     4
     5        WebKit needs a style linting tool
     6        https://bugs.webkit.org/show_bug.cgi?id=25884
     7
     8        Fix bunch of style issues in WebCore/rendering.
     9        This patch is created to demonstrate cpplint.py.
     10
     11        No testcase because it's just a style fixes.
     12
     13        * rendering/AutoTableLayout.cpp:
     14        (WebCore::AutoTableLayout::recalcColumn):
     15        (WebCore::AutoTableLayout::layout):
     16        * rendering/InlineFlowBox.cpp:
     17        (WebCore::InlineFlowBox::placeEllipsisBox):
     18        * rendering/InlineTextBox.cpp:
     19        (WebCore::InlineTextBox::paintTextMatchMarker):
     20        * rendering/MediaControlElements.cpp:
     21        (WebCore::MediaControlTimelineElement::defaultEventHandler):
     22        * rendering/MediaControlElements.h:
     23        * rendering/RenderArena.cpp:
     24        * rendering/RenderBlock.cpp:
     25        (WebCore::RenderBlock::startDelayUpdateScrollInfo):
     26        (WebCore::RenderBlock::finishDelayUpdateScrollInfo):
     27        (WebCore::RenderBlock::updateScrollInfoAfterLayout):
     28        (WebCore::RenderBlock::positionNewFloats):
     29        (WebCore::RenderBlock::newLine):
     30        (WebCore::RenderBlock::floatBottom):
     31        (WebCore::RenderBlock::leftBottom):
     32        (WebCore::RenderBlock::rightBottom):
     33        * rendering/RenderBox.cpp:
     34        (WebCore::RenderBox::calcReplacedWidthUsing):
     35        * rendering/RenderFieldset.cpp:
     36        (WebCore::RenderFieldset::layoutLegend):
     37        * rendering/RenderFlexibleBox.cpp:
     38        (WebCore::FlexBoxIterator::FlexBoxIterator):
     39        (WebCore::FlexBoxIterator::reset):
     40        (WebCore::FlexBoxIterator::first):
     41        (WebCore::FlexBoxIterator::next):
     42        (WebCore::RenderFlexibleBox::layoutVerticalBox):
     43        * rendering/RenderFrameSet.cpp:
     44        (WebCore::borderStartEdgeColor):
     45        * rendering/RenderFrameSet.h:
     46        * rendering/RenderImage.cpp:
     47        * rendering/RenderLayer.cpp:
     48        (WebCore::RenderLayer::updateVisibilityStatus):
     49        (WebCore::RenderLayer::calculateClipRects):
     50        (WebCore::RenderLayer::calculateRects):
     51        * rendering/RenderListBox.cpp:
     52        (WebCore::RenderListBox::panScroll):
     53        * rendering/RenderMarquee.cpp:
     54        (WebCore::RenderMarquee::updateMarqueeStyle):
     55        * rendering/RenderMedia.cpp:
     56        (WebCore::RenderMedia::updateControls):
     57        * rendering/RenderObject.cpp:
     58        (WebCore::RenderObject::drawLineForBoxSide):
     59        (WebCore::RenderObject::localCaretRect):
     60        * rendering/RenderSVGImage.cpp:
     61        (WebCore::RenderSVGImage::adjustRectsForAspectRatio):
     62        * rendering/RenderSlider.h:
     63        * rendering/RenderTable.cpp:
     64        (WebCore::RenderTable::outerBorderBottom):
     65        * rendering/RenderTableCol.h:
     66        * rendering/RenderTextControlSingleLine.cpp:
     67        (WebCore::RenderTextControlSingleLine::itemIsSeparator):
     68        * rendering/RenderThemeChromiumSkia.cpp:
     69        (WebCore::RenderThemeChromiumSkia::supportsHover):
     70        * rendering/RenderThemeChromiumWin.cpp:
     71        (WebCore::RenderThemeChromiumWin::supportsFocusRing):
     72        * rendering/SVGCharacterLayoutInfo.cpp:
     73        (WebCore::SVGCharacterLayoutInfo::addStackContent):
     74        * rendering/SVGCharacterLayoutInfo.h:
     75        * rendering/TextControlInnerElements.h:
     76        * rendering/bidi.cpp:
     77        (WebCore::RenderBlock::computeHorizontalPositionsForLine):
     78
    1792009-07-07  Oliver Hunt  <oliver@apple.com>
    280

  • trunk/WebCore/rendering/AutoTableLayout.cpp

    r45015 r45624  
    9595                    if (w.isNegative())
    9696                        w.setValue(0);
    97                     switch(w.type()) {
     97                    switch (w.type()) {
    9898                    case Fixed:
    9999                        // ignore width=0
    … …  
    574574                    // the lines below might look inconsistent, but that's the way it's handled in mozilla
    575575                    excess -= reduction;
    576                     int newWidth = max(int (m_layoutStruct[i].effMinWidth), w - reduction);
     576                    int newWidth = max(static_cast<int>(m_layoutStruct[i].effMinWidth), w - reduction);
    577577                    available += w - newWidth;
    578578                    m_layoutStruct[i].calcWidth = newWidth;

  • trunk/WebCore/rendering/InlineFlowBox.cpp

    r45317 r45624  
    10381038    int visibleRightEdge = blockRightEdge;
    10391039
    1040     while(box) {
     1040    while (box) {
    10411041        int currResult = box->placeEllipsisBox(ltr, visibleLeftEdge, visibleRightEdge, ellipsisWidth, foundBox);
    10421042        if (currResult != -1 && result == -1)

  • trunk/WebCore/rendering/InlineTextBox.cpp

    r44763 r45624  
    754754void InlineTextBox::paintTextMatchMarker(GraphicsContext* pt, int tx, int ty, DocumentMarker marker, RenderStyle* style, const Font& font)
    755755{
    756    // Use same y positioning and height as for selection, so that when the selection and this highlight are on
    757    // the same word there are no pieces sticking out.
     756    // Use same y positioning and height as for selection, so that when the selection and this highlight are on
     757    // the same word there are no pieces sticking out.
    758758    int y = selectionTop();
    759759    int h = selectionHeight();

  • trunk/WebCore/rendering/MediaControlElements.cpp

    r45614 r45624  
    471471    MediaControlInputElement::defaultEventHandler(event);
    472472
    473      if (event->type() == eventNames().mouseoverEvent || event->type() == eventNames().mouseoutEvent || event->type() == eventNames().mousemoveEvent ) {
     473    if (event->type() == eventNames().mouseoverEvent || event->type() == eventNames().mouseoutEvent || event->type() == eventNames().mousemoveEvent) {
    474474        return;
    475475    }

  • trunk/WebCore/rendering/MediaControlElements.h

    r45614 r45624  
    7979// ----------------------------
    8080
    81 class MediaControlElement : public HTMLDivElement
    82 {
     81class MediaControlElement : public HTMLDivElement {
    8382public:
    8483    MediaControlElement(Document*, PseudoId, HTMLMediaElement*);

  • trunk/WebCore/rendering/RenderArena.cpp

    r40229 r45624  
    4040#include <wtf/Assertions.h>
    4141
    42 #define ROUNDUP(x,y) ((((x)+((y)-1))/(y))*(y))
     42#define ROUNDUP(x, y) ((((x)+((y)-1))/(y))*(y))
    4343
    4444namespace WebCore {

  • trunk/WebCore/rendering/RenderBlock.cpp

    r45608 r45624  
    693693}
    694694
    695 void RenderBlock::startDelayUpdateScrollInfo() {
     695void RenderBlock::startDelayUpdateScrollInfo()
     696{
    696697    if (gDelayUpdateScrollInfo == 0) {
    697698        ASSERT(!gDelayedUpdateScrollInfoSet);
    … …  
    702703}
    703704
    704 void RenderBlock::finishDelayUpdateScrollInfo() {
     705void RenderBlock::finishDelayUpdateScrollInfo()
     706{
    705707    --gDelayUpdateScrollInfo;
    706708    ASSERT(gDelayUpdateScrollInfo >= 0);
    … …  
    720722}
    721723
    722 void RenderBlock::updateScrollInfoAfterLayout() {
     724void RenderBlock::updateScrollInfoAfterLayout()
     725{
    723726    if (hasOverflowClip()) {
    724727        if (gDelayUpdateScrollInfo)
    … …  
    24622465            int heightRemainingLeft = 1;
    24632466            int heightRemainingRight = 1;
    2464             int fx = leftRelOffset(y,lo, false, &heightRemainingLeft);
    2465             while (rightRelOffset(y,ro, false, &heightRemainingRight)-fx < fwidth) {
     2467            int fx = leftRelOffset(y, lo, false, &heightRemainingLeft);
     2468            while (rightRelOffset(y, ro, false, &heightRemainingRight)-fx < fwidth) {
    24662469                y += min(heightRemainingLeft, heightRemainingRight);
    2467                 fx = leftRelOffset(y,lo, false, &heightRemainingLeft);
     2470                fx = leftRelOffset(y, lo, false, &heightRemainingLeft);
    24682471            }
    24692472            fx = max(0, fx);
    … …  
    24732476            int heightRemainingLeft = 1;
    24742477            int heightRemainingRight = 1;
    2475             int fx = rightRelOffset(y,ro, false, &heightRemainingRight);
    2476             while (fx - leftRelOffset(y,lo, false, &heightRemainingLeft) < fwidth) {
     2478            int fx = rightRelOffset(y, ro, false, &heightRemainingRight);
     2479            while (fx - leftRelOffset(y, lo, false, &heightRemainingLeft) < fwidth) {
    24772480                y += min(heightRemainingLeft, heightRemainingRight);
    24782481                fx = rightRelOffset(y, ro, false, &heightRemainingRight);
    … …  
    24992502    // set y position
    25002503    int newY = 0;
    2501     switch(clear)
     2504    switch (clear)
    25022505    {
    25032506        case CLEFT:
    … …  
    26702673{
    26712674    if (!m_floatingObjects) return 0;
    2672     int bottom=0;
     2675    int bottom = 0;
    26732676    FloatingObject* r;
    26742677    DeprecatedPtrListIterator<FloatingObject> it(*m_floatingObjects);
    26752678    for ( ; (r = it.current()); ++it )
    26762679        if (r->m_bottom>bottom)
    2677             bottom=r->m_bottom;
     2680            bottom = r->m_bottom;
    26782681    return bottom;
    26792682}
    … …  
    29602963{
    29612964    if (!m_floatingObjects) return 0;
    2962     int bottom=0;
     2965    int bottom = 0;
    29632966    FloatingObject* r;
    29642967    DeprecatedPtrListIterator<FloatingObject> it(*m_floatingObjects);
    29652968    for ( ; (r = it.current()); ++it )
    29662969        if (r->m_bottom > bottom && r->type() == FloatingObject::FloatLeft)
    2967             bottom=r->m_bottom;
     2970            bottom = r->m_bottom;
    29682971
    29692972    return bottom;
    … …  
    29742977{
    29752978    if (!m_floatingObjects) return 0;
    2976     int bottom=0;
     2979    int bottom = 0;
    29772980    FloatingObject* r;
    29782981    DeprecatedPtrListIterator<FloatingObject> it(*m_floatingObjects);
    29792982    for ( ; (r = it.current()); ++it )
    29802983        if (r->m_bottom>bottom && r->type() == FloatingObject::FloatRight)
    2981             bottom=r->m_bottom;
     2984            bottom = r->m_bottom;
    29822985
    29832986    return bottom;
    … …  
    39413944}
    39423945
    3943 struct InlineMinMaxIterator
    3944 {
     3946struct InlineMinMaxIterator {
    39453947/* InlineMinMaxIterator is a class that will iterate over all render objects that contribute to
    39463948   inline min/max width calculations.  Note the following about the way it walks:

  • trunk/WebCore/rendering/RenderBox.cpp

    r45332 r45624  
    15631563            return intrinsicSize().width();
    15641564     }
    1565  }
     1565}
    15661566
    15671567int RenderBox::calcReplacedHeight() const

  • trunk/WebCore/rendering/RenderFieldset.cpp

    r45317 r45624  
    100100        int h = legend->height();
    101101        legend->setLocation(xPos, max((b-h)/2, 0));
    102         setHeight(max(b,h) + paddingTop());
     102        setHeight(max(b, h) + paddingTop());
    103103    }
    104104    return legend;

  • trunk/WebCore/rendering/RenderFlexibleBox.cpp

    r45568 r45624  
    3737class FlexBoxIterator {
    3838public:
    39     FlexBoxIterator(RenderFlexibleBox* parent) {
     39    FlexBoxIterator(RenderFlexibleBox* parent)
     40    {
    4041        box = parent;
    4142        if (box->style()->boxOrient() == HORIZONTAL && box->style()->direction() == RTL)
    … …  
    5758    }
    5859
    59     void reset() {
     60    void reset()
     61    {
    6062        current = 0;
    6163        currentOrdinal = forward ? 0 : lastOrdinal+1;
    6264    }
    6365
    64     RenderBox* first() {
     66    RenderBox* first()
     67    {
    6568        reset();
    6669        return next();
    6770    }
    6871   
    69     RenderBox* next() {
     72    RenderBox* next()
     73    {
    7074        do {
    7175            if (!current) {
    … …  
    653657    int xPos = borderLeft() + paddingLeft();
    654658    int yPos = borderTop() + paddingTop();
    655     if( style()->direction() == RTL )
     659    if (style()->direction() == RTL)
    656660        xPos = width() - paddingRight() - borderRight();
    657661    int toAdd = borderBottom() + paddingBottom() + horizontalScrollbarHeight();

  • trunk/WebCore/rendering/RenderFrameSet.cpp

    r41899 r45624  
    6666static Color borderStartEdgeColor()
    6767{
    68     return Color(170,170,170);
     68    return Color(170, 170, 170);
    6969}
    7070

  • trunk/WebCore/rendering/RenderFrameSet.h

    r40651 r45624  
    3333enum FrameEdge { LeftFrameEdge, RightFrameEdge, TopFrameEdge, BottomFrameEdge };
    3434
    35 struct FrameEdgeInfo
    36 {
     35struct FrameEdgeInfo {
    3736    FrameEdgeInfo(bool preventResize = false, bool allowBorder = true)
    3837        : m_preventResize(4)

  • trunk/WebCore/rendering/RenderImage.cpp

    r45143 r45624  
    9090};
    9191
    92 class RenderImageScaleObserver
    93 {
     92class RenderImageScaleObserver {
    9493public:
    9594    static bool shouldImagePaintAtLowQuality(RenderImage*, const IntSize&);

  • trunk/WebCore/rendering/RenderLayer.cpp

    r45613 r45624  
    461461                    do {
    462462                        r = r->parent();
    463                         if (r==renderer())
     463                        if (r == renderer())
    464464                            r = 0;
    465465                    } while (r && !r->nextSibling());
    … …  
    26382638       
    26392639        if (renderer()->hasOverflowClip()) {
    2640             IntRect newOverflowClip = toRenderBox(renderer())->overflowClipRect(x,y);
     2640            IntRect newOverflowClip = toRenderBox(renderer())->overflowClipRect(x, y);
    26412641            clipRects.setOverflowClipRect(intersection(newOverflowClip, clipRects.overflowClipRect()));
    26422642            if (renderer()->isPositioned() || renderer()->isRelPositioned())
    … …  
    26442644        }
    26452645        if (renderer()->hasClip()) {
    2646             IntRect newPosClip = toRenderBox(renderer())->clipRect(x,y);
     2646            IntRect newPosClip = toRenderBox(renderer())->clipRect(x, y);
    26472647            clipRects.setPosClipRect(intersection(newPosClip, clipRects.posClipRect()));
    26482648            clipRects.setOverflowClipRect(intersection(newPosClip, clipRects.overflowClipRect()));
    … …  
    26942694        // This layer establishes a clip of some kind.
    26952695        if (renderer()->hasOverflowClip())
    2696             foregroundRect.intersect(toRenderBox(renderer())->overflowClipRect(x,y));
     2696            foregroundRect.intersect(toRenderBox(renderer())->overflowClipRect(x, y));
    26972697        if (renderer()->hasClip()) {
    26982698            // Clip applies to *us* as well, so go ahead and update the damageRect.
    2699             IntRect newPosClip = toRenderBox(renderer())->clipRect(x,y);
     2699            IntRect newPosClip = toRenderBox(renderer())->clipRect(x, y);
    27002700            backgroundRect.intersect(newPosClip);
    27012701            foregroundRect.intersect(newPosClip);

  • trunk/WebCore/rendering/RenderListBox.cpp

    r45478 r45624  
    417417    int yDelta = currentMousePosition.y() - panStartMousePosition.y();
    418418
    419    // If the point is too far from the center we limit the speed
     419    // If the point is too far from the center we limit the speed
    420420    yDelta = max(min(yDelta, maxSpeed), -maxSpeed);
    421421   
    422     if(abs(yDelta) < iconRadius) // at the center we let the space for the icon
     422    if (abs(yDelta) < iconRadius) // at the center we let the space for the icon
    423423        return;
    424424
    … …  
    426426        //offsetY = view()->viewHeight();
    427427        absOffset.move(0, listHeight());
    428    else if (yDelta < 0)
    429        yDelta--;
     428    else if (yDelta < 0)
     429        yDelta--;
    430430
    431431    // Let's attenuate the speed
    432432    yDelta /= speedReducer;
    433433
    434     IntPoint scrollPoint(0,0);
     434    IntPoint scrollPoint(0, 0);
    435435    scrollPoint.setY(absOffset.y() + yDelta);
    436436    int newOffset = scrollToward(scrollPoint);

  • trunk/WebCore/rendering/RenderMarquee.cpp

    r43889 r45624  
    236236    if (isHorizontal()) {
    237237        if (s->height().isFixed() && s->height().value() < s->fontSize())
    238             s->setHeight(Length(s->fontSize(),Fixed));
     238            s->setHeight(Length(s->fontSize(), Fixed));
    239239    } else if (s->height().isAuto())  //vertical marquee with no specified height
    240240        s->setHeight(Length(200, Fixed));
    … …  
    310310
    311311} // namespace WebCore
    312 

  • trunk/WebCore/rendering/RenderMedia.cpp

    r45614 r45624  
    305305        if (m_timeUpdateTimer.isActive())
    306306            m_timeUpdateTimer.stop();
    307     } else if (style()->visibility() == VISIBLE && m_timeline && m_timeline->renderer() && m_timeline->renderer()->style()->display() != NONE ) {
     307    } else if (style()->visibility() == VISIBLE && m_timeline && m_timeline->renderer() && m_timeline->renderer()->style()->display() != NONE) {
    308308        m_timeUpdateTimer.startRepeating(cTimeUpdateRepeatDelay);
    309309    }

  • trunk/WebCore/rendering/RenderObject.cpp

    r45612 r45624  
    775775                    case BSRight:
    776776                        drawLineForBoxSide(graphicsContext, x1, y1 + max((adjbw1 * 2 + 1) / 3, 0),
    777                                    x1 + third, y2 - max(( adjbw2 * 2 + 1) / 3, 0),
     777                                   x1 + third, y2 - max((adjbw2 * 2 + 1) / 3, 0),
    778778                                   s, c, textcolor, SOLID, adjbw1bigthird, adjbw2bigthird);
    779779                        drawLineForBoxSide(graphicsContext, x2 - third, y1 + max((-adjbw1 * 2 + 1) / 3, 0),
    … …  
    17391739IntRect RenderObject::localCaretRect(InlineBox*, int, int* extraWidthToEndOfLine)
    17401740{
    1741    if (extraWidthToEndOfLine)
    1742        *extraWidthToEndOfLine = 0;
     1741    if (extraWidthToEndOfLine)
     1742        *extraWidthToEndOfLine = 0;
    17431743
    17441744    return IntRect();

  • trunk/WebCore/rendering/RenderSVGImage.cpp

    r45550 r45624  
    6060        if (origDestHeight > (origDestWidth * widthToHeightMultiplier)) {
    6161            destRect.setHeight(origDestWidth * widthToHeightMultiplier);
    62             switch(aspectRatio->align()) {
     62            switch (aspectRatio->align()) {
    6363                case SVGPreserveAspectRatio::SVG_PRESERVEASPECTRATIO_XMINYMID:
    6464                case SVGPreserveAspectRatio::SVG_PRESERVEASPECTRATIO_XMIDYMID:
    … …  
    7575        if (origDestWidth > (origDestHeight / widthToHeightMultiplier)) {
    7676            destRect.setWidth(origDestHeight / widthToHeightMultiplier);
    77             switch(aspectRatio->align()) {
     77            switch (aspectRatio->align()) {
    7878                case SVGPreserveAspectRatio::SVG_PRESERVEASPECTRATIO_XMIDYMIN:
    7979                case SVGPreserveAspectRatio::SVG_PRESERVEASPECTRATIO_XMIDYMID:
    … …  
    9494            float destToSrcMultiplier = srcRect.width() / destRect.width();
    9595            srcRect.setHeight(destRect.height() * destToSrcMultiplier);
    96             switch(aspectRatio->align()) {
     96            switch (aspectRatio->align()) {
    9797                case SVGPreserveAspectRatio::SVG_PRESERVEASPECTRATIO_XMINYMID:
    9898                case SVGPreserveAspectRatio::SVG_PRESERVEASPECTRATIO_XMIDYMID:
    … …  
    111111            float destToSrcMultiplier = srcRect.height() / destRect.height();
    112112            srcRect.setWidth(destRect.width() * destToSrcMultiplier);
    113             switch(aspectRatio->align()) {
     113            switch (aspectRatio->align()) {
    114114                case SVGPreserveAspectRatio::SVG_PRESERVEASPECTRATIO_XMIDYMIN:
    115115                case SVGPreserveAspectRatio::SVG_PRESERVEASPECTRATIO_XMIDYMID:

  • trunk/WebCore/rendering/RenderSlider.h

    r41972 r45624  
    4242        virtual bool isSlider() const { return true; }
    4343
    44         virtual int baselinePosition( bool, bool ) const;
     44        virtual int baselinePosition(bool, bool) const;
    4545        virtual void calcPrefWidths();
    4646        virtual void layout();

  • trunk/WebCore/rendering/RenderTable.cpp

    r45317 r45624  
    883883    else {
    884884        RenderObject* child;
    885         for (child = lastChild(); child && !child->isTableSection(); child = child->previousSibling())
    886             ;
     885        for (child = lastChild(); child && !child->isTableSection(); child = child->previousSibling()) { }
    887886        bottomSection = child ? static_cast<RenderTableSection*>(child) : 0;
    888887    }

  • trunk/WebCore/rendering/RenderTableCol.h

    r45015 r45624  
    3434namespace WebCore {
    3535
    36 class RenderTableCol : public RenderBox
    37 {
     36class RenderTableCol : public RenderBox {
    3837public:
    3938    RenderTableCol(Node*);

  • trunk/WebCore/rendering/RenderTextControlSingleLine.cpp

    r44806 r45624  
    745745bool RenderTextControlSingleLine::itemIsSeparator(unsigned listIndex) const
    746746{
    747    // The separator will be the second to last item in our list.
    748    return static_cast<int>(listIndex) == (listSize() - 2);
     747    // The separator will be the second to last item in our list.
     748    return static_cast<int>(listIndex) == (listSize() - 2);
    749749}
    750750

  • trunk/WebCore/rendering/RenderThemeChromiumSkia.cpp

    r45569 r45624  
    134134bool RenderThemeChromiumSkia::supportsHover(const RenderStyle* style) const
    135135{
    136   return true;
     136    return true;
    137137}
    138138

  • trunk/WebCore/rendering/RenderThemeChromiumWin.cpp

    r45215 r45624  
    205205bool RenderThemeChromiumWin::supportsFocusRing(const RenderStyle* style) const
    206206{
    207    // Let webkit draw one of its halo rings around any focused element,
    208    // except push buttons. For buttons we use the windows PBS_DEFAULTED
    209    // styling to give it a blue border.
    210    return style->appearance() == ButtonPart
    211        || style->appearance() == PushButtonPart;
     207    // Let webkit draw one of its halo rings around any focused element,
     208    // except push buttons. For buttons we use the windows PBS_DEFAULTED
     209    // styling to give it a blue border.
     210    return style->appearance() == ButtonPart
     211            || style->appearance() == PushButtonPart;
    212212}
    213213

  • trunk/WebCore/rendering/SVGCharacterLayoutInfo.cpp

    r40842 r45624  
    391391        dyStack.append(list);
    392392        break;
    393    case AngleStack:
     393    case AngleStack:
    394394        angleStackChanged = true;
    395395        angleStack.append(list);
    396396        break;
    397    default:
     397    default:
    398398        ASSERT_NOT_REACHED();
    399399    }

  • trunk/WebCore/rendering/SVGCharacterLayoutInfo.h

    r39635 r45624  
    4545
    4646template<class Type>
    47 class PositionedVector : public Vector<Type>
    48 {
     47class PositionedVector : public Vector<Type> {
    4948public:
    5049    PositionedVector<Type>()

  • trunk/WebCore/rendering/TextControlInnerElements.h

    r37637 r45624  
    3333class String;
    3434
    35 class TextControlInnerElement : public HTMLDivElement
    36 {
     35class TextControlInnerElement : public HTMLDivElement {
    3736public:
    3837    TextControlInnerElement(Document*, Node* shadowParent = 0);

  • trunk/WebCore/rendering/bidi.cpp

    r45608 r45624  
    642642    // justifying text.
    643643    int x = leftOffset(height(), firstLine);
    644     switch(textAlign) {
     644    switch (textAlign) {
    645645        case LEFT:
    646646        case WEBKIT_LEFT:
Note: See TracChangeset for help on using the changeset viewer.