67876 2011-09-09 16:53:28 -0700 [Chromium Win] 10% PLT regression r94589 2013-04-08 14:08:39 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) All All RESOLVED WONTFIX http://build.chromium.org/f/chromium/perf/xp-release-dual-core/intl1/report.html?history=150&rev=100517 P1 Normal --- 69237 1 abarth reed fishd ggaren japhet jar jshin kbr loislo reed sam schenney slewis oldest_to_newest 465108 0 abarth 2011-09-09 16:53:28 -0700 Something in this range: http://trac.webkit.org/log/trunk?rev=94653&stop_rev=94560&verbose=on Appears to have caused a 10% PLT regression: http://build.chromium.org/f/chromium/perf/xp-release-dual-core/intl1/report.html?history=150&rev=100517 Notice the first spike when those revs were rolled into Chromium in Chromium@99936 and then the sustained rise after they were rolling in again in Chromium@100029. 465125 1 abarth 2011-09-09 17:11:57 -0700 I don't know whether this manifests on ports besides Chromium or on other test suites besides intl1. 465128 2 slewis 2011-09-09 17:13:47 -0700 i'll check our tests on Monday. 465615 3 loislo 2011-09-12 08:22:46 -0700 (In reply to comment #2) > i'll check our tests on Monday. I've made some bisecting steps. 94619 - bad 94588 - good 465645 4 loislo 2011-09-12 09:55:46 -0700 (In reply to comment #3) > (In reply to comment #2) > > i'll check our tests on Monday. > > I've made some bisecting steps. > > 94619 - bad > 94588 - good it is 94589 Author: reed@google.com <reed@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc> Date: Tue Sep 6 19:21:52 2011 +0000 [skia] never draw with GDI, so that all text can be gpu-accelerated https://bugs.webkit.org/show_bug.cgi?id=65203 Reviewed by Kenneth Russell. Source/WebCore: * platform/graphics/chromium/FontChromiumWin.cpp: (WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::TransparencyAwareFontPainter): (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::TransparencyAwareGlyphPainter): (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::drawGlyphs): (WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::TransparencyAwareUniscribePainter): (WebCore::drawGlyphsWin): (WebCore::Font::drawComplexText): * platform/graphics/chromium/UniscribeHelper.cpp: (WebCore::UniscribeHelper::draw): * platform/graphics/skia/PlatformContextSkia.cpp: * platform/graphics/skia/PlatformContextSkia.h: * platform/graphics/skia/SkiaFontWin.cpp: (WebCore::skiaDrawText): * platform/graphics/skia/SkiaFontWin.h: LayoutTests: * platform/chromium/test_expectations.txt: 465675 5 abarth 2011-09-12 10:49:16 -0700 Would it make sense to roll out that patch while we investigate? We don't want this perf regression to hide other problems. 465676 6 abarth 2011-09-12 10:50:23 -0700 (Just so others don't spend too long looking, this patch is after the M15 branch point.) 466542 7 jshin 2011-09-13 14:35:20 -0700 http://crbug.com/95748 474020 8 abarth 2011-09-27 10:00:09 -0700 What's the current status of this issue? What the patch rolled out? Was the performance regression fixed? 476785 9 abarth 2011-10-02 22:45:22 -0700 *** Bug 69239 has been marked as a duplicate of this bug. ***