[Webkit-unassigned] [Bug 26949] WebCore part of the Haiku WebKit port

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 15 11:52:55 PDT 2009


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
  Attachment #32228|review?                     |review-
               Flag|                            |

--- Comment #19 from Eric Seidel <eric at webkit.org>  2009-07-15 11:52:55 PDT ---
(From update of attachment 32228)
Who owns the BView in the GraphicsContextPlatformPrivate?  I'm surprised that
GCPlatformPrivate doesnt' need to retain it?

npoints should be numberOfPoints or pointsLength or similar.

8     if (rects.size() > 1)
 229     {

 235         m_data->view->StrokeRect(region.Frame(),B_MIXED_COLORS);

 257 FloatRect GraphicsContext::roundToDevicePixels(const FloatRect& frect)

 287 void GraphicsContext::strokeRect(const FloatRect& r, float width)
 298 void GraphicsContext::setLineCap(LineCap lc)

No need to wrap lines like this:
void GraphicsContext::addInnerRoundedRectClip(const IntRect& rect,
 451                                               int thickness)

I'm surprised your compiler can't convert from "return 0":
    return PassRefPtr<Icon>(0);

Why are you adding the empy ImageBufferData class?

I think you mean something like "if the image hasn't fully loaded":
 130     if (!image) // If it's too early we won't have an image yet.

But I'm still confused as to why we'd be calling pattern drawing routines with
an image w/o a first frame.

I think there are functions to do this for you:
 140     ctxt->clip(IntRect(dstRect.x(), dstRect.y(), dstRect.width(),

IntRect(dstRect) probably does what you want.  Although you might want
enclosingIntRect() there.

c++ style casts please:
 57     const unsigned char* uContents = (const unsigned char*) data.data();

I'm surprised:
 50 ImageDecoder* createDecoder(const Vector<char>& data)

isn't shared between teh ports?

 45     return BRect(BPoint(x(), y()), BSize(width(),height()));

It's better to break patches down by size instead of directory.

r- for all the nits above.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list