<?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>115172</bug_id>
          
          <creation_ts>2013-04-25 05:49:09 -0700</creation_ts>
          <short_desc>REGRESSION (r147797): Animations slideshows of images on www.thesuperficial.com are slow</short_desc>
          <delta_ts>2013-04-25 06:25:13 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Page Loading</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>http://www.thesuperficial.com/photos/reese-witherspoon-at-a-screening-of-mud</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, Regression</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Antti Koivisto">koivisto</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>japhet</cc>
    
    <cc>kling</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>881209</commentid>
    <comment_count>0</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2013-04-25 05:49:09 -0700</bug_when>
    <thetext>On this page ad subframes loads on slideshow navigation switch us to throttled state.  We should only throttle layer flushes for the main load, not for any subsequent dynamically triggered subframe load.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881213</commentid>
    <comment_count>1</comment_count>
      <attachid>199655</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2013-04-25 05:58:18 -0700</bug_when>
    <thetext>Created attachment 199655
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881214</commentid>
    <comment_count>2</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2013-04-25 05:58:49 -0700</bug_when>
    <thetext>&lt;rdar://problem/13724574&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881215</commentid>
    <comment_count>3</comment_count>
      <attachid>199655</attachid>
    <who name="Andreas Kling">kling</who>
    <bug_when>2013-04-25 06:01:54 -0700</bug_when>
    <thetext>Comment on attachment 199655
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=199655&amp;action=review

r=me

&gt; Source/WebCore/loader/FrameLoader.cpp:3322
&gt; -    bool isLoadProgressing = m_frame-&gt;page()-&gt;progress()-&gt;isLoadProgressing();
&gt; +    bool isLoadProgressing = m_frame-&gt;page()-&gt;progress()-&gt;isMainLoadProgressing();

You may also want to rename the variable.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>881221</commentid>
    <comment_count>4</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2013-04-25 06:25:13 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/149105</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>199655</attachid>
            <date>2013-04-25 05:58:18 -0700</date>
            <delta_ts>2013-04-25 06:01:54 -0700</delta_ts>
            <desc>patch</desc>
            <filename>progress-tracker-frames-2.patch</filename>
            <type>text/plain</type>
            <size>4192</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE0OTEwMSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDI2IEBACisyMDEzLTA0LTI1ICBBbnR0aSBL
b2l2aXN0byAgPGFudHRpQGFwcGxlLmNvbT4KKworICAgICAgICBSRUdSRVNTSU9OIChyMTQ3Nzk3
KTogQW5pbWF0aW9ucyBzbGlkZXNob3dzIG9mIGltYWdlcyBvbiB3d3cudGhlc3VwZXJmaWNpYWwu
Y29tIGFyZSBzbG93CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD0xMTUxNzIKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAg
ICAgICBPbiB0aGlzIHBhZ2UgYWRzIGR5bmFtaWNhbGx5IGxvYWRlZCB0byBzdWJmcmFtZXMgb24g
c2xpZGVzaG93IG5hdmlnYXRpb24gc3dpdGNoIHVzIHRvIHN0YXRlIHdoZXJlIHdlIHRocm90dGxl
IGxheWVyIGZsdXNoZXMuCisgICAgICAgIAorICAgICAgICBGaXggYnkgaWdub3JpbmcgYW55IHN1
YmZyYW1lLW9yaWdpbmF0ZWQgbG9hZHMgd2hlbiBkZXRlcm1pbmluZyB0aHJvdHRsaW5nLgorCisg
ICAgICAgICogbG9hZGVyL0ZyYW1lTG9hZGVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkZyYW1l
TG9hZGVyOjpsb2FkUHJvZ3Jlc3NpbmdTdGF0dXNDaGFuZ2VkKToKKyAgICAgICAgKiBsb2FkZXIv
UHJvZ3Jlc3NUcmFja2VyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlByb2dyZXNzVHJhY2tlcjo6
aXNNYWluTG9hZFByb2dyZXNzaW5nKToKKyAgICAgICAgCisgICAgICAgICAgICBSZW5hbWUsIGln
bm9yZSBzdWJmcmFtZSBvcmlnaW5hdGVkIGxvYWRzLiBTdWJmcmFtZSBsb2FkcyB0aGF0IGFyZSBp
bml0aWF0ZWQgZHVyaW5nIHRoZSBtYWluIGxvYWQgc3RpbGwgY291bnQuCisKKyAgICAgICAgKiBs
b2FkZXIvUHJvZ3Jlc3NUcmFja2VyLmg6CisgICAgICAgIChQcm9ncmVzc1RyYWNrZXIpOgorICAg
ICAgICAqIHJlbmRlcmluZy9SZW5kZXJMYXllckJhY2tpbmcuY3BwOgorICAgICAgICAoV2ViQ29y
ZTo6Y29tcHV0ZVRpbGVDb3ZlcmFnZSk6CisKIDIwMTMtMDQtMjUgIEFuZHJlYXMgS2xpbmcgIDxh
a2xpbmdAYXBwbGUuY29tPgogCiAgICAgICAgIFJlbW92ZSBpc1BsdWdpbkVsZW1lbnQgaGFjayBp
biBEb2N1bWVudDo6c2V0Rm9jdXNlZE5vZGUoKQpJbmRleDogU291cmNlL1dlYkNvcmUvbG9hZGVy
L0ZyYW1lTG9hZGVyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9sb2FkZXIvRnJh
bWVMb2FkZXIuY3BwCShyZXZpc2lvbiAxNDg5NjkpCisrKyBTb3VyY2UvV2ViQ29yZS9sb2FkZXIv
RnJhbWVMb2FkZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0zMzE5LDcgKzMzMTksNyBAQCBOZXR3
b3JraW5nQ29udGV4dCogRnJhbWVMb2FkZXI6Om5ldHdvcmtpCiAKIHZvaWQgRnJhbWVMb2FkZXI6
OmxvYWRQcm9ncmVzc2luZ1N0YXR1c0NoYW5nZWQoKQogewotICAgIGJvb2wgaXNMb2FkUHJvZ3Jl
c3NpbmcgPSBtX2ZyYW1lLT5wYWdlKCktPnByb2dyZXNzKCktPmlzTG9hZFByb2dyZXNzaW5nKCk7
CisgICAgYm9vbCBpc0xvYWRQcm9ncmVzc2luZyA9IG1fZnJhbWUtPnBhZ2UoKS0+cHJvZ3Jlc3Mo
KS0+aXNNYWluTG9hZFByb2dyZXNzaW5nKCk7CiAgICAgRnJhbWVWaWV3KiB2aWV3ID0gbV9mcmFt
ZS0+cGFnZSgpLT5tYWluRnJhbWUoKS0+dmlldygpOwogICAgIHZpZXctPnVwZGF0ZUxheWVyRmx1
c2hUaHJvdHRsaW5nSW5BbGxGcmFtZXMoaXNMb2FkUHJvZ3Jlc3NpbmcpOwogICAgIHZpZXctPmFk
anVzdFRpbGVkQmFja2luZ0NvdmVyYWdlKCk7CkluZGV4OiBTb3VyY2UvV2ViQ29yZS9sb2FkZXIv
UHJvZ3Jlc3NUcmFja2VyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9sb2FkZXIv
UHJvZ3Jlc3NUcmFja2VyLmNwcAkocmV2aXNpb24gMTQ4OTY5KQorKysgU291cmNlL1dlYkNvcmUv
bG9hZGVyL1Byb2dyZXNzVHJhY2tlci5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTI4Myw4ICsyODMs
MTMgQEAgdW5zaWduZWQgbG9uZyBQcm9ncmVzc1RyYWNrZXI6OmNyZWF0ZVVuaQogICAgIHJldHVy
biArK3NfdW5pcXVlSWRlbnRpZmllcjsKIH0KIAotYm9vbCBQcm9ncmVzc1RyYWNrZXI6OmlzTG9h
ZFByb2dyZXNzaW5nKCkgY29uc3QKK2Jvb2wgUHJvZ3Jlc3NUcmFja2VyOjppc01haW5Mb2FkUHJv
Z3Jlc3NpbmcoKSBjb25zdAogeworICAgIGlmICghbV9vcmlnaW5hdGluZ1Byb2dyZXNzRnJhbWUp
CisgICAgICAgIHJldHVybiBmYWxzZTsKKyAgICAvLyBTZWUgaWYgdGhlIGxvYWQgb3JpZ2luYXRl
ZCBmcm9tIGEgc3ViZnJhbWUuCisgICAgaWYgKG1fb3JpZ2luYXRpbmdQcm9ncmVzc0ZyYW1lLT50
cmVlKCktPnBhcmVudCgpKQorICAgICAgICByZXR1cm4gZmFsc2U7CiAgICAgcmV0dXJuIG1fcHJv
Z3Jlc3NWYWx1ZSAmJiBtX3Byb2dyZXNzVmFsdWUgPCBmaW5hbFByb2dyZXNzVmFsdWUgJiYgbV9o
ZWFydGJlYXRzV2l0aE5vUHJvZ3Jlc3MgPCBsb2FkU3RhbGxlZEhlYXJ0YmVhdENvdW50OwogfQog
CkluZGV4OiBTb3VyY2UvV2ViQ29yZS9sb2FkZXIvUHJvZ3Jlc3NUcmFja2VyLmgKPT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PQotLS0gU291cmNlL1dlYkNvcmUvbG9hZGVyL1Byb2dyZXNzVHJhY2tlci5oCShyZXZpc2lvbiAx
NDg5NjkpCisrKyBTb3VyY2UvV2ViQ29yZS9sb2FkZXIvUHJvZ3Jlc3NUcmFja2VyLmgJKHdvcmtp
bmcgY29weSkKQEAgLTU5LDcgKzU5LDcgQEAgcHVibGljOgogICAgIGxvbmcgbG9uZyB0b3RhbFBh
Z2VBbmRSZXNvdXJjZUJ5dGVzVG9Mb2FkKCkgY29uc3QgeyByZXR1cm4gbV90b3RhbFBhZ2VBbmRS
ZXNvdXJjZUJ5dGVzVG9Mb2FkOyB9CiAgICAgbG9uZyBsb25nIHRvdGFsQnl0ZXNSZWNlaXZlZCgp
IGNvbnN0IHsgcmV0dXJuIG1fdG90YWxCeXRlc1JlY2VpdmVkOyB9CiAKLSAgICBib29sIGlzTG9h
ZFByb2dyZXNzaW5nKCkgY29uc3Q7CisgICAgYm9vbCBpc01haW5Mb2FkUHJvZ3Jlc3NpbmcoKSBj
b25zdDsKIAogcHJpdmF0ZToKICAgICBQcm9ncmVzc1RyYWNrZXIoKTsKSW5kZXg6IFNvdXJjZS9X
ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllckJhY2tpbmcuY3BwCj09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNv
dXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJMYXllckJhY2tpbmcuY3BwCShyZXZpc2lvbiAx
NDg5NjkpCisrKyBTb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyTGF5ZXJCYWNraW5nLmNw
cAkod29ya2luZyBjb3B5KQpAQCAtMjIyLDcgKzIyMiw3IEBAIHN0YXRpYyBUaWxlZEJhY2tpbmc6
OlRpbGVDb3ZlcmFnZSBjb21wdXQKICAgICBib29sIHVzZU1pbmltYWxUaWxlc0R1cmluZ0xvYWRp
bmcgPSBmYWxzZTsKICAgICAvLyBBdm9pZCBjaHVybi4KICAgICBpZiAoIWJhY2tpbmctPmRpZFN3
aXRjaFRvRnVsbFRpbGVDb3ZlcmFnZUR1cmluZ0xvYWRpbmcoKSkgewotICAgICAgICB1c2VNaW5p
bWFsVGlsZXNEdXJpbmdMb2FkaW5nID0gIWZyYW1lVmlldy0+aXNWaXN1YWxseU5vbkVtcHR5KCkg
fHwgKGZyYW1lLT5wYWdlKCktPnByb2dyZXNzKCktPmlzTG9hZFByb2dyZXNzaW5nKCkgJiYgIWZy
YW1lVmlldy0+d2FzU2Nyb2xsZWRCeVVzZXIoKSk7CisgICAgICAgIHVzZU1pbmltYWxUaWxlc0R1
cmluZ0xvYWRpbmcgPSAhZnJhbWVWaWV3LT5pc1Zpc3VhbGx5Tm9uRW1wdHkoKSB8fCAoZnJhbWUt
PnBhZ2UoKS0+cHJvZ3Jlc3MoKS0+aXNNYWluTG9hZFByb2dyZXNzaW5nKCkgJiYgIWZyYW1lVmll
dy0+d2FzU2Nyb2xsZWRCeVVzZXIoKSk7CiAgICAgICAgIGlmICghdXNlTWluaW1hbFRpbGVzRHVy
aW5nTG9hZGluZykKICAgICAgICAgICAgIGJhY2tpbmctPnNldERpZFN3aXRjaFRvRnVsbFRpbGVD
b3ZlcmFnZUR1cmluZ0xvYWRpbmcoKTsKICAgICB9Cg==
</data>
<flag name="review"
          id="221130"
          type_id="1"
          status="+"
          setter="kling"
    />
          </attachment>
      

    </bug>

</bugzilla>