Changeset 86197 in webkit


Ignore:
Timestamp:
May 10, 2011 5:10:26 PM (13 years ago)
Author:
commit-queue@webkit.org
Message:

2011-05-10 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r86195.
​http://trac.webkit.org/changeset/86195
​https://bugs.webkit.org/show_bug.cgi?id=60597

LayerOffset is a really bad name (Requested by smfr on
#webkit).

  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • page/FrameView.cpp: (WebCore::FrameView::paintScrollCorner):
  • platform/graphics/LayerOffset.h: Removed.
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paint): (WebCore::InlineTextBox::paintCustomHighlight):
  • rendering/InlineTextBox.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paintCustomHighlight):
  • rendering/RenderBox.h:
  • rendering/RenderImage.cpp: (WebCore::RenderImage::paintReplaced):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintScrollCorner): (WebCore::RenderLayer::paintResizer):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint):
  • rendering/RenderObject.h:
  • rendering/RenderScrollbar.cpp: (WebCore::RenderScrollbar::paintPart):
  • rendering/RenderScrollbarPart.cpp: (WebCore::RenderScrollbarPart::paintIntoRect):
  • rendering/RenderScrollbarPart.h:
  • rendering/RenderTable.cpp: (WebCore::RenderTable::paintObject): (WebCore::RenderTable::paintMask):
  • rendering/RenderTable.h:
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::paint):
  • rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::paintCustomHighlight): (WebCore::RootInlineBox::paint):
  • rendering/RootInlineBox.h:
Location:
trunk/Source/WebCore
Files:
1 deleted
23 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r86196 r86197  
     12011-05-10  Sheriff Bot  <webkit.review.bot@gmail.com>
     2
     3        Unreviewed, rolling out r86195.
     4        http://trac.webkit.org/changeset/86195
     5        https://bugs.webkit.org/show_bug.cgi?id=60597
     6
     7        LayerOffset is a really bad name (Requested by smfr on
     8        #webkit).
     9
     10        * GNUmakefile.list.am:
     11        * WebCore.gypi:
     12        * WebCore.pro:
     13        * WebCore.vcproj/WebCore.vcproj:
     14        * WebCore.xcodeproj/project.pbxproj:
     15        * page/FrameView.cpp:
     16        (WebCore::FrameView::paintScrollCorner):
     17        * platform/graphics/LayerOffset.h: Removed.
     18        * rendering/InlineTextBox.cpp:
     19        (WebCore::InlineTextBox::paint):
     20        (WebCore::InlineTextBox::paintCustomHighlight):
     21        * rendering/InlineTextBox.h:
     22        * rendering/RenderBox.cpp:
     23        (WebCore::RenderBox::paintCustomHighlight):
     24        * rendering/RenderBox.h:
     25        * rendering/RenderImage.cpp:
     26        (WebCore::RenderImage::paintReplaced):
     27        * rendering/RenderLayer.cpp:
     28        (WebCore::RenderLayer::paintScrollCorner):
     29        (WebCore::RenderLayer::paintResizer):
     30        * rendering/RenderListMarker.cpp:
     31        (WebCore::RenderListMarker::paint):
     32        * rendering/RenderObject.h:
     33        * rendering/RenderScrollbar.cpp:
     34        (WebCore::RenderScrollbar::paintPart):
     35        * rendering/RenderScrollbarPart.cpp:
     36        (WebCore::RenderScrollbarPart::paintIntoRect):
     37        * rendering/RenderScrollbarPart.h:
     38        * rendering/RenderTable.cpp:
     39        (WebCore::RenderTable::paintObject):
     40        (WebCore::RenderTable::paintMask):
     41        * rendering/RenderTable.h:
     42        * rendering/RenderWidget.cpp:
     43        (WebCore::RenderWidget::paint):
     44        * rendering/RootInlineBox.cpp:
     45        (WebCore::RootInlineBox::paintCustomHighlight):
     46        (WebCore::RootInlineBox::paint):
     47        * rendering/RootInlineBox.h:
     48
    1492011-05-10  Emil A Eklund  <eae@chromium.org>
    250

  • trunk/Source/WebCore/GNUmakefile.list.am

    r86195 r86197  
    24402440        Source/WebCore/platform/graphics/IntSize.h \
    24412441        Source/WebCore/platform/graphics/IntSizeHash.h \
    2442         Source/WebCore/platform/graphics/LayerOffset.h \
    24432442        Source/WebCore/platform/graphics/MediaPlayer.cpp \
    24442443        Source/WebCore/platform/graphics/MediaPlayer.h \

  • trunk/Source/WebCore/WebCore.gypi

    r86195 r86197  
    849849            'platform/graphics/IntSize.h',
    850850            'platform/graphics/IntSizeHash.h',
    851             'platform/graphics/LayerOffset.h',
    852851            'platform/graphics/MediaPlayer.h',
    853852            'platform/graphics/Path.h',

  • trunk/Source/WebCore/WebCore.pro

    r86195 r86197  
    19521952    platform/graphics/IntPointHash.h \
    19531953    platform/graphics/IntRect.h \
    1954     platform/graphics/LayerOffset.h \
    19551954    platform/graphics/MediaPlayer.h \
    19561955    platform/graphics/Path.h \

  • trunk/Source/WebCore/WebCore.vcproj/WebCore.vcproj

    r86195 r86197  
    2691826918                                </File>
    2691926919                                <File
    26920                                         RelativePath="..\platform\graphics\LayerOffset.h"
    26921                                         >
    26922                                 </File>
    26923                                 <File
    2692426920                                        RelativePath="..\platform\graphics\MediaPlayer.cpp"
    2692526921                                        >

  • trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj

    r86195 r86197  
    47994799                B8DBDB4D130B0F8A00F5CDB1 /* SpellingCorrectionCommand.cpp in Sources */ = {isa = PBXBuildFile; fileRef = B8DBDB49130B0F8A00F5CDB1 /* SpellingCorrectionCommand.cpp */; };
    48004800                B8DBDB4E130B0F8A00F5CDB1 /* SpellingCorrectionCommand.h in Headers */ = {isa = PBXBuildFile; fileRef = B8DBDB4A130B0F8A00F5CDB1 /* SpellingCorrectionCommand.h */; };
    4801                 BA5175C813789735008BFCBF /* LayerOffset.h in Headers */ = {isa = PBXBuildFile; fileRef = BA5175C713789735008BFCBF /* LayerOffset.h */; settings = {ATTRIBUTES = (Private, ); }; };
    48024801                BC00F0040E0A185500FD04E3 /* DOMFile.h in Headers */ = {isa = PBXBuildFile; fileRef = BC00EFFE0E0A185500FD04E3 /* DOMFile.h */; };
    48034802                BC00F0050E0A185500FD04E3 /* DOMFile.mm in Sources */ = {isa = PBXBuildFile; fileRef = BC00EFFF0E0A185500FD04E3 /* DOMFile.mm */; };
    … …  
    1120911208                B8DBDB49130B0F8A00F5CDB1 /* SpellingCorrectionCommand.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = SpellingCorrectionCommand.cpp; sourceTree = "<group>"; };
    1121011209                B8DBDB4A130B0F8A00F5CDB1 /* SpellingCorrectionCommand.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = SpellingCorrectionCommand.h; sourceTree = "<group>"; };
    11211                 BA5175C713789735008BFCBF /* LayerOffset.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = LayerOffset.h; sourceTree = "<group>"; };
    1121211210                BC00EFFE0E0A185500FD04E3 /* DOMFile.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = DOMFile.h; sourceTree = "<group>"; };
    1121311211                BC00EFFF0E0A185500FD04E3 /* DOMFile.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = DOMFile.mm; sourceTree = "<group>"; };
    … …  
    1768517683                                B27535470B053814002CE64F /* IntSize.h */,
    1768617684                                B27535480B053814002CE64F /* IntSizeHash.h */,
    17687                                 BA5175C713789735008BFCBF /* LayerOffset.h */,
    1768817685                                E4B41E0C0CBF90BD00AF2ECE /* MediaPlayer.cpp */,
    1768917686                                E4B41E0D0CBF90BD00AF2ECE /* MediaPlayer.h */,
    … …  
    2275222749                                37AFFDF71370A0B800E895C0 /* SharedBufferCRLFLineReader.h in Headers */,
    2275322750                                379E371713736A6600B9E919 /* QuotedPrintable.h in Headers */,
    22754                                 BA5175C813789735008BFCBF /* LayerOffset.h in Headers */,
    2275522751                                FD1660A513787C6D001FFA7B /* DenormalDisabler.h in Headers */,
    2275622752                        );

  • trunk/Source/WebCore/page/FrameView.cpp

    r86195 r86197  
    22622262
    22632263    if (m_scrollCorner) {
    2264         m_scrollCorner->paintIntoRect(context, cornerRect.location(), cornerRect);
     2264        m_scrollCorner->paintIntoRect(context, cornerRect.x(), cornerRect.y(), cornerRect);
    22652265        return;
    22662266    }

  • trunk/Source/WebCore/rendering/InlineTextBox.cpp

    r86195 r86197  
    547547        // Custom highlighters go behind everything else.
    548548        if (styleToUse->highlight() != nullAtom && !context->paintingDisabled())
    549             paintCustomHighlight(LayerOffset(tx, ty), styleToUse->highlight());
     549            paintCustomHighlight(tx, ty, styleToUse->highlight());
    550550#endif
    551551
    … …  
    858858#if PLATFORM(MAC)
    859859
    860 void InlineTextBox::paintCustomHighlight(LayerOffset layerOffset, const AtomicString& type)
     860void InlineTextBox::paintCustomHighlight(int tx, int ty, const AtomicString& type)
    861861{
    862862    Frame* frame = renderer()->frame();
    … …  
    868868
    869869    RootInlineBox* r = root();
    870     FloatRect rootRect(layerOffset.x() + r->x(), layerOffset.y() + selectionTop(), r->logicalWidth(), selectionHeight());
    871     FloatRect textRect(layerOffset.x() + x(), rootRect.y(), logicalWidth(), rootRect.height());
     870    FloatRect rootRect(tx + r->x(), ty + selectionTop(), r->logicalWidth(), selectionHeight());
     871    FloatRect textRect(tx + x(), rootRect.y(), logicalWidth(), rootRect.height());
    872872
    873873    page->chrome()->client()->paintCustomHighlight(renderer()->node(), type, textRect, rootRect, true, false);

  • trunk/Source/WebCore/rendering/InlineTextBox.h

    r86195 r86197  
    166166    void paintCompositionUnderline(GraphicsContext*, const FloatPoint& boxOrigin, const CompositionUnderline&);
    167167#if PLATFORM(MAC)
    168     void paintCustomHighlight(LayerOffset, const AtomicString& type);
     168    void paintCustomHighlight(int tx, int ty, const AtomicString& type);
    169169#endif
    170170

  • trunk/Source/WebCore/rendering/RenderBox.cpp

    r86195 r86197  
    10781078#if PLATFORM(MAC)
    10791079
    1080 void RenderBox::paintCustomHighlight(LayerOffset layoutOffset, const AtomicString& type, bool behindText)
     1080void RenderBox::paintCustomHighlight(int tx, int ty, const AtomicString& type, bool behindText)
    10811081{
    10821082    Frame* frame = this->frame();
    … …  
    10901090    RootInlineBox* r = boxWrap ? boxWrap->root() : 0;
    10911091    if (r) {
    1092         FloatRect rootRect(layoutOffset.x() + r->x(), layoutOffset.y() + r->selectionTop(), r->logicalWidth(), r->selectionHeight());
    1093         FloatRect imageRect(location() + layoutOffset, IntSize(width(), rootRect.height()));
     1092        FloatRect rootRect(tx + r->x(), ty + r->selectionTop(), r->logicalWidth(), r->selectionHeight());
     1093        FloatRect imageRect(tx + x(), rootRect.y(), width(), rootRect.height());
    10941094        page->chrome()->client()->paintCustomHighlight(node(), type, imageRect, rootRect, behindText, false);
    10951095    } else {
    1096         FloatRect imageRect(location() + layoutOffset, IntSize(width(), height()));
     1096        FloatRect imageRect(tx + x(), ty + y(), width(), height());
    10971097        page->chrome()->client()->paintCustomHighlight(node(), type, imageRect, imageRect, behindText, false);
    10981098    }

  • trunk/Source/WebCore/rendering/RenderBox.h

    r86195 r86197  
    413413
    414414#if PLATFORM(MAC)
    415     void paintCustomHighlight(LayerOffset, const AtomicString& type, bool behindText);
     415    void paintCustomHighlight(int tx, int ty, const AtomicString& type, bool behindText);
    416416#endif
    417417

  • trunk/Source/WebCore/rendering/RenderImage.cpp

    r86195 r86197  
    303303#if PLATFORM(MAC)
    304304        if (style()->highlight() != nullAtom && !paintInfo.context->paintingDisabled())
    305             paintCustomHighlight(LayerOffset(tx - x(), ty - y()), style()->highlight(), true);
     305            paintCustomHighlight(tx - x(), ty - y(), style()->highlight(), true);
    306306#endif
    307307

  • trunk/Source/WebCore/rendering/RenderLayer.cpp

    r86196 r86197  
    22972297
    22982298    if (m_scrollCorner) {
    2299         m_scrollCorner->paintIntoRect(context, LayerOffset(tx, ty), absRect);
     2299        m_scrollCorner->paintIntoRect(context, tx, ty, absRect);
    23002300        return;
    23012301    }
    … …  
    23262326   
    23272327    if (m_resizer) {
    2328         m_resizer->paintIntoRect(context, LayerOffset(tx, ty), absRect);
     2328        m_resizer->paintIntoRect(context, tx, ty, absRect);
    23292329        return;
    23302330    }

  • trunk/Source/WebCore/rendering/RenderListMarker.cpp

    r86195 r86197  
    11291129#if PLATFORM(MAC)
    11301130        if (style()->highlight() != nullAtom && !paintInfo.context->paintingDisabled())
    1131             paintCustomHighlight(LayerOffset(tx, ty), style()->highlight(), true);
     1131            paintCustomHighlight(tx, ty, style()->highlight(), true);
    11321132#endif
    11331133        context->drawImage(m_image->image(this, marker.size()).get(), style()->colorSpace(), marker);
    … …  
    11431143    // FIXME: paint gap between marker and list item proper
    11441144    if (style()->highlight() != nullAtom && !paintInfo.context->paintingDisabled())
    1145         paintCustomHighlight(LayerOffset(tx, ty), style()->highlight(), true);
     1145        paintCustomHighlight(tx, ty, style()->highlight(), true);
    11461146#endif
    11471147

  • trunk/Source/WebCore/rendering/RenderObject.h

    r86195 r86197  
    3131#include "Element.h"
    3232#include "FloatQuad.h"
    33 #include "LayerOffset.h"
    3433#include "PaintPhase.h"
    3534#include "RenderObjectChildList.h"

  • trunk/Source/WebCore/rendering/RenderScrollbar.cpp

    r86195 r86197  
    276276    if (!partRenderer)
    277277        return;
    278     partRenderer->paintIntoRect(graphicsContext, pos(), rect);
     278    partRenderer->paintIntoRect(graphicsContext, x(), y(), rect);
    279279}
    280280

  • trunk/Source/WebCore/rendering/RenderScrollbarPart.cpp

    r86195 r86197  
    160160}
    161161
    162 void RenderScrollbarPart::paintIntoRect(GraphicsContext* graphicsContext, LayerOffset layerOffset, const IntRect& rect)
     162void RenderScrollbarPart::paintIntoRect(GraphicsContext* graphicsContext, int tx, int ty, const IntRect& rect)
    163163{
    164164    // Make sure our dimensions match the rect.
    165     setLocation(rect.location() - layerOffset.toSize());
     165    setLocation(rect.x() - tx, rect.y() - ty);
    166166    setWidth(rect.width());
    167167    setHeight(rect.height());
    … …  
    172172    // Now do the paint.
    173173    PaintInfo paintInfo(graphicsContext, rect, PaintPhaseBlockBackground, false, 0, 0);
    174     paint(paintInfo, layerOffset.x(), layerOffset.y());
     174    paint(paintInfo, tx, ty);
    175175    paintInfo.phase = PaintPhaseChildBlockBackgrounds;
    176     paint(paintInfo, layerOffset.x(), layerOffset.y());
     176    paint(paintInfo, tx, ty);
    177177    paintInfo.phase = PaintPhaseFloat;
    178     paint(paintInfo, layerOffset.x(), layerOffset.y());
     178    paint(paintInfo, tx, ty);
    179179    paintInfo.phase = PaintPhaseForeground;
    180     paint(paintInfo, layerOffset.x(), layerOffset.y());
     180    paint(paintInfo, tx, ty);
    181181    paintInfo.phase = PaintPhaseOutline;
    182     paint(paintInfo, layerOffset.x(), layerOffset.y());
     182    paint(paintInfo, tx, ty);
    183183}
    184184

  • trunk/Source/WebCore/rendering/RenderScrollbarPart.h

    r86195 r86197  
    2727#define RenderScrollbarPart_h
    2828
    29 #include "LayerOffset.h"
    3029#include "RenderBlock.h"
    3130#include "ScrollTypes.h"
    … …  
    4746    virtual void computePreferredLogicalWidths();
    4847   
    49     void paintIntoRect(GraphicsContext*, LayerOffset, const IntRect&);
     48    void paintIntoRect(GraphicsContext*, int tx, int ty, const IntRect&);
    5049   
    5150protected:

  • trunk/Source/WebCore/rendering/RenderTable.cpp

    r86195 r86197  
    481481
    482482    if (paintPhase == PaintPhaseMask) {
    483         paintMask(paintInfo, LayerOffset(tx, ty));
     483        paintMask(paintInfo, tx, ty);
    484484        return;
    485485    }
    … …  
    574574}
    575575
    576 void RenderTable::paintMask(PaintInfo& paintInfo, LayerOffset layerOffset)
     576void RenderTable::paintMask(PaintInfo& paintInfo, int tx, int ty)
    577577{
    578578    if (style()->visibility() != VISIBLE || paintInfo.phase != PaintPhaseMask)
    579579        return;
    580580
    581     IntRect rect(layerOffset.toPoint(), IntSize(width(), height()));
     581    IntRect rect(tx, ty, width(), height());
    582582    subtractCaptionRect(rect);
    583583

  • trunk/Source/WebCore/rendering/RenderTable.h

    r86195 r86197  
    218218    virtual void paintObject(PaintInfo&, int tx, int ty);
    219219    virtual void paintBoxDecorations(PaintInfo&, int tx, int ty);
    220     virtual void paintMask(PaintInfo&, LayerOffset);
     220    virtual void paintMask(PaintInfo&, int tx, int ty);
    221221    virtual void layout();
    222222    virtual void computePreferredLogicalWidths();

  • trunk/Source/WebCore/rendering/RenderWidget.cpp

    r86195 r86197  
    268268#if PLATFORM(MAC)
    269269    if (style()->highlight() != nullAtom && !paintInfo.context->paintingDisabled())
    270         paintCustomHighlight(LayerOffset(tx - x(), ty - y()), style()->highlight(), true);
     270        paintCustomHighlight(tx - x(), ty - y(), style()->highlight(), true);
    271271#endif
    272272

  • trunk/Source/WebCore/rendering/RootInlineBox.cpp

    r86195 r86197  
    158158}
    159159
    160 void RootInlineBox::paintCustomHighlight(PaintInfo& paintInfo, LayerOffset layerOffset, const AtomicString& highlightType)
     160void RootInlineBox::paintCustomHighlight(PaintInfo& paintInfo, int tx, int ty, const AtomicString& highlightType)
    161161{
    162162    if (!paintInfo.shouldPaintWithinRoot(renderer()) || renderer()->style()->visibility() != VISIBLE || paintInfo.phase != PaintPhaseForeground)
    … …  
    171171
    172172    // Get the inflated rect so that we can properly hit test.
    173     FloatRect rootRect(layerOffset.x() + x(), layerOffset.y() + selectionTop(), logicalWidth(), selectionHeight());
     173    FloatRect rootRect(tx + x(), ty + selectionTop(), logicalWidth(), selectionHeight());
    174174    FloatRect inflatedRect = page->chrome()->client()->customHighlightRect(renderer()->node(), highlightType, rootRect);
    175175    if (inflatedRect.intersects(paintInfo.rect))
    … …  
    186186    RenderStyle* styleToUse = renderer()->style(m_firstLine);
    187187    if (styleToUse->highlight() != nullAtom && !paintInfo.context->paintingDisabled())
    188         paintCustomHighlight(paintInfo, LayerOffset(tx, ty), styleToUse->highlight());
     188        paintCustomHighlight(paintInfo, tx, ty, styleToUse->highlight());
    189189#endif
    190190}

  • trunk/Source/WebCore/rendering/RootInlineBox.h

    r86195 r86197  
    9393#if PLATFORM(MAC)
    9494    void addHighlightOverflow();
    95     void paintCustomHighlight(PaintInfo&, LayerOffset, const AtomicString& highlightType);
     95    void paintCustomHighlight(PaintInfo&, int tx, int ty, const AtomicString& highlightType);
    9696#endif
    9797
Note: See TracChangeset for help on using the changeset viewer.