<?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>78230</bug_id>
          
          <creation_ts>2012-02-09 06:26:09 -0800</creation_ts>
          <short_desc>Web Inspector: [refactoring] take _showShortEvents out of timeline calculator</short_desc>
          <delta_ts>2012-02-09 08:54:22 -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="Andrey Kosyakov">caseq</reporter>
          <assigned_to name="Andrey Kosyakov">caseq</assigned_to>
          <cc>apavlov</cc>
    
    <cc>bweinstein</cc>
    
    <cc>keishi</cc>
    
    <cc>loislo</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>553105</commentid>
    <comment_count>0</comment_count>
    <who name="Andrey Kosyakov">caseq</who>
    <bug_when>2012-02-09 06:26:09 -0800</bug_when>
    <thetext>The _showShortEvents property is not used by TimelineCalculator, yet the TimelinePanel put this property on TimelinePanel._calculator. _showShortEvents better belongs to TimelinePanel.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>553108</commentid>
    <comment_count>1</comment_count>
      <attachid>126293</attachid>
    <who name="Andrey Kosyakov">caseq</who>
    <bug_when>2012-02-09 06:28:19 -0800</bug_when>
    <thetext>Created attachment 126293
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>553206</commentid>
    <comment_count>2</comment_count>
    <who name="Andrey Kosyakov">caseq</who>
    <bug_when>2012-02-09 08:54:22 -0800</bug_when>
    <thetext>Committed r107251: &lt;http://trac.webkit.org/changeset/107251&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>126293</attachid>
            <date>2012-02-09 06:28:19 -0800</date>
            <delta_ts>2012-02-09 07:00:28 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-78230-20120209182817.patch</filename>
            <type>text/plain</type>
            <size>3773</size>
            <attacher name="Andrey Kosyakov">caseq</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTA3MjE2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMWQ2MWU4ZGQ3ZjIxMjhk
YWY2NmU4NGY3ZmRhY2M0ZjNlZDAxM2Y5OS4uNzM2ZDhhNjk3OTEyNDQ5YjVlMWU2MGVhZGQxZDc3
Y2UyMmY0ZWNlYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEyLTAyLTA5ICBBbmRy
ZXkgS29zeWFrb3YgIDxjYXNlcUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgV2ViIEluc3BlY3Rv
cjogW3JlZmFjdG9yaW5nXSB0YWtlIF9zaG93U2hvcnRFdmVudHMgb3V0IG9mIHRpbWVsaW5lIGNh
bGN1bGF0b3IKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk
PTc4MjMwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
KiBpbnNwZWN0b3IvZnJvbnQtZW5kL1RpbWVsaW5lUGFuZWwuanM6CisgICAgICAgIChXZWJJbnNw
ZWN0b3IuVGltZWxpbmVQYW5lbCk6CisgICAgICAgIChXZWJJbnNwZWN0b3IuVGltZWxpbmVQYW5l
bC5wcm90b3R5cGUuX3RvZ2dsZUZpbHRlckJ1dHRvbkNsaWNrZWQpOgorICAgICAgICAoV2ViSW5z
cGVjdG9yLlRpbWVsaW5lUGFuZWwucHJvdG90eXBlLl9yZWZyZXNoKToKKyAgICAgICAgKFdlYklu
c3BlY3Rvci5UaW1lbGluZVBhbmVsLnByb3RvdHlwZS5fYWRkVG9SZWNvcmRzV2luZG93KToKKwog
MjAxMi0wMi0wOSAgUm9sYW5kIFN0ZWluZXIgIDxyb2xhbmRzdGVpbmVyQGNocm9taXVtLm9yZz4K
IAogICAgICAgICBTZWxlY3RvckNoZWNrZXI6OmNoZWNrU2VsZWN0b3I6IG1vdmUgcGFyYW1ldGVy
cyBpbnRvIGEgc3RydWN0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJv
bnQtZW5kL1RpbWVsaW5lUGFuZWwuanMgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQt
ZW5kL1RpbWVsaW5lUGFuZWwuanMKaW5kZXggMjhkYjRlZDA1OTJjY2QwNTkwMDJkZTAzMGRmM2Ji
OGQ5MjAxNWI5OC4uYTdjODUyNjdmMWM0MmViOTY3YmE2ZjA4NDBkMDgwNDNiYjg3NWRjNiAxMDA2
NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9UaW1lbGluZVBhbmVs
LmpzCisrKyBiL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvVGltZWxpbmVQYW5l
bC5qcwpAQCAtOTUsNyArOTUsNiBAQCBXZWJJbnNwZWN0b3IuVGltZWxpbmVQYW5lbCA9IGZ1bmN0
aW9uKCkKICAgICB0aGlzLl9yZWdpc3RlcmVkQW5pbWF0aW9uQ2FsbGJhY2tSZWNvcmRzID0ge307
CiAKICAgICB0aGlzLl9jYWxjdWxhdG9yID0gbmV3IFdlYkluc3BlY3Rvci5UaW1lbGluZUNhbGN1
bGF0b3IoKTsKLSAgICB0aGlzLl9jYWxjdWxhdG9yLl9zaG93U2hvcnRFdmVudHMgPSBmYWxzZTsK
ICAgICB2YXIgc2hvcnRSZWNvcmRUaHJlc2hvbGRUaXRsZSA9IE51bWJlci5zZWNvbmRzVG9TdHJp
bmcoV2ViSW5zcGVjdG9yLlRpbWVsaW5lUGFuZWwuc2hvcnRSZWNvcmRUaHJlc2hvbGQpOwogICAg
IHRoaXMuX3Nob3dTaG9ydFJlY29yZHNUaXRsZVRleHQgPSBXZWJJbnNwZWN0b3IuVUlTdHJpbmco
IlNob3cgdGhlIHJlY29yZHMgdGhhdCBhcmUgc2hvcnRlciB0aGFuICVzIiwgc2hvcnRSZWNvcmRU
aHJlc2hvbGRUaXRsZSk7CiAgICAgdGhpcy5faGlkZVNob3J0UmVjb3Jkc1RpdGxlVGV4dCA9IFdl
Ykluc3BlY3Rvci5VSVN0cmluZygiSGlkZSB0aGUgcmVjb3JkcyB0aGF0IGFyZSBzaG9ydGVyIHRo
YW4gJXMiLCBzaG9ydFJlY29yZFRocmVzaG9sZFRpdGxlKTsKQEAgLTExMCw3ICsxMDksNyBAQCBX
ZWJJbnNwZWN0b3IuVGltZWxpbmVQYW5lbCA9IGZ1bmN0aW9uKCkKIAogICAgIC8vIERpc2FibGUg
c2hvcnQgZXZlbnRzIGZpbHRlciBieSBkZWZhdWx0LgogICAgIHRoaXMudG9nZ2xlRmlsdGVyQnV0
dG9uLnRvZ2dsZWQgPSB0cnVlOwotICAgIHRoaXMuX2NhbGN1bGF0b3IuX3Nob3dTaG9ydEV2ZW50
cyA9IHRoaXMudG9nZ2xlRmlsdGVyQnV0dG9uLnRvZ2dsZWQ7CisgICAgdGhpcy5fc2hvd1Nob3J0
RXZlbnRzID0gdGhpcy50b2dnbGVGaWx0ZXJCdXR0b24udG9nZ2xlZDsKICAgICB0aGlzLl90aW1l
U3RhbXBSZWNvcmRzID0gW107CiAgICAgdGhpcy5fZXhwYW5kT2Zmc2V0ID0gMTU7CiAKQEAgLTQz
NSw4ICs0MzQsOCBAQCBXZWJJbnNwZWN0b3IuVGltZWxpbmVQYW5lbC5wcm90b3R5cGUgPSB7CiAg
ICAgX3RvZ2dsZUZpbHRlckJ1dHRvbkNsaWNrZWQ6IGZ1bmN0aW9uKCkKICAgICB7CiAgICAgICAg
IHRoaXMudG9nZ2xlRmlsdGVyQnV0dG9uLnRvZ2dsZWQgPSAhdGhpcy50b2dnbGVGaWx0ZXJCdXR0
b24udG9nZ2xlZDsKLSAgICAgICAgdGhpcy5fY2FsY3VsYXRvci5fc2hvd1Nob3J0RXZlbnRzID0g
dGhpcy50b2dnbGVGaWx0ZXJCdXR0b24udG9nZ2xlZDsKLSAgICAgICAgdGhpcy50b2dnbGVGaWx0
ZXJCdXR0b24uZWxlbWVudC50aXRsZSA9IHRoaXMuX2NhbGN1bGF0b3IuX3Nob3dTaG9ydEV2ZW50
cyA/IHRoaXMuX2hpZGVTaG9ydFJlY29yZHNUaXRsZVRleHQgOiB0aGlzLl9zaG93U2hvcnRSZWNv
cmRzVGl0bGVUZXh0OworICAgICAgICB0aGlzLl9zaG93U2hvcnRFdmVudHMgPSB0aGlzLnRvZ2ds
ZUZpbHRlckJ1dHRvbi50b2dnbGVkOworICAgICAgICB0aGlzLnRvZ2dsZUZpbHRlckJ1dHRvbi5l
bGVtZW50LnRpdGxlID0gdGhpcy5fc2hvd1Nob3J0RXZlbnRzID8gdGhpcy5faGlkZVNob3J0UmVj
b3Jkc1RpdGxlVGV4dCA6IHRoaXMuX3Nob3dTaG9ydFJlY29yZHNUaXRsZVRleHQ7CiAgICAgICAg
IHRoaXMuX3NjaGVkdWxlUmVmcmVzaCh0cnVlKTsKICAgICB9LAogCkBAIC02NzIsNyArNjcxLDcg
QEAgV2ViSW5zcGVjdG9yLlRpbWVsaW5lUGFuZWwucHJvdG90eXBlID0gewogICAgICAgICAgICAg
ZGVsZXRlIHRoaXMuX3JlZnJlc2hUaW1lb3V0OwogICAgICAgICB9CiAKLSAgICAgICAgdGhpcy5f
b3ZlcnZpZXdQYW5lLnVwZGF0ZSh0aGlzLl9yb290UmVjb3JkLmNoaWxkcmVuLCB0aGlzLl9jYWxj
dWxhdG9yLl9zaG93U2hvcnRFdmVudHMpOworICAgICAgICB0aGlzLl9vdmVydmlld1BhbmUudXBk
YXRlKHRoaXMuX3Jvb3RSZWNvcmQuY2hpbGRyZW4sIHRoaXMuX3Nob3dTaG9ydEV2ZW50cyk7CiAK
ICAgICAgICAgdGhpcy5fcmVmcmVzaFJlY29yZHMoIXRoaXMuX2JvdW5kYXJpZXNBcmVWYWxpZCk7
CiAgICAgICAgIHRoaXMuX3VwZGF0ZVJlY29yZHNDb3VudGVyKCk7CkBAIC03MDAsNyArNjk5LDcg
QEAgV2ViSW5zcGVjdG9yLlRpbWVsaW5lUGFuZWwucHJvdG90eXBlID0gewogICAgICAqLwogICAg
IF9hZGRUb1JlY29yZHNXaW5kb3c6IGZ1bmN0aW9uKHJlY29yZCwgcmVjb3Jkc1dpbmRvdywgcGFy
ZW50SXNDb2xsYXBzZWQpCiAgICAgewotICAgICAgICBpZiAoIXRoaXMuX2NhbGN1bGF0b3IuX3No
b3dTaG9ydEV2ZW50cyAmJiAhcmVjb3JkLmlzTG9uZygpKQorICAgICAgICBpZiAoIXRoaXMuX3No
b3dTaG9ydEV2ZW50cyAmJiAhcmVjb3JkLmlzTG9uZygpKQogICAgICAgICAgICAgcmV0dXJuOwog
ICAgICAgICB2YXIgcGVyY2VudGFnZXMgPSB0aGlzLl9jYWxjdWxhdG9yLmNvbXB1dGVCYXJHcmFw
aFBlcmNlbnRhZ2VzKHJlY29yZCk7CiAgICAgICAgIGlmIChwZXJjZW50YWdlcy5zdGFydCA8IDEw
MCAmJiBwZXJjZW50YWdlcy5lbmRXaXRoQ2hpbGRyZW4gPj0gMCAmJiAhcmVjb3JkLmNhdGVnb3J5
LmhpZGRlbikgewo=
</data>
<flag name="review"
          id="128066"
          type_id="1"
          status="+"
          setter="pfeldman"
    />
          </attachment>
      

    </bug>

</bugzilla>