<?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>122303</bug_id>
          
          <creation_ts>2013-10-03 15:11:41 -0700</creation_ts>
          <short_desc>Add a verbose flag to binding tests</short_desc>
          <delta_ts>2014-06-02 23:52:04 -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>Tools / Tests</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></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P4</priority>
          <bug_severity>Enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Matthew Hanson">matthew_hanson</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>dpranke</cc>
    
    <cc>glenn</cc>
    
    <cc>lforschler</cc>
    
    <cc>ossy</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>936001</commentid>
    <comment_count>0</comment_count>
    <who name="Matthew Hanson">matthew_hanson</who>
    <bug_when>2013-10-03 15:11:41 -0700</bug_when>
    <thetext>While investigating Bug 122017, I found that positive BindingTests results are unconditionally printed to stdout.  It is often desirable to suppress the output of these test cases since they are run often and expected to fail rarely.

The BindingTests class is defined in Tools/Scripts/webkitpy/bindings/main.py.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>936002</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2013-10-03 15:12:04 -0700</bug_when>
    <thetext>&lt;rdar://problem/15146102&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>936007</commentid>
    <comment_count>2</comment_count>
      <attachid>213305</attachid>
    <who name="Matthew Hanson">matthew_hanson</who>
    <bug_when>2013-10-03 15:36:33 -0700</bug_when>
    <thetext>Created attachment 213305
v1 Patch for 122303

Added a verbose keyword argument to the BindingsTests constructor.
Verbose mode, which is on by default, prints positive results to stdout as before.
When the verbose flag is set to false, positive results are not printed at all.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>941153</commentid>
    <comment_count>3</comment_count>
      <attachid>213305</attachid>
    <who name="Peter Gal">galpeter</who>
    <bug_when>2013-10-18 05:13:41 -0700</bug_when>
    <thetext>Comment on attachment 213305
v1 Patch for 122303

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

The &apos;run-bindings-test&apos; script should be adapted for this change. Without that there is no way to disable it from command line.

&gt; Tools/Scripts/webkitpy/bindings/main.py:117
&gt; +                if self.verbose is True:

A simple &apos;if self.verbose&apos; is enough. Also &apos;is&apos; is for identity checking see http://docs.python.org/2/library/operator.html#operator.is_</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>957317</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2013-12-06 11:25:59 -0800</bug_when>
    <thetext>&lt;rdar://problem/15605926
&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>960591</commentid>
    <comment_count>5</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2013-12-17 11:24:20 -0800</bug_when>
    <thetext>&lt;rdar://problem/15680835
&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>977168</commentid>
    <comment_count>6</comment_count>
      <attachid>213305</attachid>
    <who name="Andreas Kling">kling</who>
    <bug_when>2014-02-05 17:38:02 -0800</bug_when>
    <thetext>Comment on attachment 213305
v1 Patch for 122303

r-, please fix the problems pointed out by Peter Gal.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1012413</commentid>
    <comment_count>7</comment_count>
      <attachid>232246</attachid>
    <who name="Tibor Mészáros">mtiborinf</who>
    <bug_when>2014-05-29 07:08:46 -0700</bug_when>
    <thetext>Created attachment 232246
Patch v2

I had finished Matthew&apos;s patch. The following changes where made:
- Added --no-verbose option to the run-bindigs-test.
- Changed &quot;if self.verbose is True&quot; to &quot;self.verbose&quot; as Peter Gal wanted.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1012433</commentid>
    <comment_count>8</comment_count>
      <attachid>232246</attachid>
    <who name="Peter Gal">galpeter</who>
    <bug_when>2014-05-29 09:02:15 -0700</bug_when>
    <thetext>Comment on attachment 232246
Patch v2

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

&gt; Tools/Scripts/run-bindings-tests:47
&gt; +    if &quot;--no-verbose&quot; in argv:
&gt; +        verbose = False
&gt; +    else:
&gt; +        verbose = True

We should make this a bit simpler a one-liner, like in the line 42. Eg.: verbose = not &quot;--no-verbose&quot; in argv

&gt; Tools/Scripts/run-bindings-tests:52
&gt; +        &apos;GObject&apos;,

I don&apos;t see any point of this change, why is this needed?

&gt; Tools/Scripts/webkitpy/bindings/main.py:117
&gt;              else:
&gt; -                print &apos;PASS: (%s) %s&apos; % (generator, output_file)
&gt; +                if self.verbose:

Looking at this know I realize we can merge the &apos;else&apos; and &apos;if self.verbose&apos; into an &apos;elsif self.verbose&apos; :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1012435</commentid>
    <comment_count>9</comment_count>
      <attachid>232246</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2014-05-29 09:03:55 -0700</bug_when>
    <thetext>Comment on attachment 232246
Patch v2

r-, please fix the problems pointed out by Péter Gál.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1012599</commentid>
    <comment_count>10</comment_count>
      <attachid>232292</attachid>
    <who name="Tibor Mészáros">mtiborinf</who>
    <bug_when>2014-05-30 05:13:50 -0700</bug_when>
    <thetext>Created attachment 232292
Patch v3

All has been fixed, that Peter mentioned :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1012600</commentid>
    <comment_count>11</comment_count>
      <attachid>232292</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2014-05-30 05:47:09 -0700</bug_when>
    <thetext>Comment on attachment 232292
Patch v3

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

&gt; Tools/Scripts/run-bindings-tests:43
&gt; +    

This extra newline is unnecessary.

&gt; Tools/Scripts/webkitpy/bindings/main.py:117
&gt; +            elif self.verbose:
&gt; +                    print &apos;PASS: (%s) %s&apos; % (generator, output_file)

Please remove this extra indentation.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1012927</commentid>
    <comment_count>12</comment_count>
      <attachid>232375</attachid>
    <who name="Tibor Mészáros">mtiborinf</who>
    <bug_when>2014-06-02 04:14:50 -0700</bug_when>
    <thetext>Created attachment 232375
Patch v4

The newline and the extra indentation has been removed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1013057</commentid>
    <comment_count>13</comment_count>
      <attachid>232375</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-06-02 23:52:00 -0700</bug_when>
    <thetext>Comment on attachment 232375
Patch v4

Clearing flags on attachment: 232375

Committed r169558: &lt;http://trac.webkit.org/changeset/169558&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1013058</commentid>
    <comment_count>14</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-06-02 23:52:04 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>213305</attachid>
            <date>2013-10-03 15:36:33 -0700</date>
            <delta_ts>2014-05-30 05:47:26 -0700</delta_ts>
            <desc>v1 Patch for 122303</desc>
            <filename>0001-Added-a-verbose-keyword-argument-to-the-BindingsTest.patch</filename>
            <type>text/plain</type>
            <size>2642</size>
            <attacher name="Matthew Hanson">matthew_hanson</attacher>
            
              <data encoding="base64">RnJvbSA1NWMxMTViODI2MmJkM2JiOGI4MTBiMjllNWI2YjU4MmJiZTBhODNmIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBNYXR0aGV3IEhhbnNvbiA8bWF0dGhld19oYW5zb25AYXBwbGUu
Y29tPgpEYXRlOiBUaHUsIDMgT2N0IDIwMTMgMTU6MzE6NDAgLTA3MDAKU3ViamVjdDogW1BBVENI
XSBBZGRlZCBhIHZlcmJvc2Uga2V5d29yZCBhcmd1bWVudCB0byB0aGUgQmluZGluZ3NUZXN0cwog
Y29uc3RydWN0b3IuIFZlcmJvc2UgbW9kZSwgd2hpY2ggaXMgb24gYnkgZGVmYXVsdCwgcHJpbnRz
IHBvc2l0aXZlIHJlc3VsdHMgdG8KIHN0ZG91dCBhcyBiZWZvcmUuIFdoZW4gdGhlIHZlcmJvc2Ug
ZmxhZyBpcyBzZXQgdG8gZmFsc2UsIHBvc2l0aXZlIHJlc3VsdHMgYXJlCiBub3QgcHJpbnRlZCBh
dCBhbGwuCgotLS0KIFRvb2xzL0NoYW5nZUxvZyAgICAgICAgICAgICAgICAgICAgICAgICB8IDE1
ICsrKysrKysrKysrKysrKwogVG9vbHMvU2NyaXB0cy93ZWJraXRweS9iaW5kaW5ncy9tYWluLnB5
IHwgIDYgKysrKy0tCiAyIGZpbGVzIGNoYW5nZWQsIDE5IGluc2VydGlvbnMoKyksIDIgZGVsZXRp
b25zKC0pCgpkaWZmIC0tZ2l0IGEvVG9vbHMvQ2hhbmdlTG9nIGIvVG9vbHMvQ2hhbmdlTG9nCmlu
ZGV4IGJmOTVkNDcuLjViYTI0MjEgMTAwNjQ0Ci0tLSBhL1Rvb2xzL0NoYW5nZUxvZworKysgYi9U
b29scy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxOCBAQAorMjAxMy0xMC0wMyAgTWF0dGhldyBIYW5z
b24gIDxtYXR0aGV3X2hhbnNvbkBhcHBsZS5jb20+CisKKyAgICAgICAgQWRkIGEgdmVyYm9zZSBm
bGFnIHRvIGJpbmRpbmcgdGVzdHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTEyMjMwMworCisgICAgICAgIEFkZGVkIGEgdmVyYm9zZSBrZXl3b3JkIGFy
Z3VtZW50IHRvIHRoZSBCaW5kaW5nc1Rlc3RzIGNvbnN0cnVjdG9yLgorICAgICAgICBWZXJib3Nl
IG1vZGUsIHdoaWNoIGlzIG9uIGJ5IGRlZmF1bHQsIHByaW50cyBwb3NpdGl2ZSByZXN1bHRzIHRv
IHN0ZG91dCBhcyBiZWZvcmUuCisgICAgICAgIFdoZW4gdGhlIHZlcmJvc2UgZmxhZyBpcyBzZXQg
dG8gZmFsc2UsIHBvc2l0aXZlIHJlc3VsdHMgYXJlIG5vdCBwcmludGVkIGF0IGFsbC4KKworICAg
ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIFNjcmlwdHMvd2Vi
a2l0cHkvYmluZGluZ3MvbWFpbi5weToKKyAgICAgICAgKEJpbmRpbmdzVGVzdHMuX19pbml0X18p
OgorICAgICAgICAoQmluZGluZ3NUZXN0cy5kZXRlY3RfY2hhbmdlcyk6CisKIDIwMTMtMTAtMDMg
IEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEBhcHBsZS5jb20+CiAKICAgICAgICAgRml4IExlYWtz
Vmlld2VyIHRvIGxvYWQgbGVha3MgcmVzdWx0cyBmcm9tIE1vdW50YWluTGlvbiwgbm90IExpb24u
CmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2JpbmRpbmdzL21haW4ucHkgYi9U
b29scy9TY3JpcHRzL3dlYmtpdHB5L2JpbmRpbmdzL21haW4ucHkKaW5kZXggOWM4MmI5YS4uMzY1
MjI3NyAxMDA2NDQKLS0tIGEvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9iaW5kaW5ncy9tYWluLnB5
CisrKyBiL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvYmluZGluZ3MvbWFpbi5weQpAQCAtMzQsMTAg
KzM0LDExIEBAIGZyb20gd2Via2l0cHkuY29tbW9uLnN5c3RlbS5leGVjdXRpdmUgaW1wb3J0IFNj
cmlwdEVycm9yCiAKIGNsYXNzIEJpbmRpbmdzVGVzdHM6CiAKLSAgICBkZWYgX19pbml0X18oc2Vs
ZiwgcmVzZXRfcmVzdWx0cywgZ2VuZXJhdG9ycywgZXhlY3V0aXZlKToKKyAgICBkZWYgX19pbml0
X18oc2VsZiwgcmVzZXRfcmVzdWx0cywgZ2VuZXJhdG9ycywgZXhlY3V0aXZlLCB2ZXJib3NlPVRy
dWUpOgogICAgICAgICBzZWxmLnJlc2V0X3Jlc3VsdHMgPSByZXNldF9yZXN1bHRzCiAgICAgICAg
IHNlbGYuZ2VuZXJhdG9ycyA9IGdlbmVyYXRvcnMKICAgICAgICAgc2VsZi5leGVjdXRpdmUgPSBl
eGVjdXRpdmUKKyAgICAgICAgc2VsZi52ZXJib3NlID0gdmVyYm9zZQogCiAgICAgZGVmIGdlbmVy
YXRlX2Zyb21faWRsKHNlbGYsIGdlbmVyYXRvciwgaWRsX2ZpbGUsIG91dHB1dF9kaXJlY3Rvcnks
IHN1cHBsZW1lbnRhbF9kZXBlbmRlbmN5X2ZpbGUpOgogICAgICAgICBjbWQgPSBbJ3BlcmwnLCAn
LXcnLApAQCAtMTEzLDcgKzExNCw4IEBAIGNsYXNzIEJpbmRpbmdzVGVzdHM6CiAgICAgICAgICAg
ICAgICAgcHJpbnQgb3V0cHV0CiAgICAgICAgICAgICAgICAgY2hhbmdlc19mb3VuZCA9IFRydWUK
ICAgICAgICAgICAgIGVsc2U6Ci0gICAgICAgICAgICAgICAgcHJpbnQgJ1BBU1M6ICglcykgJXMn
ICUgKGdlbmVyYXRvciwgb3V0cHV0X2ZpbGUpCisgICAgICAgICAgICAgICAgaWYgc2VsZi52ZXJi
b3NlIGlzIFRydWU6CisgICAgICAgICAgICAgICAgICAgIHByaW50ICdQQVNTOiAoJXMpICVzJyAl
IChnZW5lcmF0b3IsIG91dHB1dF9maWxlKQogICAgICAgICByZXR1cm4gY2hhbmdlc19mb3VuZAog
CiAgICAgZGVmIHJ1bl90ZXN0cyhzZWxmLCBnZW5lcmF0b3IsIGlucHV0X2RpcmVjdG9yeSwgcmVm
ZXJlbmNlX2RpcmVjdG9yeSwgc3VwcGxlbWVudGFsX2RlcGVuZGVuY3lfZmlsZSk6Ci0tIAoxLjgu
My40IChBcHBsZSBHaXQtNDcpCgo=
</data>
<flag name="review"
          id="235650"
          type_id="1"
          status="-"
          setter="kling"
    />
    <flag name="commit-queue"
          id="235651"
          type_id="3"
          status="-"
          setter="kling"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>232246</attachid>
            <date>2014-05-29 07:08:46 -0700</date>
            <delta_ts>2014-05-30 05:13:50 -0700</delta_ts>
            <desc>Patch v2</desc>
            <filename>verbose.patch</filename>
            <type>text/plain</type>
            <size>0</size>
            <attacher name="Tibor Mészáros">mtiborinf</attacher>
            
              <data encoding="base64"></data>
<flag name="review"
          id="256736"
          type_id="1"
          status="-"
          setter="ossy"
    />
    <flag name="commit-queue"
          id="256737"
          type_id="3"
          status="-"
          setter="ossy"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>232292</attachid>
            <date>2014-05-30 05:13:50 -0700</date>
            <delta_ts>2014-06-02 04:14:50 -0700</delta_ts>
            <desc>Patch v3</desc>
            <filename>verbose.patch</filename>
            <type>text/plain</type>
            <size>3006</size>
            <attacher name="Tibor Mészáros">mtiborinf</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA2Mzhj
ODhiLi40OTNiYWIyIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMjQgQEAKKzIwMTQtMDUtMzAgIFRpYm9yIE1lc3phcm9zICA8dG1l
c3phcm9zLnUtc3plZ2VkQHBhcnRuZXIuc2Ftc3VuZy5jb20+CisKKyAgICAgICAgQWRkIGEgdmVy
Ym9zZSBmbGFnIHRvIGJpbmRpbmcgdGVzdHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTEyMjMwMworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo
T09QUyEpLgorCisgICAgICAgIFRoaXMgcGF0Y2ggd2lsbDoKKyAgICAgICAgLSBBZGQgYSB2ZXJi
b3NlIGtleXdvcmQgYXJndW1lbnQgdG8gdGhlIEJpbmRpbmdzVGVzdHMgY29uc3RydWN0b3IsCisg
ICAgICAgIC0gQWRkIC0tbm8tdmVyYm9zZSBvcHRpb24gdG8gdGhlIHJ1bi1iaW5kaWdzLXRlc3Qu
CisgICAgICAgIFZlcmJvc2UgbW9kZSwgd2hpY2ggaXMgb24gYnkgZGVmYXVsdCwgcHJpbnRzIHBv
c2l0aXZlIHJlc3VsdHMgdG8gc3Rkb3V0IGFzIGJlZm9yZS4KKyAgICAgICAgV2hlbiB0aGUgLS1u
by12ZXJib3NlIG9wdGlvbiBhZGRlZCB0byBydW4tYmluZGluZ3MtdGVzdHMsIHBvc2l0aXZlIHJl
c3VsdHMgYXJlIG5vdCBwcmludGVkIGF0IGFsbC4KKworICAgICAgICBPcmlnaW5hbCBwYXRjaCBm
cm9tIE1hdHRoZXcgSGFuc29uIDxtYXR0aGV3X2hhbnNvbkBhcHBsZS5jb20+IAorICAgICAgICAK
KyAgICAgICAgKiBTY3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0czoKKyAgICAgICAgKG1haW4pOgor
ICAgICAgICAqIFNjcmlwdHMvd2Via2l0cHkvYmluZGluZ3MvbWFpbi5weToKKyAgICAgICAgKEJp
bmRpbmdzVGVzdHMuX19pbml0X18pOgorICAgICAgICAoQmluZGluZ3NUZXN0cy5kZXRlY3RfY2hh
bmdlcyk6CisKIDIwMTQtMDUtMzAgIEphcmVrIEN6ZWthbHNraSAgPGphcmVrY3pla0Bwb2N6dGEu
b25ldC5wbD4KIAogICAgICAgICBbQVRLXSBBZGRlZCBuZXcgYWNjZXNzaWJpbGl0eSBzaWduYWw6
IHRleHQtY2FyZXQtbW92ZWQuCmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3J1bi1iaW5kaW5n
cy10ZXN0cyBiL1Rvb2xzL1NjcmlwdHMvcnVuLWJpbmRpbmdzLXRlc3RzCmluZGV4IGU3MDgzOWQu
LmRiNjY4MjAgMTAwNzU1Ci0tLSBhL1Rvb2xzL1NjcmlwdHMvcnVuLWJpbmRpbmdzLXRlc3RzCisr
KyBiL1Rvb2xzL1NjcmlwdHMvcnVuLWJpbmRpbmdzLXRlc3RzCkBAIC00MCw2ICs0MCw4IEBAIGRl
ZiBtYWluKGFyZ3YpOgogCiAgICAgIiIiCiAgICAgcmVzZXRfcmVzdWx0cyA9ICItLXJlc2V0LXJl
c3VsdHMiIGluIGFyZ3YKKyAgICAKKyAgICB2ZXJib3NlID0gbm90ICItLW5vLXZlcmJvc2UiIGlu
IGFyZ3YKIAogICAgIGdlbmVyYXRvcnMgPSBbCiAgICAgICAgICdKUycsCkBAIC00OSw3ICs1MSw3
IEBAIGRlZiBtYWluKGFyZ3YpOgogCiAgICAgZnJvbSB3ZWJraXRweS5iaW5kaW5ncy5tYWluIGlt
cG9ydCBCaW5kaW5nc1Rlc3RzCiAKLSAgICByZXR1cm4gQmluZGluZ3NUZXN0cyhyZXNldF9yZXN1
bHRzLCBnZW5lcmF0b3JzLCBleGVjdXRpdmUuRXhlY3V0aXZlKCkpLm1haW4oKQorICAgIHJldHVy
biBCaW5kaW5nc1Rlc3RzKHJlc2V0X3Jlc3VsdHMsIGdlbmVyYXRvcnMsIGV4ZWN1dGl2ZS5FeGVj
dXRpdmUoKSwgdmVyYm9zZSkubWFpbigpCiAKIAogaWYgX19uYW1lX18gPT0gJ19fbWFpbl9fJzoK
ZGlmZiAtLWdpdCBhL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvYmluZGluZ3MvbWFpbi5weSBiL1Rv
b2xzL1NjcmlwdHMvd2Via2l0cHkvYmluZGluZ3MvbWFpbi5weQppbmRleCA5NGI4MjVjLi4zNGY4
ZjQwIDEwMDY0NAotLS0gYS9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2JpbmRpbmdzL21haW4ucHkK
KysrIGIvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9iaW5kaW5ncy9tYWluLnB5CkBAIC0zNCwxMCAr
MzQsMTEgQEAgZnJvbSB3ZWJraXRweS5jb21tb24uc3lzdGVtLmV4ZWN1dGl2ZSBpbXBvcnQgU2Ny
aXB0RXJyb3IKIAogY2xhc3MgQmluZGluZ3NUZXN0czoKIAotICAgIGRlZiBfX2luaXRfXyhzZWxm
LCByZXNldF9yZXN1bHRzLCBnZW5lcmF0b3JzLCBleGVjdXRpdmUpOgorICAgIGRlZiBfX2luaXRf
XyhzZWxmLCByZXNldF9yZXN1bHRzLCBnZW5lcmF0b3JzLCBleGVjdXRpdmUsIHZlcmJvc2UpOgog
ICAgICAgICBzZWxmLnJlc2V0X3Jlc3VsdHMgPSByZXNldF9yZXN1bHRzCiAgICAgICAgIHNlbGYu
Z2VuZXJhdG9ycyA9IGdlbmVyYXRvcnMKICAgICAgICAgc2VsZi5leGVjdXRpdmUgPSBleGVjdXRp
dmUKKyAgICAgICAgc2VsZi52ZXJib3NlID0gdmVyYm9zZQogCiAgICAgZGVmIGdlbmVyYXRlX2Zy
b21faWRsKHNlbGYsIGdlbmVyYXRvciwgaWRsX2ZpbGUsIG91dHB1dF9kaXJlY3RvcnksIHN1cHBs
ZW1lbnRhbF9kZXBlbmRlbmN5X2ZpbGUpOgogICAgICAgICBjbWQgPSBbJ3BlcmwnLCAnLXcnLApA
QCAtMTEyLDggKzExMyw4IEBAIGNsYXNzIEJpbmRpbmdzVGVzdHM6CiAgICAgICAgICAgICAgICAg
cHJpbnQgJ0ZBSUw6ICglcykgJXMnICUgKGdlbmVyYXRvciwgb3V0cHV0X2ZpbGUpCiAgICAgICAg
ICAgICAgICAgcHJpbnQgb3V0cHV0CiAgICAgICAgICAgICAgICAgY2hhbmdlc19mb3VuZCA9IFRy
dWUKLSAgICAgICAgICAgIGVsc2U6Ci0gICAgICAgICAgICAgICAgcHJpbnQgJ1BBU1M6ICglcykg
JXMnICUgKGdlbmVyYXRvciwgb3V0cHV0X2ZpbGUpCisgICAgICAgICAgICBlbGlmIHNlbGYudmVy
Ym9zZToKKyAgICAgICAgICAgICAgICAgICAgcHJpbnQgJ1BBU1M6ICglcykgJXMnICUgKGdlbmVy
YXRvciwgb3V0cHV0X2ZpbGUpCiAgICAgICAgIHJldHVybiBjaGFuZ2VzX2ZvdW5kCiAKICAgICBk
ZWYgcnVuX3Rlc3RzKHNlbGYsIGdlbmVyYXRvciwgaW5wdXRfZGlyZWN0b3J5LCByZWZlcmVuY2Vf
ZGlyZWN0b3J5LCBzdXBwbGVtZW50YWxfZGVwZW5kZW5jeV9maWxlKToK
</data>
<flag name="review"
          id="256786"
          type_id="1"
          status="-"
          setter="ossy"
    />
    <flag name="commit-queue"
          id="256787"
          type_id="3"
          status="-"
          setter="ossy"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>232375</attachid>
            <date>2014-06-02 04:14:50 -0700</date>
            <delta_ts>2014-06-02 23:51:59 -0700</delta_ts>
            <desc>Patch v4</desc>
            <filename>verbose_V2.patch</filename>
            <type>text/plain</type>
            <size>2808</size>
            <attacher name="Tibor Mészáros">mtiborinf</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhZDE2
YWE5Li4zMzc0YjJkIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMjQgQEAKKzIwMTQtMDYtMDIgIFRpYm9yIE1lc3phcm9zICA8dG1l
c3phcm9zLnUtc3plZ2VkQHBhcnRuZXIuc2Ftc3VuZy5jb20+CisKKyAgICAgICAgQWRkIGEgdmVy
Ym9zZSBmbGFnIHRvIGJpbmRpbmcgdGVzdHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTEyMjMwMworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo
T09QUyEpLgorCisgICAgICAgIFRoaXMgcGF0Y2ggd2lsbDoKKyAgICAgICAgLSBBZGQgYSB2ZXJi
b3NlIGtleXdvcmQgYXJndW1lbnQgdG8gdGhlIEJpbmRpbmdzVGVzdHMgY29uc3RydWN0b3IsCisg
ICAgICAgIC0gQWRkIC0tbm8tdmVyYm9zZSBvcHRpb24gdG8gdGhlIHJ1bi1iaW5kaWdzLXRlc3Qu
CisgICAgICAgIFZlcmJvc2UgbW9kZSwgd2hpY2ggaXMgb24gYnkgZGVmYXVsdCwgcHJpbnRzIHBv
c2l0aXZlIHJlc3VsdHMgdG8gc3Rkb3V0IGFzIGJlZm9yZS4KKyAgICAgICAgV2hlbiB0aGUgLS1u
by12ZXJib3NlIG9wdGlvbiBhZGRlZCB0byBydW4tYmluZGluZ3MtdGVzdHMsIHBvc2l0aXZlIHJl
c3VsdHMgYXJlIG5vdCBwcmludGVkIGF0IGFsbC4KKworICAgICAgICBPcmlnaW5hbCBwYXRjaCBm
cm9tIE1hdHRoZXcgSGFuc29uIDxtYXR0aGV3X2hhbnNvbkBhcHBsZS5jb20+CisKKyAgICAgICAg
KiBTY3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0czoKKyAgICAgICAgKG1haW4pOgorICAgICAgICAq
IFNjcmlwdHMvd2Via2l0cHkvYmluZGluZ3MvbWFpbi5weToKKyAgICAgICAgKEJpbmRpbmdzVGVz
dHMuX19pbml0X18pOgorICAgICAgICAoQmluZGluZ3NUZXN0cy5kZXRlY3RfY2hhbmdlcyk6CisK
IDIwMTQtMDUtMzEgIERhcmluIEFkbGVyICA8ZGFyaW5AYXBwbGUuY29tPgogCiAgICAgICAgIFZp
ZXdwb3J0IHBlcmNlbnRhZ2UgdGVzdHMgdGhhdCByZXNpemUgdGhlIHZpZXdwb3J0IGFyZSBmbGFr
eQpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9ydW4tYmluZGluZ3MtdGVzdHMgYi9Ub29scy9T
Y3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0cwppbmRleCBlNzA4MzlkLi5kMzE2ZmUyIDEwMDc1NQot
LS0gYS9Ub29scy9TY3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0cworKysgYi9Ub29scy9TY3JpcHRz
L3J1bi1iaW5kaW5ncy10ZXN0cwpAQCAtNDAsNiArNDAsNyBAQCBkZWYgbWFpbihhcmd2KToKIAog
ICAgICIiIgogICAgIHJlc2V0X3Jlc3VsdHMgPSAiLS1yZXNldC1yZXN1bHRzIiBpbiBhcmd2Cisg
ICAgdmVyYm9zZSA9IG5vdCAiLS1uby12ZXJib3NlIiBpbiBhcmd2CiAKICAgICBnZW5lcmF0b3Jz
ID0gWwogICAgICAgICAnSlMnLApAQCAtNDksNyArNTAsNyBAQCBkZWYgbWFpbihhcmd2KToKIAog
ICAgIGZyb20gd2Via2l0cHkuYmluZGluZ3MubWFpbiBpbXBvcnQgQmluZGluZ3NUZXN0cwogCi0g
ICAgcmV0dXJuIEJpbmRpbmdzVGVzdHMocmVzZXRfcmVzdWx0cywgZ2VuZXJhdG9ycywgZXhlY3V0
aXZlLkV4ZWN1dGl2ZSgpKS5tYWluKCkKKyAgICByZXR1cm4gQmluZGluZ3NUZXN0cyhyZXNldF9y
ZXN1bHRzLCBnZW5lcmF0b3JzLCBleGVjdXRpdmUuRXhlY3V0aXZlKCksIHZlcmJvc2UpLm1haW4o
KQogCiAKIGlmIF9fbmFtZV9fID09ICdfX21haW5fXyc6CmRpZmYgLS1naXQgYS9Ub29scy9TY3Jp
cHRzL3dlYmtpdHB5L2JpbmRpbmdzL21haW4ucHkgYi9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2Jp
bmRpbmdzL21haW4ucHkKaW5kZXggOTRiODI1Yy4uYjRhOGJjYiAxMDA2NDQKLS0tIGEvVG9vbHMv
U2NyaXB0cy93ZWJraXRweS9iaW5kaW5ncy9tYWluLnB5CisrKyBiL1Rvb2xzL1NjcmlwdHMvd2Vi
a2l0cHkvYmluZGluZ3MvbWFpbi5weQpAQCAtMzQsMTAgKzM0LDExIEBAIGZyb20gd2Via2l0cHku
Y29tbW9uLnN5c3RlbS5leGVjdXRpdmUgaW1wb3J0IFNjcmlwdEVycm9yCiAKIGNsYXNzIEJpbmRp
bmdzVGVzdHM6CiAKLSAgICBkZWYgX19pbml0X18oc2VsZiwgcmVzZXRfcmVzdWx0cywgZ2VuZXJh
dG9ycywgZXhlY3V0aXZlKToKKyAgICBkZWYgX19pbml0X18oc2VsZiwgcmVzZXRfcmVzdWx0cywg
Z2VuZXJhdG9ycywgZXhlY3V0aXZlLCB2ZXJib3NlKToKICAgICAgICAgc2VsZi5yZXNldF9yZXN1
bHRzID0gcmVzZXRfcmVzdWx0cwogICAgICAgICBzZWxmLmdlbmVyYXRvcnMgPSBnZW5lcmF0b3Jz
CiAgICAgICAgIHNlbGYuZXhlY3V0aXZlID0gZXhlY3V0aXZlCisgICAgICAgIHNlbGYudmVyYm9z
ZSA9IHZlcmJvc2UKIAogICAgIGRlZiBnZW5lcmF0ZV9mcm9tX2lkbChzZWxmLCBnZW5lcmF0b3Is
IGlkbF9maWxlLCBvdXRwdXRfZGlyZWN0b3J5LCBzdXBwbGVtZW50YWxfZGVwZW5kZW5jeV9maWxl
KToKICAgICAgICAgY21kID0gWydwZXJsJywgJy13JywKQEAgLTExMiw3ICsxMTMsNyBAQCBjbGFz
cyBCaW5kaW5nc1Rlc3RzOgogICAgICAgICAgICAgICAgIHByaW50ICdGQUlMOiAoJXMpICVzJyAl
IChnZW5lcmF0b3IsIG91dHB1dF9maWxlKQogICAgICAgICAgICAgICAgIHByaW50IG91dHB1dAog
ICAgICAgICAgICAgICAgIGNoYW5nZXNfZm91bmQgPSBUcnVlCi0gICAgICAgICAgICBlbHNlOgor
ICAgICAgICAgICAgZWxpZiBzZWxmLnZlcmJvc2U6CiAgICAgICAgICAgICAgICAgcHJpbnQgJ1BB
U1M6ICglcykgJXMnICUgKGdlbmVyYXRvciwgb3V0cHV0X2ZpbGUpCiAgICAgICAgIHJldHVybiBj
aGFuZ2VzX2ZvdW5kCiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>