[Webkit-unassigned] [Bug 92252] New: Grid Demo spends 1.5% of total time allocating Path objects in RenderBoxModelObject::paintBorderSides

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=92252

           Summary: Grid Demo spends 1.5% of total time allocating Path
                    objects in RenderBoxModelObject::paintBorderSides
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Layout and Rendering
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: eric at webkit.org
                CC: bdakin at apple.com, fishd at chromium.org,
                    jchaffraix at webkit.org, simon.fraser at apple.com


Grid Demo spends 1.5% of total time allocating Path objects in RenderBoxModelObject::paintBorderSides

The problem is that Path::Path() actually does a malloc() (at least on Mac), because it holds a reference to a CGPath.

void RenderBoxModelObject::paintBorderSides(...)
{
    bool renderRadii = outerBorder.isRounded();

    Path roundedPath;
    if (renderRadii)
        roundedPath.addRoundedRect(outerBorder);

always performs this malloc, even if roundedPath is not needed.

I suspect any painting benchmark which includes borders is hitting this.

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