<?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>100761</bug_id>
          
          <creation_ts>2012-10-30 07:54:19 -0700</creation_ts>
          <short_desc>Web Inspector: Timeline: make &quot;addRecord&quot; unambiguous</short_desc>
          <delta_ts>2012-11-01 06:50:56 -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>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>
          
          <blocked>99830</blocked>
          <everconfirmed>0</everconfirmed>
          <reporter>eustas.bug</reporter>
          <assigned_to>eustas.bug</assigned_to>
          <cc>apavlov</cc>
    
    <cc>bweinstein</cc>
    
    <cc>caseq</cc>
    
    <cc>eustas.bug</cc>
    
    <cc>joepeck</cc>
    
    <cc>keishi</cc>
    
    <cc>loislo</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>vsevik</cc>
    
    <cc>web-inspector-bugs</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>754366</commentid>
    <comment_count>0</comment_count>
    <who name="">eustas.bug</who>
    <bug_when>2012-10-30 07:54:19 -0700</bug_when>
    <thetext>TimelinePresentationModel.addRecord accepts two parameters - record and parentRecord.
parentRecord is always root record, but that is not clear.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754369</commentid>
    <comment_count>1</comment_count>
      <attachid>171451</attachid>
    <who name="">eustas.bug</who>
    <bug_when>2012-10-30 07:58:08 -0700</bug_when>
    <thetext>Created attachment 171451
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754381</commentid>
    <comment_count>2</comment_count>
      <attachid>171451</attachid>
    <who name="Andrey Kosyakov">caseq</who>
    <bug_when>2012-10-30 08:15:33 -0700</bug_when>
    <thetext>Comment on attachment 171451
Patch

LGTM</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>755085</commentid>
    <comment_count>3</comment_count>
      <attachid>171451</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-31 02:47:25 -0700</bug_when>
    <thetext>Comment on attachment 171451
Patch

Rejecting attachment 171451 from commit-queue.

Failed to run &quot;[&apos;/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch&apos;, &apos;--status-host=queues.webkit.org&apos;, &apos;-...&quot; exit_code: 1

Last 500 characters of output:
t/git/webkit-commit-queue/Source/WebKit/chromium/v8 --revision 12787 --non-interactive --force --accept theirs-conflict --ignore-externals&apos; in &apos;/mnt/git/webkit-commit-queue/Source/WebKit/chromium&apos;
50&gt;At revision 12787.

________ running &apos;/usr/bin/python tools/clang/scripts/update.py --mac-only&apos; in &apos;/mnt/git/webkit-commit-queue/Source/WebKit/chromium&apos;

________ running &apos;/usr/bin/python gyp_webkit&apos; in &apos;/mnt/git/webkit-commit-queue/Source/WebKit/chromium&apos;
Updating webkit projects from gyp files...

Full output: http://queues.webkit.org/results/14687094</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>756177</commentid>
    <comment_count>4</comment_count>
      <attachid>171451</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-11-01 06:50:52 -0700</bug_when>
    <thetext>Comment on attachment 171451
Patch

Clearing flags on attachment: 171451

Committed r133157: &lt;http://trac.webkit.org/changeset/133157&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>756178</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-11-01 06:50:56 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>171451</attachid>
            <date>2012-10-30 07:58:08 -0700</date>
            <delta_ts>2012-11-01 06:50:52 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-100761-20121030185632.patch</filename>
            <type>text/plain</type>
            <size>3847</size>
            <attacher>eustas.bug</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMyODk1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYzExYWIzZTNhMmRhMWVk
ZTkxODMwMmNhZTE3NjE4ZDY4OTE3MDI1Mi4uZWNjM2RkNzVkNWUxNzBjNzY3ZDJlMDk5NWJmNjNi
YjcxNTJiNDlkZSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5IEBACisyMDEyLTEwLTMwICBFdWdl
bmUgS2x5dWNobmlrb3YgIDxldXN0YXMuYnVnQGdtYWlsLmNvbT4KKworICAgICAgICBXZWIgSW5z
cGVjdG9yOiBUaW1lbGluZTogbWFrZSAiYWRkUmVjb3JkIiB1bmFtYmlndW91cworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTAwNzYxCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGltZWxpbmVQcmVzZW50YXRp
b25Nb2RlbC5hZGRSZWNvcmQgYWNjZXB0cyB0d28gcGFyYW1ldGVyczoKKyAgICAgICAgcmVjb3Jk
IGFuZCBwYXJlbnRSZWNvcmQuIHBhcmVudFJlY29yZCBpcyBhbHdheXMgcm9vdCByZWNvcmQuCisg
ICAgICAgIE1ha2UgdGhpcyBleHBsaWNpdCBieSByZW1vdmluZyBwYXJlbnRSZWNvcmQgcGFyYW1l
dGVyLgorCisgICAgICAgICogaW5zcGVjdG9yL2Zyb250LWVuZC9UaW1lbGluZVBhbmVsLmpzOiBS
ZW1vdmVkIHBhcmFtZXRlci4KKyAgICAgICAgKiBpbnNwZWN0b3IvZnJvbnQtZW5kL1RpbWVsaW5l
UHJlc2VudGF0aW9uTW9kZWwuanM6CisgICAgICAgIChXZWJJbnNwZWN0b3IuVGltZWxpbmVQcmVz
ZW50YXRpb25Nb2RlbC5wcm90b3R5cGUuYWRkUmVjb3JkKToKKyAgICAgICAgVXNlZCByb290IHJl
Y29lZCBpbnN0ZWFkIG9mIHBhcmVudCByZWNvcmQuCisKIDIwMTItMTAtMzAgIEtlbnQgVGFtdXJh
ICA8dGtlbnRAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIEZpeCBjcmFzaCBieSBjYWxlbmRhciBw
aWNrZXIgb3Igc3VnZ2VzdGlvbiBwaWNrZXIKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2lu
c3BlY3Rvci9mcm9udC1lbmQvVGltZWxpbmVQYW5lbC5qcyBiL1NvdXJjZS9XZWJDb3JlL2luc3Bl
Y3Rvci9mcm9udC1lbmQvVGltZWxpbmVQYW5lbC5qcwppbmRleCAxNWM5YTViOGY3ODhiYWUxZjIz
MmEzZWVlMTYyYzI3YjAyMWNkMzgxLi5mM2E5ZWIzMTkwNWUxMTAzMTUxMzczODJjMmU5OWM4OWNi
MDFiNzRkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1Rp
bWVsaW5lUGFuZWwuanMKKysrIGIvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9U
aW1lbGluZVBhbmVsLmpzCkBAIC01NTMsMTcgKzU1MywxNyBAQCBXZWJJbnNwZWN0b3IuVGltZWxp
bmVQYW5lbC5wcm90b3R5cGUgPSB7CiAgICAgICAgIHRoaXMuX2F1dG9tYXRpY2FsbHlTaXplV2lu
ZG93ID0gZmFsc2U7CiAgICAgICAgIHZhciByZWNvcmRzID0gdGhpcy5fbW9kZWwucmVjb3JkczsK
ICAgICAgICAgZm9yICh2YXIgaSA9IDA7IGkgPCByZWNvcmRzLmxlbmd0aDsgKytpKQotICAgICAg
ICAgICAgdGhpcy5faW5uZXJBZGRSZWNvcmRUb1RpbWVsaW5lKHJlY29yZHNbaV0sIHRoaXMuX3Jv
b3RSZWNvcmQoKSk7CisgICAgICAgICAgICB0aGlzLl9pbm5lckFkZFJlY29yZFRvVGltZWxpbmUo
cmVjb3Jkc1tpXSk7CiAgICAgICAgIHRoaXMuX2ludmFsaWRhdGVBbmRTY2hlZHVsZVJlZnJlc2go
ZmFsc2UpOwogICAgIH0sCiAKICAgICBfb25UaW1lbGluZUV2ZW50UmVjb3JkZWQ6IGZ1bmN0aW9u
KGV2ZW50KQogICAgIHsKLSAgICAgICAgaWYgKHRoaXMuX2lubmVyQWRkUmVjb3JkVG9UaW1lbGlu
ZShldmVudC5kYXRhLCB0aGlzLl9yb290UmVjb3JkKCkpKQorICAgICAgICBpZiAodGhpcy5faW5u
ZXJBZGRSZWNvcmRUb1RpbWVsaW5lKGV2ZW50LmRhdGEpKQogICAgICAgICAgICAgdGhpcy5faW52
YWxpZGF0ZUFuZFNjaGVkdWxlUmVmcmVzaChmYWxzZSk7CiAgICAgfSwKIAotICAgIF9pbm5lckFk
ZFJlY29yZFRvVGltZWxpbmU6IGZ1bmN0aW9uKHJlY29yZCwgcGFyZW50UmVjb3JkKQorICAgIF9p
bm5lckFkZFJlY29yZFRvVGltZWxpbmU6IGZ1bmN0aW9uKHJlY29yZCkKICAgICB7CiAgICAgICAg
IGlmIChyZWNvcmQudHlwZSA9PT0gV2ViSW5zcGVjdG9yLlRpbWVsaW5lTW9kZWwuUmVjb3JkVHlw
ZS5Qcm9ncmFtKSB7CiAgICAgICAgICAgICB0aGlzLl9tYWluVGhyZWFkVGFza3MucHVzaCh7CkBA
IC01NzIsNyArNTcyLDcgQEAgV2ViSW5zcGVjdG9yLlRpbWVsaW5lUGFuZWwucHJvdG90eXBlID0g
ewogICAgICAgICAgICAgfSk7CiAgICAgICAgIH0KIAotICAgICAgICB2YXIgcmVjb3JkcyA9IHRo
aXMuX3ByZXNlbnRhdGlvbk1vZGVsLmFkZFJlY29yZChyZWNvcmQsIHBhcmVudFJlY29yZCk7Cisg
ICAgICAgIHZhciByZWNvcmRzID0gdGhpcy5fcHJlc2VudGF0aW9uTW9kZWwuYWRkUmVjb3JkKHJl
Y29yZCk7CiAgICAgICAgIHRoaXMuX2FsbFJlY29yZHNDb3VudCArPSByZWNvcmRzLmxlbmd0aDsK
ICAgICAgICAgdmFyIHRpbWVTdGFtcFJlY29yZHMgPSB0aGlzLl90aW1lU3RhbXBSZWNvcmRzOwog
ICAgICAgICB2YXIgaGFzVmlzaWJsZVJlY29yZHMgPSBmYWxzZTsKZGlmZiAtLWdpdCBhL1NvdXJj
ZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvVGltZWxpbmVQcmVzZW50YXRpb25Nb2RlbC5q
cyBiL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvVGltZWxpbmVQcmVzZW50YXRp
b25Nb2RlbC5qcwppbmRleCA3MDIwZDFhYTM3NzFhMTI5NDQ0NmE5ODYwZjdlYzI0NWYyOWE2MzFm
Li5hNWJiNjA0Njk3ODRmMDViYTllM2Y0ZDA4NDg5ODkwYTgxNGI1NDdlIDEwMDY0NAotLS0gYS9T
b3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1RpbWVsaW5lUHJlc2VudGF0aW9uTW9k
ZWwuanMKKysrIGIvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9UaW1lbGluZVBy
ZXNlbnRhdGlvbk1vZGVsLmpzCkBAIC0yNzQsNyArMjc0LDcgQEAgV2ViSW5zcGVjdG9yLlRpbWVs
aW5lUHJlc2VudGF0aW9uTW9kZWwucHJvdG90eXBlID0gewogICAgICAgICB0aGlzLl9mcmFtZXMu
cHVzaChmcmFtZSk7CiAgICAgfSwKIAotICAgIGFkZFJlY29yZDogZnVuY3Rpb24ocmVjb3JkLCBw
YXJlbnRSZWNvcmQpCisgICAgYWRkUmVjb3JkOiBmdW5jdGlvbihyZWNvcmQpCiAgICAgewogICAg
ICAgICBpZiAodGhpcy5fbWluaW11bVJlY29yZFRpbWUgPT09IC0xIHx8IHJlY29yZC5zdGFydFRp
bWUgPCB0aGlzLl9taW5pbXVtUmVjb3JkVGltZSkKICAgICAgICAgICAgIHRoaXMuX21pbmltdW1S
ZWNvcmRUaW1lID0gV2ViSW5zcGVjdG9yLlRpbWVsaW5lTW9kZWwuc3RhcnRUaW1lSW5TZWNvbmRz
KHJlY29yZCk7CkBAIC0yODgsNyArMjg4LDcgQEAgV2ViSW5zcGVjdG9yLlRpbWVsaW5lUHJlc2Vu
dGF0aW9uTW9kZWwucHJvdG90eXBlID0gewogICAgICAgICB2YXIgZm9ybWF0dGVkUmVjb3JkcyA9
IFtdOwogICAgICAgICB2YXIgcmVjb3Jkc0NvdW50ID0gcmVjb3Jkcy5sZW5ndGg7CiAgICAgICAg
IGZvciAodmFyIGkgPSAwOyBpIDwgcmVjb3Jkc0NvdW50OyArK2kpCi0gICAgICAgICAgICBmb3Jt
YXR0ZWRSZWNvcmRzLnB1c2godGhpcy5faW5uZXJBZGRSZWNvcmQocmVjb3Jkc1tpXSwgcGFyZW50
UmVjb3JkKSk7CisgICAgICAgICAgICBmb3JtYXR0ZWRSZWNvcmRzLnB1c2godGhpcy5faW5uZXJB
ZGRSZWNvcmQocmVjb3Jkc1tpXSwgdGhpcy5fcm9vdFJlY29yZCkpOwogICAgICAgICByZXR1cm4g
Zm9ybWF0dGVkUmVjb3JkczsKICAgICB9LAogCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>