[Webkit-unassigned] [Bug 112125] New: 1.3% mac-release-10.6-webkit-latest/intl2/times/t change after rev 145300

[bugzilla-daemon at webkit.org]

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

           Summary: 1.3% mac-release-10.6-webkit-latest/intl2/times/t
                    change after rev 145300
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P1
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: nbarth at chromium.org
                CC: robert at webkit.org


There looks like there's a performance regression caused by r145305 fixing
Bug 108357 REGRESSION(r140907): Incorrect baseline for cells with media content during load

Concretely:
1.3% change in mac-release-10.6-webkit-latest/intl2/times/t after revision 187133 (WebKit revision 145300)
Graph:
http://chromium-perf.appspot.com/?details=true&tab=mac-release-10.6-webkit-latest&testSuite=intl2&rev=187433&graph=times&trace=t&history=150&master=ChromiumWebkit

(Pretty clear regression, narrow range, this revision changes layout, hence likely cause.)

I understand that this revision fixes a layout bug and is thus necessary in some form, but it looks like it has performance impact, perhaps due to additional computation and layout being necessary in some cases.

Could you take a look at it and see that any regressions are necessary and expected?
(Perhaps there's another way of doing it with less impact.)

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