<?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>168626</bug_id>
          
          <creation_ts>2017-02-20 17:38:42 -0800</creation_ts>
          <short_desc>Add machine-readable results for bindings tests</short_desc>
          <delta_ts>2017-03-03 12:50:20 -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>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=138235</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=169144</see_also>
          <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>1</everconfirmed>
          <reporter name="Srinivasan Vijayaraghavan">webkit</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>aakash_jain</cc>
    
    <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>glenn</cc>
    
    <cc>jake.nielsen.webkit</cc>
    
    <cc>lforschler</cc>
    
    <cc>webkit</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1279157</commentid>
    <comment_count>0</comment_count>
    <who name="Srinivasan Vijayaraghavan">webkit</who>
    <bug_when>2017-02-20 17:38:42 -0800</bug_when>
    <thetext>Adding JSON results to bindings tests is a prerequisite to running bindings tests on EWS.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1279159</commentid>
    <comment_count>1</comment_count>
      <attachid>302213</attachid>
    <who name="Srinivasan Vijayaraghavan">webkit</who>
    <bug_when>2017-02-20 17:39:33 -0800</bug_when>
    <thetext>Created attachment 302213
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1279160</commentid>
    <comment_count>2</comment_count>
    <who name="Srinivasan Vijayaraghavan">webkit</who>
    <bug_when>2017-02-20 17:42:15 -0800</bug_when>
    <thetext>Note: webkit-patch upload failed trying to obsolete the attachment on the original bug, so I created a new bug and added the original to the &apos;See also&apos; field.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1279229</commentid>
    <comment_count>3</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2017-02-20 22:35:33 -0800</bug_when>
    <thetext>*** Bug 138235 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1279230</commentid>
    <comment_count>4</comment_count>
      <attachid>302213</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2017-02-20 22:39:07 -0800</bug_when>
    <thetext>Comment on attachment 302213
Patch

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

Looks reasonable to me, but I&apos;d like Python experts to take a look too.

&gt; Tools/Scripts/webkitpy/bindings/main.py:209
&gt; +                &apos;passes&apos;: self.passes,

Why is it desirable to add passes to the results too? I don&apos;t think that we do it elsewhere.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1279392</commentid>
    <comment_count>5</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2017-02-21 09:59:29 -0800</bug_when>
    <thetext>looks good to me, apart from Alexey comment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1280506</commentid>
    <comment_count>6</comment_count>
    <who name="Srinivasan Vijayaraghavan">webkit</who>
    <bug_when>2017-02-23 14:31:27 -0800</bug_when>
    <thetext>(In reply to comment #4)
&gt; Comment on attachment 302213 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=302213&amp;action=review
&gt; 
&gt; Looks reasonable to me, but I&apos;d like Python experts to take a look too.
&gt; 
&gt; &gt; Tools/Scripts/webkitpy/bindings/main.py:209
&gt; &gt; +                &apos;passes&apos;: self.passes,
&gt; 
&gt; Why is it desirable to add passes to the results too? I don&apos;t think that we
&gt; do it elsewhere.

There&apos;s no need for it; I&apos;ll upload a new patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1280511</commentid>
    <comment_count>7</comment_count>
      <attachid>302581</attachid>
    <who name="Srinivasan Vijayaraghavan">webkit</who>
    <bug_when>2017-02-23 14:43:01 -0800</bug_when>
    <thetext>Created attachment 302581
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1280588</commentid>
    <comment_count>8</comment_count>
      <attachid>302581</attachid>
    <who name="Dean Johnson">dean_johnson</who>
    <bug_when>2017-02-23 17:27:25 -0800</bug_when>
    <thetext>Comment on attachment 302581
Patch

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

Looks good overall. Food for thought below.

&gt; Tools/Scripts/webkitpy/bindings/main.py:47
&gt; +        if self.json_file_name:

All of the &apos;if self.json_file_name&apos; might be a bit pre-mature for optimization. I don&apos;t think we will really see a performance hit from always appending errors/failure. The only one conditional I feel should remain is the one that actually does the writing to a file.

&gt; Tools/Scripts/webkitpy/bindings/main.py:118
&gt; +                if self.json_file_name:

Ditto.

&gt; Tools/Scripts/webkitpy/bindings/main.py:125
&gt; +                if self.json_file_name:

Ditto.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1280872</commentid>
    <comment_count>9</comment_count>
      <attachid>302581</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2017-02-24 14:18:28 -0800</bug_when>
    <thetext>Comment on attachment 302581
Patch

I don&apos;t have a string opinion on the premature optimization. I agree that this is not necessary for performance, but the code may be easier to read when unnecessary work is not being done.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1281560</commentid>
    <comment_count>10</comment_count>
      <attachid>302581</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-02-27 15:09:18 -0800</bug_when>
    <thetext>Comment on attachment 302581
Patch

Clearing flags on attachment: 302581

Committed r213097: &lt;http://trac.webkit.org/changeset/213097&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1281561</commentid>
    <comment_count>11</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-02-27 15:09:23 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>302213</attachid>
            <date>2017-02-20 17:39:33 -0800</date>
            <delta_ts>2017-02-23 14:42:57 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-168626-20170220173650.patch</filename>
            <type>text/plain</type>
            <size>5105</size>
            <attacher name="Srinivasan Vijayaraghavan">webkit</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjEyNjcxCmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggNTk4NmE0NjQ3Zjg0YjJkZjA4MjAzYjgwZDU5NTc4NjIw
NjQxMjkwOS4uYTU2OGYwOWZjMjgwYzg5ZmI3MmFlOTdjNDI2ZDJmOWI1MTdhOGI4ZiAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3
IEBACisyMDE3LTAyLTIwICBTcmluaXZhc2FuIFZpamF5YXJhZ2hhdmFuICA8c3ZpamF5YXJhZ2hh
dmFuQGFwcGxlLmNvbT4KKworICAgICAgICBBZGQgbWFjaGluZS1yZWFkYWJsZSByZXN1bHRzIGZv
ciBiaW5kaW5ncyB0ZXN0cworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MTY4NjI2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgKiBTY3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0czoKKyAgICAgICAgKG1haW4pOiBB
ZGQgb3B0aW9uYWwgLS1qc29uLW91dHB1dCBjb21tYW5kLWxpbmUgcGFyYW1ldGVyLgorICAgICAg
ICAqIFNjcmlwdHMvd2Via2l0cHkvYmluZGluZ3MvbWFpbi5weToKKyAgICAgICAgKEJpbmRpbmdz
VGVzdHMuX19pbml0X18pOiBBZGQgdmFyaWFibGVzIHRvIHN0b3JlIG1hY2hpbmUtcmVhZGFibGUg
cmVzdWx0cy4KKyAgICAgICAgKEJpbmRpbmdzVGVzdHMuZGV0ZWN0X2NoYW5nZXMpOiBTdG9yZSBy
ZXN1bHRzIGluIG1hY2hpbmUtcmVhZGFibGUgZm9ybSBpZiBhcHBsaWNhYmxlLgorICAgICAgICAo
QmluZGluZ3NUZXN0cy5tYWluKTogV3JpdGUgZGF0YSB0byBKU09OIGZpbGUgaWYgYXBwbGljYWJs
ZS4KKwogMjAxNy0wMi0xNyAgQW5kZXJzIENhcmxzc29uICA8YW5kZXJzY2FAYXBwbGUuY29tPgog
CiAgICAgICAgIEFkZCBhIG5ldyBkcmFnIGNvZGUgcGF0aCBhbmQgdXNlIGl0IGZvciBkcmFnZ2lu
ZyBwbGFpbiB0ZXh0CmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0
cyBiL1Rvb2xzL1NjcmlwdHMvcnVuLWJpbmRpbmdzLXRlc3RzCmluZGV4IDlmN2MxYTFlOTMwMzBk
ODQ1NjY0ODAwNTQ4NTNiZDFiNDFkNTEyMDAuLmMwOGI2M2NmYmZiNDdhM2I2YzJkMjQ3MTRmYjVl
ZGFiYTY3ZGQ0YmUgMTAwNzU1Ci0tLSBhL1Rvb2xzL1NjcmlwdHMvcnVuLWJpbmRpbmdzLXRlc3Rz
CisrKyBiL1Rvb2xzL1NjcmlwdHMvcnVuLWJpbmRpbmdzLXRlc3RzCkBAIC0yOSw2ICsyOSw3IEBA
CiAjIHBhdGNoLiBUaGlzIG1ha2VzIGl0IGVhc2llciB0byB0cmFjayBhbmQgcmV2aWV3IGNoYW5n
ZXMgaW4gZ2VuZXJhdGVkIGNvZGUuCiAKIGltcG9ydCBvcHRwYXJzZQoraW1wb3J0IG9zCiBpbXBv
cnQgc3lzCiBmcm9tIHdlYmtpdHB5LmNvbW1vbi5zeXN0ZW0gaW1wb3J0IGV4ZWN1dGl2ZQogCkBA
IC00MywxMSArNDQsMTYgQEAgZGVmIG1haW4oYXJndik6CiAgICAgICAgIGhlbHA9J0Rpc2FibGUg
dmVyb2JzZSBvdXRwdXQnKQogICAgIG9wdGlvbl9wYXJzZXIuYWRkX29wdGlvbignLS1yZXNldC1y
ZXN1bHRzJywgYWN0aW9uPSdzdG9yZV90cnVlJywgZGVmYXVsdD1GYWxzZSwKICAgICAgICAgaGVs
cD0nT3ZlcndyaXRlcyB0aGUgcmVmZXJlbmNlIGZpbGVzIHdpdGggdGhlIGdlbmVyYXRlZCByZXN1
bHRzJykKKyAgICBvcHRpb25fcGFyc2VyLmFkZF9vcHRpb24oJy0tanNvbi1vdXRwdXQnLCBhY3Rp
b249J3N0b3JlJywgdHlwZT0nc3RyaW5nJywgZGVzdD0nanNvbl9maWxlX25hbWUnLAorICAgICAg
ICBoZWxwPSdDcmVhdGUgYSBmaWxlIGF0IHNwZWNpZmllZCBwYXRoLCBsaXN0aW5nIHRlc3QgcmVz
dWx0cyBpbiBKU09OIGZvcm1hdC4nKQogICAgIG9wdGlvbnMsIGFyZ3MgPSBvcHRpb25fcGFyc2Vy
LnBhcnNlX2FyZ3MoKSAgICAKIAogICAgIGZyb20gd2Via2l0cHkuYmluZGluZ3MubWFpbiBpbXBv
cnQgQmluZGluZ3NUZXN0cwogCi0gICAgcmV0dXJuIEJpbmRpbmdzVGVzdHMob3B0aW9ucy5yZXNl
dF9yZXN1bHRzLCBbICdKUycgXSwgZXhlY3V0aXZlLkV4ZWN1dGl2ZSgpLCBub3Qgb3B0aW9ucy5u
b192ZXJib3NlLCBhcmdzKS5tYWluKCkKKyAgICBpZiBvcHRpb25zLmpzb25fZmlsZV9uYW1lOgor
ICAgICAgICBvcHRpb25zLmpzb25fZmlsZV9uYW1lID0gb3MucGF0aC5hYnNwYXRoKG9wdGlvbnMu
anNvbl9maWxlX25hbWUpCisKKyAgICByZXR1cm4gQmluZGluZ3NUZXN0cyhvcHRpb25zLnJlc2V0
X3Jlc3VsdHMsIFsgJ0pTJyBdLCBleGVjdXRpdmUuRXhlY3V0aXZlKCksIG5vdCBvcHRpb25zLm5v
X3ZlcmJvc2UsIGFyZ3MsIG9wdGlvbnMuanNvbl9maWxlX25hbWUpLm1haW4oKQogCiAKIGlmIF9f
bmFtZV9fID09ICdfX21haW5fXyc6CmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3dlYmtpdHB5
L2JpbmRpbmdzL21haW4ucHkgYi9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2JpbmRpbmdzL21haW4u
cHkKaW5kZXggOGIzMmY1YmNmMDFiNjEwMzAxNzYwZGZkYmJkM2NmYzJlNTkwNDI5ZS4uNmM4MmI4
YTVkNDhkNTdkMjEwZGMzMmJhNTBjOTYxMGU1Y2I3Mzg4MSAxMDA2NDQKLS0tIGEvVG9vbHMvU2Ny
aXB0cy93ZWJraXRweS9iaW5kaW5ncy9tYWluLnB5CisrKyBiL1Rvb2xzL1NjcmlwdHMvd2Via2l0
cHkvYmluZGluZ3MvbWFpbi5weQpAQCAtMjMsNiArMjMsNyBAQAogIwogCiBpbXBvcnQgZm5tYXRj
aAoraW1wb3J0IGpzb24KIGltcG9ydCBvcwogaW1wb3J0IG9zLnBhdGgKIGltcG9ydCBzaHV0aWwK
QEAgLTM1LDEyICszNiwxOCBAQCBmcm9tIHdlYmtpdHB5LmNvbW1vbi5zeXN0ZW0uZXhlY3V0aXZl
IGltcG9ydCBTY3JpcHRFcnJvcgogCiBjbGFzcyBCaW5kaW5nc1Rlc3RzOgogCi0gICAgZGVmIF9f
aW5pdF9fKHNlbGYsIHJlc2V0X3Jlc3VsdHMsIGdlbmVyYXRvcnMsIGV4ZWN1dGl2ZSwgdmVyYm9z
ZSwgcGF0dGVybnMpOgorICAgIGRlZiBfX2luaXRfXyhzZWxmLCByZXNldF9yZXN1bHRzLCBnZW5l
cmF0b3JzLCBleGVjdXRpdmUsIHZlcmJvc2UsIHBhdHRlcm5zLCBqc29uX2ZpbGVfbmFtZSk6CiAg
ICAgICAgIHNlbGYucmVzZXRfcmVzdWx0cyA9IHJlc2V0X3Jlc3VsdHMKICAgICAgICAgc2VsZi5n
ZW5lcmF0b3JzID0gZ2VuZXJhdG9ycwogICAgICAgICBzZWxmLmV4ZWN1dGl2ZSA9IGV4ZWN1dGl2
ZQogICAgICAgICBzZWxmLnZlcmJvc2UgPSB2ZXJib3NlCiAgICAgICAgIHNlbGYucGF0dGVybnMg
PSBwYXR0ZXJucworICAgICAgICBzZWxmLmpzb25fZmlsZV9uYW1lID0ganNvbl9maWxlX25hbWUK
KworICAgICAgICBpZiBzZWxmLmpzb25fZmlsZV9uYW1lOgorICAgICAgICAgICAgc2VsZi5wYXNz
ZXMgPSBbXQorICAgICAgICAgICAgc2VsZi5mYWlsdXJlcyA9IFtdCisgICAgICAgICAgICBzZWxm
LmVycm9ycyA9IFtdCiAKICAgICBkZWYgZ2VuZXJhdGVfZnJvbV9pZGwoc2VsZiwgZ2VuZXJhdG9y
LCBpZGxfZmlsZSwgb3V0cHV0X2RpcmVjdG9yeSwgc3VwcGxlbWVudGFsX2RlcGVuZGVuY3lfZmls
ZSk6CiAgICAgICAgIGNtZCA9IFsncGVybCcsICctdycsCkBAIC0xMDksMTMgKzExNiwyMCBAQCBj
bGFzcyBCaW5kaW5nc1Rlc3RzOgogICAgICAgICAgICAgZXhjZXB0IFNjcmlwdEVycm9yLCBlOgog
ICAgICAgICAgICAgICAgIG91dHB1dCA9IGUub3V0cHV0CiAgICAgICAgICAgICAgICAgZXhpdF9j
b2RlID0gZS5leGl0X2NvZGUKKyAgICAgICAgICAgICAgICBpZiBzZWxmLmpzb25fZmlsZV9uYW1l
OgorICAgICAgICAgICAgICAgICAgICBzZWxmLmVycm9ycy5hcHBlbmQoIiglcykgJXMiICUgKGdl
bmVyYXRvciwgb3V0cHV0X2ZpbGUpKQogCiAgICAgICAgICAgICBpZiBleGl0X2NvZGUgb3Igb3V0
cHV0OgogICAgICAgICAgICAgICAgIHByaW50ICdGQUlMOiAoJXMpICVzJyAlIChnZW5lcmF0b3Is
IG91dHB1dF9maWxlKQogICAgICAgICAgICAgICAgIHByaW50IG91dHB1dAogICAgICAgICAgICAg
ICAgIGNoYW5nZXNfZm91bmQgPSBUcnVlCi0gICAgICAgICAgICBlbGlmIHNlbGYudmVyYm9zZToK
LSAgICAgICAgICAgICAgICBwcmludCAnUEFTUzogKCVzKSAlcycgJSAoZ2VuZXJhdG9yLCBvdXRw
dXRfZmlsZSkKKyAgICAgICAgICAgICAgICBpZiBzZWxmLmpzb25fZmlsZV9uYW1lOgorICAgICAg
ICAgICAgICAgICAgICBzZWxmLmZhaWx1cmVzLmFwcGVuZCgiKCVzKSAlcyIgJSAoZ2VuZXJhdG9y
LCBvdXRwdXRfZmlsZSkpCisgICAgICAgICAgICBlbHNlOgorICAgICAgICAgICAgICAgIGlmIHNl
bGYudmVyYm9zZToKKyAgICAgICAgICAgICAgICAgICAgcHJpbnQgJ1BBU1M6ICglcykgJXMnICUg
KGdlbmVyYXRvciwgb3V0cHV0X2ZpbGUpCisgICAgICAgICAgICAgICAgaWYgc2VsZi5qc29uX2Zp
bGVfbmFtZToKKyAgICAgICAgICAgICAgICAgICAgc2VsZi5wYXNzZXMuYXBwZW5kKCIoJXMpICVz
IiAlIChnZW5lcmF0b3IsIG91dHB1dF9maWxlKSkKICAgICAgICAgcmV0dXJuIGNoYW5nZXNfZm91
bmQKIAogICAgIGRlZiB0ZXN0X21hdGNoZXNfcGF0dGVybnMoc2VsZiwgdGVzdCk6CkBAIC0xODks
NiArMjAzLDE3IEBAIGNsYXNzIEJpbmRpbmdzVGVzdHM6CiAgICAgICAgIG9zLnJlbW92ZSh3aW5k
b3dfY29uc3RydWN0b3JzX2ZpbGUpCiAgICAgICAgIG9zLnJlbW92ZSh3b3JrZXJnbG9iYWxzY29w
ZV9jb25zdHJ1Y3RvcnNfZmlsZSkKICAgICAgICAgb3MucmVtb3ZlKGRlZGljYXRlZHdvcmtlcmds
b2JhbHNjb3BlX2NvbnN0cnVjdG9yc19maWxlKQorCisgICAgICAgIGlmIHNlbGYuanNvbl9maWxl
X25hbWU6CisgICAgICAgICAgICBqc29uX2RhdGEgPSB7CisgICAgICAgICAgICAgICAgJ3Bhc3Nl
cyc6IHNlbGYucGFzc2VzLAorICAgICAgICAgICAgICAgICdmYWlsdXJlcyc6IHNlbGYuZmFpbHVy
ZXMsCisgICAgICAgICAgICAgICAgJ2Vycm9ycyc6IHNlbGYuZXJyb3JzLAorICAgICAgICAgICAg
fQorCisgICAgICAgICAgICB3aXRoIG9wZW4oc2VsZi5qc29uX2ZpbGVfbmFtZSwgJ3cnKSBhcyBq
c29uX2ZpbGU6CisgICAgICAgICAgICAgICAganNvbi5kdW1wKGpzb25fZGF0YSwganNvbl9maWxl
KQorCiAgICAgICAgIHByaW50ICcnCiAgICAgICAgIGlmIGFsbF90ZXN0c19wYXNzZWQ6CiAgICAg
ICAgICAgICBwcmludCAnQWxsIHRlc3RzIFBBU1MhJwo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>302581</attachid>
            <date>2017-02-23 14:43:01 -0800</date>
            <delta_ts>2017-02-27 15:09:18 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-168626-20170223144011.patch</filename>
            <type>text/plain</type>
            <size>4723</size>
            <attacher name="Srinivasan Vijayaraghavan">webkit</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjEyOTIzCmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggYjg5NmRmNTlhNTc4MWU2NzdkYjM2NzRmYmZkY2UwNDlj
YzcyN2NkZS4uNzkwZjQ4Nzk2MzVlZDRlYjhmYTYzMGFiZmQ4MTU5ZDI2ZmJiM2VjNSAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3
IEBACisyMDE3LTAyLTIzICBTcmluaXZhc2FuIFZpamF5YXJhZ2hhdmFuICA8c3ZpamF5YXJhZ2hh
dmFuQGFwcGxlLmNvbT4KKworICAgICAgICBBZGQgbWFjaGluZS1yZWFkYWJsZSByZXN1bHRzIGZv
ciBiaW5kaW5ncyB0ZXN0cworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MTY4NjI2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgKiBTY3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0czoKKyAgICAgICAgKG1haW4pOiBB
ZGQgb3B0aW9uYWwgLS1qc29uLW91dHB1dCBjb21tYW5kLWxpbmUgcGFyYW1ldGVyLgorICAgICAg
ICAqIFNjcmlwdHMvd2Via2l0cHkvYmluZGluZ3MvbWFpbi5weToKKyAgICAgICAgKEJpbmRpbmdz
VGVzdHMuX19pbml0X18pOiBBZGQgdmFyaWFibGVzIHRvIHN0b3JlIG1hY2hpbmUtcmVhZGFibGUg
cmVzdWx0cy4KKyAgICAgICAgKEJpbmRpbmdzVGVzdHMuZGV0ZWN0X2NoYW5nZXMpOiBTdG9yZSBy
ZXN1bHRzIGluIG1hY2hpbmUtcmVhZGFibGUgZm9ybSBpZiBhcHBsaWNhYmxlLgorICAgICAgICAo
QmluZGluZ3NUZXN0cy5tYWluKTogV3JpdGUgZGF0YSB0byBKU09OIGZpbGUgaWYgYXBwbGljYWJs
ZS4KKwogMjAxNy0wMi0yMyAgU2FhbSBCYXJhdGkgIDxzYmFyYXRpQGFwcGxlLmNvbT4KIAogICAg
ICAgICBTcGVjdWxhdGl2ZSBmaXggZm9yIGpzYyB0aW1pbmcgb3V0IHRlc3RzCmRpZmYgLS1naXQg
YS9Ub29scy9TY3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0cyBiL1Rvb2xzL1NjcmlwdHMvcnVuLWJp
bmRpbmdzLXRlc3RzCmluZGV4IDlmN2MxYTFlOTMwMzBkODQ1NjY0ODAwNTQ4NTNiZDFiNDFkNTEy
MDAuLmMwOGI2M2NmYmZiNDdhM2I2YzJkMjQ3MTRmYjVlZGFiYTY3ZGQ0YmUgMTAwNzU1Ci0tLSBh
L1Rvb2xzL1NjcmlwdHMvcnVuLWJpbmRpbmdzLXRlc3RzCisrKyBiL1Rvb2xzL1NjcmlwdHMvcnVu
LWJpbmRpbmdzLXRlc3RzCkBAIC0yOSw2ICsyOSw3IEBACiAjIHBhdGNoLiBUaGlzIG1ha2VzIGl0
IGVhc2llciB0byB0cmFjayBhbmQgcmV2aWV3IGNoYW5nZXMgaW4gZ2VuZXJhdGVkIGNvZGUuCiAK
IGltcG9ydCBvcHRwYXJzZQoraW1wb3J0IG9zCiBpbXBvcnQgc3lzCiBmcm9tIHdlYmtpdHB5LmNv
bW1vbi5zeXN0ZW0gaW1wb3J0IGV4ZWN1dGl2ZQogCkBAIC00MywxMSArNDQsMTYgQEAgZGVmIG1h
aW4oYXJndik6CiAgICAgICAgIGhlbHA9J0Rpc2FibGUgdmVyb2JzZSBvdXRwdXQnKQogICAgIG9w
dGlvbl9wYXJzZXIuYWRkX29wdGlvbignLS1yZXNldC1yZXN1bHRzJywgYWN0aW9uPSdzdG9yZV90
cnVlJywgZGVmYXVsdD1GYWxzZSwKICAgICAgICAgaGVscD0nT3ZlcndyaXRlcyB0aGUgcmVmZXJl
bmNlIGZpbGVzIHdpdGggdGhlIGdlbmVyYXRlZCByZXN1bHRzJykKKyAgICBvcHRpb25fcGFyc2Vy
LmFkZF9vcHRpb24oJy0tanNvbi1vdXRwdXQnLCBhY3Rpb249J3N0b3JlJywgdHlwZT0nc3RyaW5n
JywgZGVzdD0nanNvbl9maWxlX25hbWUnLAorICAgICAgICBoZWxwPSdDcmVhdGUgYSBmaWxlIGF0
IHNwZWNpZmllZCBwYXRoLCBsaXN0aW5nIHRlc3QgcmVzdWx0cyBpbiBKU09OIGZvcm1hdC4nKQog
ICAgIG9wdGlvbnMsIGFyZ3MgPSBvcHRpb25fcGFyc2VyLnBhcnNlX2FyZ3MoKSAgICAKIAogICAg
IGZyb20gd2Via2l0cHkuYmluZGluZ3MubWFpbiBpbXBvcnQgQmluZGluZ3NUZXN0cwogCi0gICAg
cmV0dXJuIEJpbmRpbmdzVGVzdHMob3B0aW9ucy5yZXNldF9yZXN1bHRzLCBbICdKUycgXSwgZXhl
Y3V0aXZlLkV4ZWN1dGl2ZSgpLCBub3Qgb3B0aW9ucy5ub192ZXJib3NlLCBhcmdzKS5tYWluKCkK
KyAgICBpZiBvcHRpb25zLmpzb25fZmlsZV9uYW1lOgorICAgICAgICBvcHRpb25zLmpzb25fZmls
ZV9uYW1lID0gb3MucGF0aC5hYnNwYXRoKG9wdGlvbnMuanNvbl9maWxlX25hbWUpCisKKyAgICBy
ZXR1cm4gQmluZGluZ3NUZXN0cyhvcHRpb25zLnJlc2V0X3Jlc3VsdHMsIFsgJ0pTJyBdLCBleGVj
dXRpdmUuRXhlY3V0aXZlKCksIG5vdCBvcHRpb25zLm5vX3ZlcmJvc2UsIGFyZ3MsIG9wdGlvbnMu
anNvbl9maWxlX25hbWUpLm1haW4oKQogCiAKIGlmIF9fbmFtZV9fID09ICdfX21haW5fXyc6CmRp
ZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2JpbmRpbmdzL21haW4ucHkgYi9Ub29s
cy9TY3JpcHRzL3dlYmtpdHB5L2JpbmRpbmdzL21haW4ucHkKaW5kZXggOGIzMmY1YmNmMDFiNjEw
MzAxNzYwZGZkYmJkM2NmYzJlNTkwNDI5ZS4uMDgxYWQyN2NiYzIxNjE0MTc4NzQwOTc0MDBjOWI5
ZTk4NmE3NTQzNSAxMDA2NDQKLS0tIGEvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9iaW5kaW5ncy9t
YWluLnB5CisrKyBiL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvYmluZGluZ3MvbWFpbi5weQpAQCAt
MjMsNiArMjMsNyBAQAogIwogCiBpbXBvcnQgZm5tYXRjaAoraW1wb3J0IGpzb24KIGltcG9ydCBv
cwogaW1wb3J0IG9zLnBhdGgKIGltcG9ydCBzaHV0aWwKQEAgLTM1LDEyICszNiwxNyBAQCBmcm9t
IHdlYmtpdHB5LmNvbW1vbi5zeXN0ZW0uZXhlY3V0aXZlIGltcG9ydCBTY3JpcHRFcnJvcgogCiBj
bGFzcyBCaW5kaW5nc1Rlc3RzOgogCi0gICAgZGVmIF9faW5pdF9fKHNlbGYsIHJlc2V0X3Jlc3Vs
dHMsIGdlbmVyYXRvcnMsIGV4ZWN1dGl2ZSwgdmVyYm9zZSwgcGF0dGVybnMpOgorICAgIGRlZiBf
X2luaXRfXyhzZWxmLCByZXNldF9yZXN1bHRzLCBnZW5lcmF0b3JzLCBleGVjdXRpdmUsIHZlcmJv
c2UsIHBhdHRlcm5zLCBqc29uX2ZpbGVfbmFtZSk6CiAgICAgICAgIHNlbGYucmVzZXRfcmVzdWx0
cyA9IHJlc2V0X3Jlc3VsdHMKICAgICAgICAgc2VsZi5nZW5lcmF0b3JzID0gZ2VuZXJhdG9ycwog
ICAgICAgICBzZWxmLmV4ZWN1dGl2ZSA9IGV4ZWN1dGl2ZQogICAgICAgICBzZWxmLnZlcmJvc2Ug
PSB2ZXJib3NlCiAgICAgICAgIHNlbGYucGF0dGVybnMgPSBwYXR0ZXJucworICAgICAgICBzZWxm
Lmpzb25fZmlsZV9uYW1lID0ganNvbl9maWxlX25hbWUKKworICAgICAgICBpZiBzZWxmLmpzb25f
ZmlsZV9uYW1lOgorICAgICAgICAgICAgc2VsZi5mYWlsdXJlcyA9IFtdCisgICAgICAgICAgICBz
ZWxmLmVycm9ycyA9IFtdCiAKICAgICBkZWYgZ2VuZXJhdGVfZnJvbV9pZGwoc2VsZiwgZ2VuZXJh
dG9yLCBpZGxfZmlsZSwgb3V0cHV0X2RpcmVjdG9yeSwgc3VwcGxlbWVudGFsX2RlcGVuZGVuY3lf
ZmlsZSk6CiAgICAgICAgIGNtZCA9IFsncGVybCcsICctdycsCkBAIC0xMDksMTEgKzExNSwxNSBA
QCBjbGFzcyBCaW5kaW5nc1Rlc3RzOgogICAgICAgICAgICAgZXhjZXB0IFNjcmlwdEVycm9yLCBl
OgogICAgICAgICAgICAgICAgIG91dHB1dCA9IGUub3V0cHV0CiAgICAgICAgICAgICAgICAgZXhp
dF9jb2RlID0gZS5leGl0X2NvZGUKKyAgICAgICAgICAgICAgICBpZiBzZWxmLmpzb25fZmlsZV9u
YW1lOgorICAgICAgICAgICAgICAgICAgICBzZWxmLmVycm9ycy5hcHBlbmQoIiglcykgJXMiICUg
KGdlbmVyYXRvciwgb3V0cHV0X2ZpbGUpKQogCiAgICAgICAgICAgICBpZiBleGl0X2NvZGUgb3Ig
b3V0cHV0OgogICAgICAgICAgICAgICAgIHByaW50ICdGQUlMOiAoJXMpICVzJyAlIChnZW5lcmF0
b3IsIG91dHB1dF9maWxlKQogICAgICAgICAgICAgICAgIHByaW50IG91dHB1dAogICAgICAgICAg
ICAgICAgIGNoYW5nZXNfZm91bmQgPSBUcnVlCisgICAgICAgICAgICAgICAgaWYgc2VsZi5qc29u
X2ZpbGVfbmFtZToKKyAgICAgICAgICAgICAgICAgICAgc2VsZi5mYWlsdXJlcy5hcHBlbmQoIigl
cykgJXMiICUgKGdlbmVyYXRvciwgb3V0cHV0X2ZpbGUpKQogICAgICAgICAgICAgZWxpZiBzZWxm
LnZlcmJvc2U6CiAgICAgICAgICAgICAgICAgcHJpbnQgJ1BBU1M6ICglcykgJXMnICUgKGdlbmVy
YXRvciwgb3V0cHV0X2ZpbGUpCiAgICAgICAgIHJldHVybiBjaGFuZ2VzX2ZvdW5kCkBAIC0xODks
NiArMTk5LDE2IEBAIGNsYXNzIEJpbmRpbmdzVGVzdHM6CiAgICAgICAgIG9zLnJlbW92ZSh3aW5k
b3dfY29uc3RydWN0b3JzX2ZpbGUpCiAgICAgICAgIG9zLnJlbW92ZSh3b3JrZXJnbG9iYWxzY29w
ZV9jb25zdHJ1Y3RvcnNfZmlsZSkKICAgICAgICAgb3MucmVtb3ZlKGRlZGljYXRlZHdvcmtlcmds
b2JhbHNjb3BlX2NvbnN0cnVjdG9yc19maWxlKQorCisgICAgICAgIGlmIHNlbGYuanNvbl9maWxl
X25hbWU6CisgICAgICAgICAgICBqc29uX2RhdGEgPSB7CisgICAgICAgICAgICAgICAgJ2ZhaWx1
cmVzJzogc2VsZi5mYWlsdXJlcywKKyAgICAgICAgICAgICAgICAnZXJyb3JzJzogc2VsZi5lcnJv
cnMsCisgICAgICAgICAgICB9CisKKyAgICAgICAgICAgIHdpdGggb3BlbihzZWxmLmpzb25fZmls
ZV9uYW1lLCAndycpIGFzIGpzb25fZmlsZToKKyAgICAgICAgICAgICAgICBqc29uLmR1bXAoanNv
bl9kYXRhLCBqc29uX2ZpbGUpCisKICAgICAgICAgcHJpbnQgJycKICAgICAgICAgaWYgYWxsX3Rl
c3RzX3Bhc3NlZDoKICAgICAgICAgICAgIHByaW50ICdBbGwgdGVzdHMgUEFTUyEnCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>