<?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>80409</bug_id>
          
          <creation_ts>2012-03-06 05:28:39 -0800</creation_ts>
          <short_desc>Web Inspector: Heap Snapshot performance test improvements.</short_desc>
          <delta_ts>2012-03-06 05:41:48 -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>Web Inspector (Deprecated)</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Ilya Tikhonovsky">loislo</reporter>
          <assigned_to name="Ilya Tikhonovsky">loislo</assigned_to>
          <cc>apavlov</cc>
    
    <cc>bweinstein</cc>
    
    <cc>joepeck</cc>
    
    <cc>keishi</cc>
    
    <cc>loislo</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>timothy</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>571763</commentid>
    <comment_count>0</comment_count>
    <who name="Ilya Tikhonovsky">loislo</who>
    <bug_when>2012-03-06 05:28:39 -0800</bug_when>
    <thetext>The test has only overall snapshot time metric at the moment.
It&apos;d be much more interesting to see the times for each snapshot stage.

the sample:
RESULT panel-update: take-snapshot= 461 ms
RESULT panel-update: transfer-snapshot= 789 ms
RESULT panel-update: show-snapshot= 1121 ms
RESULT panel-update: summary-snapshot-time= 2373 ms
RESULT panel-update: switch-to-containment-view= 2788 ms
RESULT panel-update: clear-snapshot= 13 ms
RESULT heap-delta: detailed-heapshots-smoke-test= 240 kB</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>571766</commentid>
    <comment_count>1</comment_count>
      <attachid>130362</attachid>
    <who name="Ilya Tikhonovsky">loislo</who>
    <bug_when>2012-03-06 05:29:53 -0800</bug_when>
    <thetext>Created attachment 130362
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>571773</commentid>
    <comment_count>2</comment_count>
    <who name="Ilya Tikhonovsky">loislo</who>
    <bug_when>2012-03-06 05:41:48 -0800</bug_when>
    <thetext>Committed r109913: &lt;http://trac.webkit.org/changeset/109913&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>130362</attachid>
            <date>2012-03-06 05:29:53 -0800</date>
            <delta_ts>2012-03-06 05:32:17 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-80409-20120306172951.patch</filename>
            <type>text/plain</type>
            <size>3218</size>
            <attacher name="Ilya Tikhonovsky">loislo</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTA5OTA4CmRpZmYgLS1naXQgYS9QZXJmb3JtYW5jZVRlc3Rz
L0NoYW5nZUxvZyBiL1BlcmZvcm1hbmNlVGVzdHMvQ2hhbmdlTG9nCmluZGV4IDM4YmMxNjQ1ZjM1
NTM3MTE5NzdjZmQ0ZTk5OTNjZTk4ZjdjYTE3MGUuLjI2ZTY1M2FkODUxN2RjMzEyZjZiNzFmMTQ0
MmMwYjgxYTBhZDBjMjkgMTAwNjQ0Ci0tLSBhL1BlcmZvcm1hbmNlVGVzdHMvQ2hhbmdlTG9nCisr
KyBiL1BlcmZvcm1hbmNlVGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTItMDMt
MDYgIElseWEgVGlraG9ub3Zza3kgIDxsb2lzbG9AY2hyb21pdW0ub3JnPgorCisgICAgICAgIFdl
YiBJbnNwZWN0b3I6IEhlYXAgU25hcHNob3QgdGVzdDogc3BsaXQgb3ZlcmFsIHRpbWUgbWV0cmlj
IGludG8gc2VwYXJhdGUgbWV0cmljcy4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTgwNDA5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgKiBpbnNwZWN0b3IvZGV0YWlsZWQtaGVhcHNob3RzLXNtb2tlLXRlc3Qu
aHRtbDoKKwogMjAxMi0wMy0wNSAgSWx5YSBUaWtob25vdnNreSAgPGxvaXNsb0BjaHJvbWl1bS5v
cmc+CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogW2Nocm9taXVtXSBpbnRyb2R1Y2UgSGVhcFNu
YXBzaG90IHBlcmZvcm1hbmNlIHRlc3QuCmRpZmYgLS1naXQgYS9QZXJmb3JtYW5jZVRlc3RzL2lu
c3BlY3Rvci9kZXRhaWxlZC1oZWFwc2hvdHMtc21va2UtdGVzdC5odG1sIGIvUGVyZm9ybWFuY2VU
ZXN0cy9pbnNwZWN0b3IvZGV0YWlsZWQtaGVhcHNob3RzLXNtb2tlLXRlc3QuaHRtbAppbmRleCA3
OTEyMzI0NGY4YzNjNGQ5NjcwMDFhNmRlN2QxMzA4OGI4MGRkNzgyLi4xMzEzNzQyMDU4ZTcyMDAx
OGM2ZTQyNmQ1YzNhNDkwNmM2M2IyODMwIDEwMDY0NAotLS0gYS9QZXJmb3JtYW5jZVRlc3RzL2lu
c3BlY3Rvci9kZXRhaWxlZC1oZWFwc2hvdHMtc21va2UtdGVzdC5odG1sCisrKyBiL1BlcmZvcm1h
bmNlVGVzdHMvaW5zcGVjdG9yL2RldGFpbGVkLWhlYXBzaG90cy1zbW9rZS10ZXN0Lmh0bWwKQEAg
LTEwLDExICsxMCwxOSBAQCBmdW5jdGlvbiB0ZXN0KCkKICAgICBmdW5jdGlvbiBwZXJmb3JtYW5j
ZVRlc3QodGltZXIpCiAgICAgewogCi0gICAgICAgIHZhciBzaG93UGFuZWxUaW1lckNvb2tpZSA9
IHRpbWVyLnN0YXJ0KCJwYW5lbC11cGRhdGUiKTsKKyAgICAgICAgdmFyIHRyYW5zZmVyVGltZXJD
b29raWU7CisgICAgICAgIHZhciBzaG93VGltZXJDb29raWU7CisgICAgICAgIHZhciBjaGFuZ2VW
aWV3VGltZXJDb29raWU7CisgICAgICAgIHZhciBjbGVhclRpbWVyQ29va2llOworCisgICAgICAg
IHZhciBhbGxUaW1lckNvb2tpZSA9IHRpbWVyLnN0YXJ0KCJzdW1tYXJ5LXNuYXBzaG90LXRpbWUi
KTsKKyAgICAgICAgdmFyIGJhY2tlbmRUaW1lckNvb2tpZSA9IHRpbWVyLnN0YXJ0KCJ0YWtlLXNu
YXBzaG90Iik7CiAgICAgICAgIFByb2ZpbGVyQWdlbnQudGFrZUhlYXBTbmFwc2hvdChzdGVwMCk7
CiAKICAgICAgICAgZnVuY3Rpb24gc3RlcDAoKQogICAgICAgICB7CisgICAgICAgICAgICB0aW1l
ci5maW5pc2goYmFja2VuZFRpbWVyQ29va2llKTsKKyAgICAgICAgICAgIHRyYW5zZmVyVGltZXJD
b29raWUgPSB0aW1lci5zdGFydCgidHJhbnNmZXItc25hcHNob3QiKTsKICAgICAgICAgICAgIHZh
ciBwcm9maWxlcyA9IFdlYkluc3BlY3Rvci5wYW5lbHMucHJvZmlsZXMuZ2V0UHJvZmlsZXMoIkhF
QVAiKTsKICAgICAgICAgICAgIFdlYkluc3BlY3Rvci5wYW5lbHMucHJvZmlsZXMuc2hvd1Byb2Zp
bGUocHJvZmlsZXNbcHJvZmlsZXMubGVuZ3RoIC0gMV0pOwogICAgICAgICAgICAgSW5zcGVjdG9y
VGVzdC5hZGRTbmlmZmVyKFdlYkluc3BlY3Rvci5wYW5lbHMucHJvZmlsZXMsICJfZmluaXNoSGVh
cFNuYXBzaG90Iiwgc3RlcDEpOwpAQCAtMjIsNiArMzAsOCBAQCBmdW5jdGlvbiB0ZXN0KCkKIAog
ICAgICAgICBmdW5jdGlvbiBzdGVwMSh1aWQpCiAgICAgICAgIHsKKyAgICAgICAgICAgIHRpbWVy
LmZpbmlzaCh0cmFuc2ZlclRpbWVyQ29va2llKTsKKyAgICAgICAgICAgIHNob3dUaW1lckNvb2tp
ZSA9IHRpbWVyLnN0YXJ0KCJzaG93LXNuYXBzaG90Iik7CiAgICAgICAgICAgICB2YXIgcGFuZWwg
PSBXZWJJbnNwZWN0b3IucGFuZWxzLnByb2ZpbGVzOwogICAgICAgICAgICAgdmFyIHByb2ZpbGUg
PSBwYW5lbC5fcHJvZmlsZXNJZE1hcFtwYW5lbC5fbWFrZUtleSh1aWQsIFdlYkluc3BlY3Rvci5E
ZXRhaWxlZEhlYXBzaG90UHJvZmlsZVR5cGUuVHlwZUlkKV07CiAgICAgICAgICAgICBJbnNwZWN0
b3JUZXN0LmFkZFNuaWZmZXIocHJvZmlsZS5wcm94eSwgIl9jYWxsTG9hZENhbGxiYWNrcyIsIHN0
ZXAyKTsKQEAgLTI5LDE5ICszOSwyNCBAQCBmdW5jdGlvbiB0ZXN0KCkKIAogICAgICAgICBmdW5j
dGlvbiBzdGVwMigpCiAgICAgICAgIHsKKyAgICAgICAgICAgIHRpbWVyLmZpbmlzaChzaG93VGlt
ZXJDb29raWUpOworICAgICAgICAgICAgdGltZXIuZmluaXNoKGFsbFRpbWVyQ29va2llKTsKKyAg
ICAgICAgICAgIGNoYW5nZVZpZXdUaW1lckNvb2tpZSA9IHRpbWVyLnN0YXJ0KCJzd2l0Y2gtdG8t
Y29udGFpbm1lbnQtdmlldyIpOwogICAgICAgICAgICAgSW5zcGVjdG9yVGVzdC5zd2l0Y2hUb1Zp
ZXcoIkNvbnRhaW5tZW50IiwgY2xlYW51cCk7CiAgICAgICAgIH0KIAogICAgICAgICBmdW5jdGlv
biBjbGVhbnVwKCkKICAgICAgICAgewotICAgICAgICAgICAgdGltZXIuZmluaXNoKHNob3dQYW5l
bFRpbWVyQ29va2llKTsKKyAgICAgICAgICAgIHRpbWVyLmZpbmlzaChjaGFuZ2VWaWV3VGltZXJD
b29raWUpOworICAgICAgICAgICAgY2xlYXJUaW1lckNvb2tpZSA9IHRpbWVyLnN0YXJ0KCJjbGVh
ci1zbmFwc2hvdCIpOwogICAgICAgICAgICAgUHJvZmlsZXJBZ2VudC5jbGVhclByb2ZpbGVzKGRv
bmUpOwogICAgICAgICAgICAgV2ViSW5zcGVjdG9yLnBhbmVscy5wcm9maWxlcy5fcmVzZXQoKTsK
ICAgICAgICAgfQogCiAgICAgICAgIGZ1bmN0aW9uIGRvbmUoKQogICAgICAgICB7Ci0gICAgICAg
ICAgICB0aW1lci5kb25lKCJ0YWtlLXNuYXBzaG90Iik7CisgICAgICAgICAgICB0aW1lci5maW5p
c2goY2xlYXJUaW1lckNvb2tpZSk7CisgICAgICAgICAgICB0aW1lci5kb25lKCJwYW5lbC11cGRh
dGUiKTsKICAgICAgICAgfQogICAgIH0KIAo=
</data>
<flag name="review"
          id="133344"
          type_id="1"
          status="+"
          setter="yurys"
    />
          </attachment>
      

    </bug>

</bugzilla>