<?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>37786</bug_id>
          
          <creation_ts>2010-04-18 17:51:11 -0700</creation_ts>
          <short_desc>new-run-webkit-tests: checking the build should be optional</short_desc>
          <delta_ts>2010-04-19 17:33:14 -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>PC</rep_platform>
          <op_sys>OS X 10.5</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>1</everconfirmed>
          <reporter name="Dirk Pranke">dpranke</reporter>
          <assigned_to name="Dirk Pranke">dpranke</assigned_to>
          <cc>abarth</cc>
    
    <cc>cjerdonek</cc>
    
    <cc>eric</cc>
    
    <cc>ojan</cc>
    
    <cc>tony</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>213871</commentid>
    <comment_count>0</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-04-18 17:51:11 -0700</bug_when>
    <thetext>Checking whether or not a build is up-to-date can take 5-10 seconds or longer, and sometime you may wish to run the tests without rebuilding, so there should be a way to skip the check_build step.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>213872</commentid>
    <comment_count>1</comment_count>
      <attachid>53648</attachid>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-04-18 18:01:30 -0700</bug_when>
    <thetext>Created attachment 53648
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>213912</commentid>
    <comment_count>2</comment_count>
      <attachid>53648</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-04-18 22:25:19 -0700</bug_when>
    <thetext>Comment on attachment 53648
Patch

Two notes:
1.  Sad that we need this.  Seems like a hack around build system lameness.
2.  I hate that optparse doesn&apos;t have built in support for --no-.  We need to make our own make_option wrapper which does automatic --no- generation as well as puts the default value in the help text.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214033</commentid>
    <comment_count>3</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2010-04-19 08:03:21 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; 2.  I hate that optparse doesn&apos;t have built in support for --no-.  We need to
&gt; make our own make_option wrapper which does automatic --no- generation as well
&gt; as puts the default value in the help text.

Why do we need to have both flags? Since it defaults to True, don&apos;t we only need --no-check-build?

It seems like it&apos;s the actual rebuilding that slows things down. It&apos;s a shame to get rid of all the other checks in check_build just to avoid the rebuilding. How about adding a --no-build instead? I also think that would be more intuitive to users. If I didn&apos;t know the code, it wouldn&apos;t be clear to me that check-build is the thing that causes DRT to be built.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214090</commentid>
    <comment_count>4</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-04-19 10:30:27 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; (In reply to comment #2)
&gt; &gt; 2.  I hate that optparse doesn&apos;t have built in support for --no-.  We need to
&gt; &gt; make our own make_option wrapper which does automatic --no- generation as well
&gt; &gt; as puts the default value in the help text.
&gt; 
&gt; Why do we need to have both flags? Since it defaults to True, don&apos;t we only
&gt; need --no-check-build?

This may be a personal thing, but I don&apos;t like having boolean command line flags where both versions of the flag aren&apos;t legal.
 
&gt; It seems like it&apos;s the actual rebuilding that slows things down. It&apos;s a shame
&gt; to get rid of all the other checks in check_build just to avoid the rebuilding.
&gt; How about adding a --no-build instead? I also think that would be more
&gt; intuitive to users. If I didn&apos;t know the code, it wouldn&apos;t be clear to me that
&gt; check-build is the thing that causes DRT to be built.

This would be fine, and is probably safer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214280</commentid>
    <comment_count>5</comment_count>
      <attachid>53727</attachid>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-04-19 15:16:55 -0700</bug_when>
    <thetext>Created attachment 53727
revise w/  ojan&apos;s feedback (rename --check-build to --build)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214287</commentid>
    <comment_count>6</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2010-04-19 15:26:00 -0700</bug_when>
    <thetext>LGTM.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214314</commentid>
    <comment_count>7</comment_count>
      <attachid>53727</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-04-19 16:12:37 -0700</bug_when>
    <thetext>Comment on attachment 53727
revise w/  ojan&apos;s feedback (rename --check-build to --build)

The option name and help are misleading.  This will only make sure that DumpRenderTree is built.  Not webkit itself.

I&apos;ve been trying to use full sentences in option help (start with a capital, end with a period).

I think  the option names can stay, but we should fix the help to explain it only builds the helper tools, not WebKit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214334</commentid>
    <comment_count>8</comment_count>
      <attachid>53738</attachid>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-04-19 16:45:56 -0700</bug_when>
    <thetext>Created attachment 53738
tweak the help text, changelog</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214337</commentid>
    <comment_count>9</comment_count>
      <attachid>53738</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-04-19 16:48:56 -0700</bug_when>
    <thetext>Comment on attachment 53738
tweak the help text, changelog

OK.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214369</commentid>
    <comment_count>10</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-04-19 17:33:14 -0700</bug_when>
    <thetext>Committed r57858: &lt;http://trac.webkit.org/changeset/57858&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>53648</attachid>
            <date>2010-04-18 18:01:30 -0700</date>
            <delta_ts>2010-04-19 15:16:55 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-37786-20100418180128.patch</filename>
            <type>text/plain</type>
            <size>2057</size>
            <attacher name="Dirk Pranke">dpranke</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCA5OThhNmQzLi5kZDU0YzNjIDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9DaGFuZ2VM
b2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDQt
MTggIERpcmsgUHJhbmtlICA8ZHByYW5rZUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgbmV3LXJ1bi13ZWJraXQtdGVzdHM6IGFk
ZCAtLWNoZWNrLWJ1aWxkIGFuZCAtLW5vLWNoZWNrLWJ1aWxkCisgICAgICAgIG9wdGlvbnMgdG8g
bWFrZSB0aGF0IHN0ZXAgb3B0aW9uYWwuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTM3Nzg2CisKKyAgICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2xh
eW91dF90ZXN0cy9ydW5fd2Via2l0X3Rlc3RzLnB5OgorCiAyMDEwLTA0LTE1ICBTYW0gV2Vpbmln
ICA8c2FtQHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQW5kZXJzIENhcmxzc29u
LgpkaWZmIC0tZ2l0IGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMv
cnVuX3dlYmtpdF90ZXN0cy5weSBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0
X3Rlc3RzL3J1bl93ZWJraXRfdGVzdHMucHkKaW5kZXggMDU5ZmQwOS4uZWE0YWM4NiAxMDA3NTUK
LS0tIGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvcnVuX3dlYmtp
dF90ZXN0cy5weQorKysgYi9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0
cy9ydW5fd2Via2l0X3Rlc3RzLnB5CkBAIC0xNTAwLDkgKzE1MDAsMTAgQEAgZGVmIG1haW4ob3B0
aW9ucywgYXJncywgcHJpbnRfcmVzdWx0cz1UcnVlKToKICAgICB0ZXN0X3J1bm5lci5wYXJzZV9l
eHBlY3RhdGlvbnMocG9ydF9vYmoudGVzdF9wbGF0Zm9ybV9uYW1lKCksCiAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgIG9wdGlvbnMuY29uZmlndXJhdGlvbiA9PSAnRGVidWcnKQog
Ci0gICAgbWV0ZXIudXBkYXRlKCJDaGVja2luZyBidWlsZCAuLi4iKQotICAgIGlmIG5vdCBwb3J0
X29iai5jaGVja19idWlsZCh0ZXN0X3J1bm5lci5uZWVkc19odHRwKCkpOgotICAgICAgICByZXR1
cm4gLTEKKyAgICBpZiBvcHRpb25zLmNoZWNrX2J1aWxkOgorICAgICAgICBtZXRlci51cGRhdGUo
IkNoZWNraW5nIGJ1aWxkIC4uLiIpCisgICAgICAgIGlmIG5vdCBwb3J0X29iai5jaGVja19idWls
ZCh0ZXN0X3J1bm5lci5uZWVkc19odHRwKCkpOgorICAgICAgICAgICAgcmV0dXJuIC0xCiAKICAg
ICBtZXRlci51cGRhdGUoIlN0YXJ0aW5nIGhlbHBlciAuLi4iKQogICAgIHBvcnRfb2JqLnN0YXJ0
X2hlbHBlcigpCkBAIC0xNjM4LDYgKzE2MzksMTIgQEAgZGVmIHBhcnNlX2FyZ3MoYXJncz1Ob25l
KToKICAgICBdCiAKICAgICB0ZXN0X29wdGlvbnMgPSBbCisgICAgICAgIG9wdHBhcnNlLm1ha2Vf
b3B0aW9uKCItLWNoZWNrLWJ1aWxkIiwgZGVzdD0iY2hlY2tfYnVpbGQiLAorICAgICAgICAgICAg
YWN0aW9uPSJzdG9yZV90cnVlIiwgZGVmYXVsdD1UcnVlLAorICAgICAgICAgICAgaGVscD0iY2hl
Y2sgdG8gZW5zdXJlIHRoZSBidWlsZCBpcyB1cC10by1kYXRlIChkZWZhdWx0KS4iKSwKKyAgICAg
ICAgb3B0cGFyc2UubWFrZV9vcHRpb24oIi0tbm8tY2hlY2stYnVpbGQiLCBkZXN0PSJjaGVja19i
dWlsZCIsCisgICAgICAgICAgICBhY3Rpb249InN0b3JlX2ZhbHNlIiwKKyAgICAgICAgICAgIGhl
bHA9ImRvbid0IGNoZWNrIHRvIHNlZSBpZiB0aGUgYnVpbGQgaXMgdXAtdG8tZGF0ZS4iKSwKICAg
ICAgICAgIyBvbGQtcnVuLXdlYmtpdC10ZXN0cyBoYXMgLS12YWxncmluZCBpbnN0ZWFkIG9mIHdy
YXBwZXIuCiAgICAgICAgIG9wdHBhcnNlLm1ha2Vfb3B0aW9uKCItLXdyYXBwZXIiLAogICAgICAg
ICAgICAgaGVscD0id3JhcHBlciBjb21tYW5kIHRvIGluc2VydCBiZWZvcmUgaW52b2NhdGlvbnMg
b2YgIgo=
</data>
<flag name="review"
          id="37415"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>53727</attachid>
            <date>2010-04-19 15:16:55 -0700</date>
            <delta_ts>2010-04-19 16:48:25 -0700</delta_ts>
            <desc>revise w/  ojan&apos;s feedback (rename --check-build to --build)</desc>
            <filename>wk37786_skip_build.diff</filename>
            <type>text/plain</type>
            <size>2782</size>
            <attacher name="Dirk Pranke">dpranke</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCA5OThhNmQzLi42NjNkZTJlIDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9DaGFuZ2VM
b2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDQt
MTggIERpcmsgUHJhbmtlICA8ZHByYW5rZUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgbmV3LXJ1bi13ZWJraXQtdGVzdHM6IGFk
ZCAtLWJ1aWxkIChkZWZhdWx0KSBhbmQgLS1uby1idWlsZAorICAgICAgICBvcHRpb25zIHRvIG1h
a2UgdGhhdCBzdGVwIG9wdGlvbmFsLgorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0zNzc4NgorCisgICAgICAgICogU2NyaXB0cy93ZWJraXRweS9sYXlv
dXRfdGVzdHMvcnVuX3dlYmtpdF90ZXN0cy5weToKKwogMjAxMC0wNC0xNSAgU2FtIFdlaW5pZyAg
PHNhbUB3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEFuZGVycyBDYXJsc3Nvbi4K
ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3Bv
cnQvY2hyb21pdW0ucHkgYi9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0
cy9wb3J0L2Nocm9taXVtLnB5CmluZGV4IGI2NjE4NGMuLjVhOGNmYmQgMTAwNjQ0Ci0tLSBhL1dl
YktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvY2hyb21pdW0ucHkK
KysrIGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvcG9ydC9jaHJv
bWl1bS5weQpAQCAtODAsNyArODAsNyBAQCBjbGFzcyBDaHJvbWl1bVBvcnQoYmFzZS5Qb3J0KToK
ICAgICAgICAgZHVtcF9yZW5kZXJfdHJlZV9iaW5hcnlfcGF0aCA9IHNlbGYuX3BhdGhfdG9fZHJp
dmVyKCkKICAgICAgICAgcmVzdWx0ID0gY2hlY2tfZmlsZV9leGlzdHMoZHVtcF9yZW5kZXJfdHJl
ZV9iaW5hcnlfcGF0aCwKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgJ3Rlc3Qg
ZHJpdmVyJykKLSAgICAgICAgaWYgcmVzdWx0OgorICAgICAgICBpZiByZXN1bHQgYW5kIHNlbGYu
X29wdGlvbnMuYnVpbGQ6CiAgICAgICAgICAgICByZXN1bHQgPSAoc2VsZi5fY2hlY2tfZHJpdmVy
X2J1aWxkX3VwX3RvX2RhdGUoc2VsZi5fb3B0aW9ucy5jb25maWd1cmF0aW9uKQogICAgICAgICAg
ICAgICAgICAgICAgIGFuZCByZXN1bHQpCiAgICAgICAgIGVsc2U6CmRpZmYgLS1naXQgYS9XZWJL
aXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L3dlYmtpdC5weSBiL1dl
YktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvd2Via2l0LnB5Cmlu
ZGV4IGQxMGZhOTAuLmNmNjE0NTUgMTAwNjQ0Ci0tLSBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Vi
a2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvd2Via2l0LnB5CisrKyBiL1dlYktpdFRvb2xzL1Njcmlw
dHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvd2Via2l0LnB5CkBAIC04OSw3ICs4OSw3IEBA
IGNsYXNzIFdlYktpdFBvcnQoYmFzZS5Qb3J0KToKICAgICAgICAgcmV0dXJuIFRydWUKIAogICAg
IGRlZiBjaGVja19idWlsZChzZWxmLCBuZWVkc19odHRwKToKLSAgICAgICAgaWYgbm90IHNlbGYu
X2J1aWxkX2RyaXZlcigpOgorICAgICAgICBpZiBzZWxmLl9vcHRpb25zLmJ1aWxkIGFuZCBub3Qg
c2VsZi5fYnVpbGRfZHJpdmVyKCk6CiAgICAgICAgICAgICByZXR1cm4gRmFsc2UKICAgICAgICAg
aWYgbm90IHNlbGYuX2NoZWNrX2RyaXZlcigpOgogICAgICAgICAgICAgcmV0dXJuIEZhbHNlCmRp
ZmYgLS1naXQgYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9ydW5f
d2Via2l0X3Rlc3RzLnB5IGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVz
dHMvcnVuX3dlYmtpdF90ZXN0cy5weQppbmRleCAwNTlmZDA5Li4yZTE2YTgxIDEwMDc1NQotLS0g
YS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9ydW5fd2Via2l0X3Rl
c3RzLnB5CisrKyBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3J1
bl93ZWJraXRfdGVzdHMucHkKQEAgLTE2MzgsNiArMTYzOCwxMiBAQCBkZWYgcGFyc2VfYXJncyhh
cmdzPU5vbmUpOgogICAgIF0KIAogICAgIHRlc3Rfb3B0aW9ucyA9IFsKKyAgICAgICAgb3B0cGFy
c2UubWFrZV9vcHRpb24oIi0tYnVpbGQiLCBkZXN0PSJidWlsZCIsCisgICAgICAgICAgICBhY3Rp
b249InN0b3JlX3RydWUiLCBkZWZhdWx0PVRydWUsCisgICAgICAgICAgICBoZWxwPSJjaGVjayB0
byBlbnN1cmUgdGhlIGJ1aWxkIGlzIHVwLXRvLWRhdGUgKGRlZmF1bHQpLiIpLAorICAgICAgICBv
cHRwYXJzZS5tYWtlX29wdGlvbigiLS1uby1idWlsZCIsIGRlc3Q9ImJ1aWxkIiwKKyAgICAgICAg
ICAgIGFjdGlvbj0ic3RvcmVfZmFsc2UiLAorICAgICAgICAgICAgaGVscD0iZG9uJ3QgY2hlY2sg
dG8gc2VlIGlmIHRoZSBidWlsZCBpcyB1cC10by1kYXRlLiIpLAogICAgICAgICAjIG9sZC1ydW4t
d2Via2l0LXRlc3RzIGhhcyAtLXZhbGdyaW5kIGluc3RlYWQgb2Ygd3JhcHBlci4KICAgICAgICAg
b3B0cGFyc2UubWFrZV9vcHRpb24oIi0td3JhcHBlciIsCiAgICAgICAgICAgICBoZWxwPSJ3cmFw
cGVyIGNvbW1hbmQgdG8gaW5zZXJ0IGJlZm9yZSBpbnZvY2F0aW9ucyBvZiAiCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>53738</attachid>
            <date>2010-04-19 16:45:56 -0700</date>
            <delta_ts>2010-04-19 16:48:56 -0700</delta_ts>
            <desc>tweak the help text, changelog</desc>
            <filename>wk37786_skip_build.diff</filename>
            <type>text/plain</type>
            <size>3011</size>
            <attacher name="Dirk Pranke">dpranke</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCA5OThhNmQzLi43ZjIxNGYwIDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9DaGFuZ2VM
b2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTAtMDQt
MTggIERpcmsgUHJhbmtlICA8ZHByYW5rZUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgbmV3LXJ1bi13ZWJraXQtdGVzdHM6IGFk
ZCAtLWJ1aWxkIChkZWZhdWx0KSBhbmQgLS1uby1idWlsZAorICAgICAgICBvcHRpb25zIHRvIG1h
a2UgdGhhdCBzdGVwIG9wdGlvbmFsLiBUaGlzIGZsYWcgbW9kaWZpZXMgd2hhdCBoYXBwZW5zCisg
ICAgICAgIGluIHBvcnQuY2hlY2tfYnVpbGQoKS4KKworICAgICAgICBodHRwczovL2J1Z3Mud2Vi
a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mzc3ODYKKworICAgICAgICAqIFNjcmlwdHMvd2Via2l0
cHkvbGF5b3V0X3Rlc3RzL3J1bl93ZWJraXRfdGVzdHMucHk6CisgICAgICAgICogU2NyaXB0cy93
ZWJraXRweS9sYXlvdXRfdGVzdHMvcG9ydC9jaHJvbWl1bS5weToKKyAgICAgICAgKiBTY3JpcHRz
L3dlYmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L3dlYmtpdC5weToKKwogMjAxMC0wNC0xNSAgU2Ft
IFdlaW5pZyAgPHNhbUB3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEFuZGVycyBD
YXJsc3Nvbi4KZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0
X3Rlc3RzL3BvcnQvY2hyb21pdW0ucHkgYi9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xh
eW91dF90ZXN0cy9wb3J0L2Nocm9taXVtLnB5CmluZGV4IGI2NjE4NGMuLjVhOGNmYmQgMTAwNjQ0
Ci0tLSBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvY2hy
b21pdW0ucHkKKysrIGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMv
cG9ydC9jaHJvbWl1bS5weQpAQCAtODAsNyArODAsNyBAQCBjbGFzcyBDaHJvbWl1bVBvcnQoYmFz
ZS5Qb3J0KToKICAgICAgICAgZHVtcF9yZW5kZXJfdHJlZV9iaW5hcnlfcGF0aCA9IHNlbGYuX3Bh
dGhfdG9fZHJpdmVyKCkKICAgICAgICAgcmVzdWx0ID0gY2hlY2tfZmlsZV9leGlzdHMoZHVtcF9y
ZW5kZXJfdHJlZV9iaW5hcnlfcGF0aCwKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgJ3Rlc3QgZHJpdmVyJykKLSAgICAgICAgaWYgcmVzdWx0OgorICAgICAgICBpZiByZXN1bHQg
YW5kIHNlbGYuX29wdGlvbnMuYnVpbGQ6CiAgICAgICAgICAgICByZXN1bHQgPSAoc2VsZi5fY2hl
Y2tfZHJpdmVyX2J1aWxkX3VwX3RvX2RhdGUoc2VsZi5fb3B0aW9ucy5jb25maWd1cmF0aW9uKQog
ICAgICAgICAgICAgICAgICAgICAgIGFuZCByZXN1bHQpCiAgICAgICAgIGVsc2U6CmRpZmYgLS1n
aXQgYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L3dlYmtp
dC5weSBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvd2Vi
a2l0LnB5CmluZGV4IGQxMGZhOTAuLmNmNjE0NTUgMTAwNjQ0Ci0tLSBhL1dlYktpdFRvb2xzL1Nj
cmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvd2Via2l0LnB5CisrKyBiL1dlYktpdFRv
b2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvd2Via2l0LnB5CkBAIC04OSw3
ICs4OSw3IEBAIGNsYXNzIFdlYktpdFBvcnQoYmFzZS5Qb3J0KToKICAgICAgICAgcmV0dXJuIFRy
dWUKIAogICAgIGRlZiBjaGVja19idWlsZChzZWxmLCBuZWVkc19odHRwKToKLSAgICAgICAgaWYg
bm90IHNlbGYuX2J1aWxkX2RyaXZlcigpOgorICAgICAgICBpZiBzZWxmLl9vcHRpb25zLmJ1aWxk
IGFuZCBub3Qgc2VsZi5fYnVpbGRfZHJpdmVyKCk6CiAgICAgICAgICAgICByZXR1cm4gRmFsc2UK
ICAgICAgICAgaWYgbm90IHNlbGYuX2NoZWNrX2RyaXZlcigpOgogICAgICAgICAgICAgcmV0dXJu
IEZhbHNlCmRpZmYgLS1naXQgYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90
ZXN0cy9ydW5fd2Via2l0X3Rlc3RzLnB5IGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9s
YXlvdXRfdGVzdHMvcnVuX3dlYmtpdF90ZXN0cy5weQppbmRleCAwNTlmZDA5Li45MDZlMjg4IDEw
MDc1NQotLS0gYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9ydW5f
d2Via2l0X3Rlc3RzLnB5CisrKyBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0
X3Rlc3RzL3J1bl93ZWJraXRfdGVzdHMucHkKQEAgLTE2MzgsNiArMTYzOCwxMyBAQCBkZWYgcGFy
c2VfYXJncyhhcmdzPU5vbmUpOgogICAgIF0KIAogICAgIHRlc3Rfb3B0aW9ucyA9IFsKKyAgICAg
ICAgb3B0cGFyc2UubWFrZV9vcHRpb24oIi0tYnVpbGQiLCBkZXN0PSJidWlsZCIsCisgICAgICAg
ICAgICBhY3Rpb249InN0b3JlX3RydWUiLCBkZWZhdWx0PVRydWUsCisgICAgICAgICAgICBoZWxw
PSJDaGVjayB0byBlbnN1cmUgdGhlIER1bXBSZW5kZXJUcmVlIGJ1aWxkIGlzIHVwLXRvLWRhdGUg
IgorICAgICAgICAgICAgIihkZWZhdWx0KS4iKSwKKyAgICAgICAgb3B0cGFyc2UubWFrZV9vcHRp
b24oIi0tbm8tYnVpbGQiLCBkZXN0PSJidWlsZCIsCisgICAgICAgICAgICBhY3Rpb249InN0b3Jl
X2ZhbHNlIiwgaGVscD0iRG9uJ3QgY2hlY2sgdG8gc2VlIGlmIHRoZSAiCisgICAgICAgICAgICAi
RHVtcFJlbmRlclRyZWUgYnVpbGQgaXMgdXAtdG8tZGF0ZS4iKSwKICAgICAgICAgIyBvbGQtcnVu
LXdlYmtpdC10ZXN0cyBoYXMgLS12YWxncmluZCBpbnN0ZWFkIG9mIHdyYXBwZXIuCiAgICAgICAg
IG9wdHBhcnNlLm1ha2Vfb3B0aW9uKCItLXdyYXBwZXIiLAogICAgICAgICAgICAgaGVscD0id3Jh
cHBlciBjb21tYW5kIHRvIGluc2VydCBiZWZvcmUgaW52b2NhdGlvbnMgb2YgIgo=
</data>
<flag name="review"
          id="37532"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>