<?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>100102</bug_id>
          
          <creation_ts>2012-10-23 03:43:37 -0700</creation_ts>
          <short_desc>[WK2] [GTK] TestWebKitAccessibility is not being run</short_desc>
          <delta_ts>2012-10-24 01:13:06 -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>WebKitGTK</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>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mario Sanchez Prada">mario</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>apinheiro</cc>
    
    <cc>cgarcia</cc>
    
    <cc>jdiggs</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>748454</commentid>
    <comment_count>0</comment_count>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2012-10-23 03:43:37 -0700</bug_when>
    <thetext>It seems a line got in run-gtk-tests by mistake along with SVN r120944, causing this test not to be run in WK2 bots:

     def _start_accessibility_daemons(self):
         return False
         spi_bus_launcher_path = self._lookup_atspi2_binary(&apos;at-spi-bus-launcher&apos;)
         spi_registryd_path = self._lookup_atspi2_binary(&apos;at-spi2-registryd&apos;)
         [...]

Additionally, there seems to be a mistake in configure.ac as well, since the following block is now _after_ the definition of enable_webkit2, meaning that the PKG_CHECK_MODULES is never executed:

  # check if atspi2 is available (only needed for Webkit2 unit tests)
  if test &quot;$enable_webkit2&quot; = &quot;yes&quot;; then
     PKG_CHECK_MODULES([ATSPI2],
                       [atspi-2 &gt;= $ATSPI2_REQUIRED_VERSION],
                       [have_atspi2=yes], [have_atspi2=no])

     AC_SUBST([ATSPI2_CFLAGS])
     AC_SUBST([ATSPI2_LIBS])
  fi

This needs to be fixed</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>748464</commentid>
    <comment_count>1</comment_count>
      <attachid>170110</attachid>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2012-10-23 03:53:02 -0700</bug_when>
    <thetext>Created attachment 170110
Patch proposal</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>749409</commentid>
    <comment_count>2</comment_count>
      <attachid>170110</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2012-10-24 00:00:19 -0700</bug_when>
    <thetext>Comment on attachment 170110
Patch proposal

Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>749457</commentid>
    <comment_count>3</comment_count>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2012-10-24 01:13:06 -0700</bug_when>
    <thetext>Committed r132328: &lt;http://trac.webkit.org/changeset/132328&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>170110</attachid>
            <date>2012-10-23 03:53:02 -0700</date>
            <delta_ts>2012-10-24 00:00:19 -0700</delta_ts>
            <desc>Patch proposal</desc>
            <filename>0001-2012-10-23-Mario-Sanchez-Prada-mario-webkit.org.patch</filename>
            <type>text/plain</type>
            <size>4217</size>
            <attacher name="Mario Sanchez Prada">mario</attacher>
            
              <data encoding="base64">RnJvbSBkOGEwMDdhNjI4NjFhMzg2OTJlYTdiZTVhMTNmNTAwNGFkNWRjMWRkIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBNYXJpbyBTYW5jaGV6IFByYWRhIDxtYXJpb0BtYXJpb3Nwci5v
cmc+CkRhdGU6IFR1ZSwgMjMgT2N0IDIwMTIgMTI6NTA6MzggKzAyMDAKU3ViamVjdDogW1BBVENI
XSAyMDEyLTEwLTIzICBNYXJpbyBTYW5jaGV6IFByYWRhICA8bWFyaW9Ad2Via2l0Lm9yZz4KCiAg
ICAgICAgW1dLMl0gW0dUS10gVGVzdFdlYktpdEFjY2Vzc2liaWxpdHkgaXMgbm90IGJlaW5nIHJ1
bgogICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDAxMDIK
CiAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCgogICAgICAgIE1vdmVkIGNoZWNr
IGZvciBhdC1zcGkyIGFmdGVyIHRoZSBkZWZpbml0aW9uIG9mIGVuYWJsZV93ZWJraXQyLAogICAg
ICAgIHNpbmNlIGl0J3Mgb25seSBuZWVkZWQgZm9yIFdlYktpdDJHVEsgQVBJIHRlc3RzLgoKICAg
ICAgICAqIGNvbmZpZ3VyZS5hYzogTW92ZWQgdGhlIGNoZWNrIGZvciBhdC1zcGkyIGRvd24gaW4g
dGhlIGZpbGUuCgoyMDEyLTEwLTIzICBNYXJpbyBTYW5jaGV6IFByYWRhICA8bWFyaW9Ad2Via2l0
Lm9yZz4KCiAgICAgICAgW1dLMl0gW0dUS10gVGVzdFdlYktpdEFjY2Vzc2liaWxpdHkgaXMgbm90
IGJlaW5nIHJ1bgogICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD0xMDAxMDIKCiAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCgogICAgICAgIFJl
bW92ZWQgd3JvbmcgZWFybHkgcmV0dXJuIGluIF9zdGFydF9hY2Nlc3NpYmlsaXR5X2RhZW1vbnMo
KS4KCiAgICAgICAgKiBTY3JpcHRzL3J1bi1ndGstdGVzdHM6CiAgICAgICAgKFRlc3RSdW5uZXIu
X2xvb2t1cF9hdHNwaTJfYmluYXJ5KTogUmVtb3ZlZCB3cm9uZyBsaW5lLgotLS0KIENoYW5nZUxv
ZyAgICAgICAgICAgICAgICAgICB8IDEyICsrKysrKysrKysrKwogVG9vbHMvQ2hhbmdlTG9nICAg
ICAgICAgICAgIHwgMTIgKysrKysrKysrKysrCiBUb29scy9TY3JpcHRzL3J1bi1ndGstdGVzdHMg
fCAgMSAtCiBjb25maWd1cmUuYWMgICAgICAgICAgICAgICAgfCAyMCArKysrKysrKysrLS0tLS0t
LS0tLQogNCBmaWxlcyBjaGFuZ2VkLCAzNCBpbnNlcnRpb25zKCspLCAxMSBkZWxldGlvbnMoLSkK
CmRpZmYgLS1naXQgYS9DaGFuZ2VMb2cgYi9DaGFuZ2VMb2cKaW5kZXggMTJhNDFmYjYuLjg1Nzgx
NDEgMTAwNjQ0Ci0tLSBhL0NoYW5nZUxvZworKysgYi9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNSBA
QAorMjAxMi0xMC0yMyAgTWFyaW8gU2FuY2hleiBQcmFkYSAgPG1hcmlvQHdlYmtpdC5vcmc+CisK
KyAgICAgICAgW1dLMl0gW0dUS10gVGVzdFdlYktpdEFjY2Vzc2liaWxpdHkgaXMgbm90IGJlaW5n
IHJ1bgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTAw
MTAyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTW92
ZWQgY2hlY2sgZm9yIGF0LXNwaTIgYWZ0ZXIgdGhlIGRlZmluaXRpb24gb2YgZW5hYmxlX3dlYmtp
dDIsCisgICAgICAgIHNpbmNlIGl0J3Mgb25seSBuZWVkZWQgZm9yIFdlYktpdDJHVEsgQVBJIHRl
c3RzLgorCisgICAgICAgICogY29uZmlndXJlLmFjOiBNb3ZlZCB0aGUgY2hlY2sgZm9yIGF0LXNw
aTIgZG93biBpbiB0aGUgZmlsZS4KKwogMjAxMi0xMC0yMiAgUGF2ZWwgRmVsZG1hbiAgPHBmZWxk
bWFuQGNocm9taXVtLm9yZz4KIAogICAgICAgICBXZWIgSW5zcGVjdG9yOiBtZXJnZSAiZG9ja2Vk
IiBzdGF0ZSBpbnRvIHRoZSAiZG9jayBzaWRlIiBlbnVtLgpkaWZmIC0tZ2l0IGEvVG9vbHMvQ2hh
bmdlTG9nIGIvVG9vbHMvQ2hhbmdlTG9nCmluZGV4IDIzNGJiODAuLjRmMmZlMDIgMTAwNjQ0Ci0t
LSBhL1Rvb2xzL0NoYW5nZUxvZworKysgYi9Ub29scy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNSBA
QAorMjAxMi0xMC0yMyAgTWFyaW8gU2FuY2hleiBQcmFkYSAgPG1hcmlvQHdlYmtpdC5vcmc+CisK
KyAgICAgICAgW1dLMl0gW0dUS10gVGVzdFdlYktpdEFjY2Vzc2liaWxpdHkgaXMgbm90IGJlaW5n
IHJ1bgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTAw
MTAyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVt
b3ZlZCB3cm9uZyBlYXJseSByZXR1cm4gaW4gX3N0YXJ0X2FjY2Vzc2liaWxpdHlfZGFlbW9ucygp
LgorCisgICAgICAgICogU2NyaXB0cy9ydW4tZ3RrLXRlc3RzOgorICAgICAgICAoVGVzdFJ1bm5l
ci5fbG9va3VwX2F0c3BpMl9iaW5hcnkpOiBSZW1vdmVkIHdyb25nIGxpbmUuCisKIDIwMTItMTAt
MjIgIEpvY2hlbiBFaXNpbmdlciAgPGpvY2hlbkBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgW2No
cm9taXVtXSBNb3ZlIHRoZSBjYWxsIHRvIFJlZ2lzdGVySXNvbGF0ZWRGaWxlU3lzdGVtIHRvIHRo
ZSBXZWJUZXN0RGVsZWdhdGUgaW50ZXJmYWNlCmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3J1
bi1ndGstdGVzdHMgYi9Ub29scy9TY3JpcHRzL3J1bi1ndGstdGVzdHMKaW5kZXggMTI3NjU5Ni4u
ZjVjYTRmNCAxMDA3NTUKLS0tIGEvVG9vbHMvU2NyaXB0cy9ydW4tZ3RrLXRlc3RzCisrKyBiL1Rv
b2xzL1NjcmlwdHMvcnVuLWd0ay10ZXN0cwpAQCAtMTE5LDcgKzExOSw2IEBAIGNsYXNzIFRlc3RS
dW5uZXI6CiAgICAgICAgIHJldHVybiBOb25lCiAKICAgICBkZWYgX3N0YXJ0X2FjY2Vzc2liaWxp
dHlfZGFlbW9ucyhzZWxmKToKLSAgICAgICAgcmV0dXJuIEZhbHNlCiAgICAgICAgIHNwaV9idXNf
bGF1bmNoZXJfcGF0aCA9IHNlbGYuX2xvb2t1cF9hdHNwaTJfYmluYXJ5KCdhdC1zcGktYnVzLWxh
dW5jaGVyJykKICAgICAgICAgc3BpX3JlZ2lzdHJ5ZF9wYXRoID0gc2VsZi5fbG9va3VwX2F0c3Bp
Ml9iaW5hcnkoJ2F0LXNwaTItcmVnaXN0cnlkJykKICAgICAgICAgaWYgbm90IHNwaV9idXNfbGF1
bmNoZXJfcGF0aCBvciBub3Qgc3BpX3JlZ2lzdHJ5ZF9wYXRoOgpkaWZmIC0tZ2l0IGEvY29uZmln
dXJlLmFjIGIvY29uZmlndXJlLmFjCmluZGV4IDM3NjYzZTIuLjEwNWMzMWMgMTAwNjQ0Ci0tLSBh
L2NvbmZpZ3VyZS5hYworKysgYi9jb25maWd1cmUuYWMKQEAgLTEwNDQsMTYgKzEwNDQsNiBAQCBp
ZiB0ZXN0ICIkd2l0aF9hY2NlbGVyYXRpb25fYmFja2VuZCIgPSAib3BlbmdsIjsgdGhlbgogZmkK
IEFDX1NVQlNUKFtPUEVOR0xfTElCU10pCiAKLSMgY2hlY2sgaWYgYXRzcGkyIGlzIGF2YWlsYWJs
ZSAob25seSBuZWVkZWQgZm9yIFdlYmtpdDIgdW5pdCB0ZXN0cykKLWlmIHRlc3QgIiRlbmFibGVf
d2Via2l0MiIgPSAieWVzIjsgdGhlbgotICAgUEtHX0NIRUNLX01PRFVMRVMoW0FUU1BJMl0sCi0g
ICAgICAgICAgICAgICAgICAgICBbYXRzcGktMiA+PSAkQVRTUEkyX1JFUVVJUkVEX1ZFUlNJT05d
LAotICAgICAgICAgICAgICAgICAgICAgW2hhdmVfYXRzcGkyPXllc10sIFtoYXZlX2F0c3BpMj1u
b10pCi0KLSAgIEFDX1NVQlNUKFtBVFNQSTJfQ0ZMQUdTXSkKLSAgIEFDX1NVQlNUKFtBVFNQSTJf
TElCU10pCi1maQotCiBpZiB0ZXN0ICIkZW5hYmxlX2dhbWVwYWQiID0gInllcyI7IHRoZW4KICAg
IFBLR19DSEVDS19NT0RVTEVTKFtHQU1FUEFEXSwgW2dpby11bml4LTIuMCBndWRldi0xLjBdKQog
CkBAIC0xMTE0LDYgKzExMDQsMTYgQEAgQUNfU1VCU1QoR1RLMl9MSUJTKQogCiBHVEtfRE9DX0NI
RUNLKFsxLjEwXSkKIAorIyBXZSBuZWVkIGF0c3BpMiBmb3IgV2Via2l0MiB1bml0IHRlc3RzCitp
ZiB0ZXN0ICIkZW5hYmxlX3dlYmtpdDIiID0gInllcyI7IHRoZW4KKyAgIFBLR19DSEVDS19NT0RV
TEVTKFtBVFNQSTJdLAorICAgICAgICAgICAgICAgICAgICAgW2F0c3BpLTIgPj0gJEFUU1BJMl9S
RVFVSVJFRF9WRVJTSU9OXSwKKyAgICAgICAgICAgICAgICAgICAgIFtoYXZlX2F0c3BpMj15ZXNd
LCBbaGF2ZV9hdHNwaTI9bm9dKQorCisgICBBQ19TVUJTVChbQVRTUEkyX0NGTEFHU10pCisgICBB
Q19TVUJTVChbQVRTUEkyX0xJQlNdKQorZmkKKwogIyBPUyBjb25kaXRpb25hbHMKIEFNX0NPTkRJ
VElPTkFMKFtPU19XSU4zMl0sW3Rlc3QgIiRvc193aW4zMiIgPSAieWVzIl0pCiBBTV9DT05ESVRJ
T05BTChbT1NfVU5JWF0sW3Rlc3QgIiRvc193aW4zMiIgPSAibm8iXSkKLS0gCjEuNy4xMS43Cgo=
</data>
<flag name="review"
          id="183601"
          type_id="1"
          status="+"
          setter="cgarcia"
    />
          </attachment>
      

    </bug>

</bugzilla>