<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>76725</bug_id>
          
          <creation_ts>2012-01-20 12:08:27 -0800</creation_ts>
          <short_desc>[Regression] 2% Page Cycler Moz regression</short_desc>
          <delta_ts>2012-01-22 11:17:18 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>76726</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Barth">abarth</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>hyatt</cc>
    
    <cc>kenrb</cc>
    
    <cc>mihnea</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>thorton</cc>
    
    <cc>tony</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>539679</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-01-20 12:08:27 -0800</bug_when>
    <thetext>Looks to be something in this range: http://trac.webkit.org/log/trunk/Source?rev=105427&amp;stop_rev=105419&amp;verbose=on

http://build.chromium.org/f/chromium/perf/xp-release-dual-core/moz/report.html?history=150&amp;rev=118458&amp;graph=times</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>539680</commentid>
    <comment_count>1</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-01-20 12:08:48 -0800</bug_when>
    <thetext>http://code.google.com/p/chromium/issues/detail?id=110915</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>539685</commentid>
    <comment_count>2</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-01-20 12:12:07 -0800</bug_when>
    <thetext>These two look like the leading candidate for causing the regression:

http://trac.webkit.org/changeset/105426/trunk/Source
http://trac.webkit.org/changeset/105423/trunk/Source</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>539688</commentid>
    <comment_count>3</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-01-20 12:13:13 -0800</bug_when>
    <thetext>I think we should try rolling out http://trac.webkit.org/changeset/105426 because the ChangeLog mentions there&apos;s a risk of a perf regression.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>539697</commentid>
    <comment_count>4</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-01-20 12:23:27 -0800</bug_when>
    <thetext>Tried reverting 105426 in http://trac.webkit.org/changeset/105532</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>539918</commentid>
    <comment_count>5</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-01-20 16:38:44 -0800</bug_when>
    <thetext>Reverting that patch seems to have fixed the regression.

http://build.chromium.org/f/chromium/perf/vista-release-webkit-latest/moz/report.html?history=150&amp;rev=-1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>540225</commentid>
    <comment_count>6</comment_count>
    <who name="Mihnea Ovidenie">mihnea</who>
    <bug_when>2012-01-22 00:03:15 -0800</bug_when>
    <thetext>This is huge regression. In my tests, i have never had a regression of this scale. The initial regression introduced by the first version of the region styling patch was 2-3%. I am puzzled but i will try to take a look again at the problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>540260</commentid>
    <comment_count>7</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2012-01-22 09:14:29 -0800</bug_when>
    <thetext>I think abarth just read the graph wrong.  It&apos;s about 2%, like before.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>540294</commentid>
    <comment_count>8</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-01-22 11:17:18 -0800</bug_when>
    <thetext>Sorry, I was just repeating what the performance sheriff wrote.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>