<?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>57614</bug_id>
          
          <creation_ts>2011-04-01 04:49:13 -0700</creation_ts>
          <short_desc>ARIA Grid tables should return GridRole in roleValue() method</short_desc>
          <delta_ts>2011-04-14 07:17: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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>REOPENED</bug_status>
          <resolution></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>
          <dependson>57643</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mario Sanchez Prada">mario</reporter>
          <assigned_to name="Mario Sanchez Prada">mario</assigned_to>
          <cc>abarth</cc>
    
    <cc>cfleizach</cc>
    
    <cc>eric</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>377712</commentid>
    <comment_count>0</comment_count>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2011-04-01 04:49:13 -0700</bug_when>
    <thetext>ARIA Grid tables should return GridRole in roleValue() method</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>377713</commentid>
    <comment_count>1</comment_count>
      <attachid>87840</attachid>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2011-04-01 04:53:43 -0700</bug_when>
    <thetext>Created attachment 87840
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>377715</commentid>
    <comment_count>2</comment_count>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2011-04-01 04:54:44 -0700</bug_when>
    <thetext>Added Chris to CC</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>377826</commentid>
    <comment_count>3</comment_count>
      <attachid>87840</attachid>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2011-04-01 08:58:28 -0700</bug_when>
    <thetext>Comment on attachment 87840
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>377847</commentid>
    <comment_count>4</comment_count>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2011-04-01 09:16:16 -0700</bug_when>
    <thetext>Committed r82687: &lt;http://trac.webkit.org/changeset/82687&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>377900</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-01 10:29:17 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/82687 might have broken SnowLeopard Intel Release (Tests)
The following tests are not passing:
platform/mac/accessibility/aria-treegrid.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>377912</commentid>
    <comment_count>6</comment_count>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2011-04-01 10:41:51 -0700</bug_when>
    <thetext>Reopening, as current patch broke one test in the Mac bots:

http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r82689%20(27607)/platform/mac/accessibility

...seems like something else in the Mac code needs changing. Ideas?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>377917</commentid>
    <comment_count>7</comment_count>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2011-04-01 10:55:51 -0700</bug_when>
    <thetext>i don&apos;t know. i&apos;ll have to apply the patch and figure out</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>385745</commentid>
    <comment_count>8</comment_count>
      <attachid>87840</attachid>
    <who name="Mario Sanchez Prada">mario</who>
    <bug_when>2011-04-14 07:17:14 -0700</bug_when>
    <thetext>Comment on attachment 87840
Patch

Clearing flags. Not proposing anything yet since I found out that some more changes would be needed in the GTK port in order to keep working as expected (like checking GridRole&apos;s in places where only TableRole is checked at the moment), hence a Layout Test would also be needed so we can automate the detection for this kind of regressions.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>87840</attachid>
            <date>2011-04-01 04:53:43 -0700</date>
            <delta_ts>2011-04-14 07:17:14 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-57614-20110401135300.patch</filename>
            <type>text/plain</type>
            <size>4171</size>
            <attacher name="Mario Sanchez Prada">mario</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODI2NjEKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCBmMzczZWRlODNiMzgyNTdi
ZjE1ZmM2OTY5OGZmM2Y1MTZiN2JkOGUwLi5kOGY4ZjA3NjkzY2I0ZDllZmIwNDcxZGQ0NTk1MTg2
Y2EzNWMzNjllIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291
cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjUgQEAKKzIwMTEtMDQtMDEgIE1hcmlv
IFNhbmNoZXogUHJhZGEgIDxtc2FuY2hlekBpZ2FsaWEuY29tPgorCisgICAgICAgIFJldmlld2Vk
IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFSSUEgR3JpZCB0YWJsZXMgc2hvdWxkIHJl
dHVybiBHcmlkUm9sZSBpbiByb2xlVmFsdWUoKSBtZXRob2QKKyAgICAgICAgaHR0cHM6Ly9idWdz
LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTU3NjE0CisKKyAgICAgICAgVGhpcyBjaGFuZ2Ug
ZG9lcyBub3QgbmVlZCBhbnkgdGVzdCBzaW5jZSBpdCBkb2Vzbid0IGNoYW5nZSBhbnl0aGluZwor
ICAgICAgICBmcm9tIHRoZSBwb2ludCBvZiB2aWV3IG9mIHRoZSBjb25zdW1lcnMgKEFzc2lzdGl2
ZSBUZWNobm9nbG9naWVzKS4KKyAgICAgICAgSXQncyBqdXN0IGFuIGludGVybmFsIGNoYW5nZSB0
byBzaW1wbGlmeSBpZGVudGlmeWluZyBIVE1MIGFuZCBBUklBCisgICAgICAgIHRhYmxlcyBieSBj
YWxsaW5nIHRvIHRoZSBBY2Nlc3NpYmlsaXR5T2JqZWN0Ojpyb2xlVmFsdWUgbWV0aG9kLgorCisg
ICAgICAgICogYWNjZXNzaWJpbGl0eS9BY2Nlc3NpYmlsaXR5QVJJQUdyaWQuaDoKKyAgICAgICAg
KiBhY2Nlc3NpYmlsaXR5L0FjY2Vzc2liaWxpdHlBUklBR3JpZC5jcHA6CisgICAgICAgIChXZWJD
b3JlOjpBY2Nlc3NpYmlsaXR5QVJJQUdyaWQ6OnJvbGVWYWx1ZSk6IFJldHVybiBHcmlkUm9sZS4K
KworICAgICAgICAqIGFjY2Vzc2liaWxpdHkvZ3RrL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVy
QXRrLmNwcDoKKyAgICAgICAgKGF0a1JvbGUpOiBSZW1vdmVkIHVubmVlZGVkIGNvbW1lbnQuCisK
KyAgICAgICAgKiBhY2Nlc3NpYmlsaXR5L21hYy9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlci5t
bTogTWFwIEdyaWRSb2xlIHRvCisgICAgICAgIE5TQWNjZXNzaWJpbGl0eVRhYmxlUm9sZSwgdG8g
a2VlcCB0aGUgc2FtZSBiZWhhdmlvdXIuCisKIDIwMTEtMDMtMzEgIFNoZXJpZmYgQm90ICA8d2Vi
a2l0LnJldmlldy5ib3RAZ21haWwuY29tPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIHJvbGxpbmcg
b3V0IHI4MjY1Mi4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkvQWNj
ZXNzaWJpbGl0eUFSSUFHcmlkLmNwcCBiL1NvdXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkvQWNj
ZXNzaWJpbGl0eUFSSUFHcmlkLmNwcAppbmRleCBkNTE3NTBhY2NiM2Y0ZjlkN2I2NzA3YmQxMDQz
MWE3N2NmYjY4ZWUwLi44NDIzZjVmNzM5OGJhMTMyYzJlNzQ4ZDhmNzU1MGNhODg4OTllOThjIDEw
MDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9hY2Nlc3NpYmlsaXR5L0FjY2Vzc2liaWxpdHlBUklB
R3JpZC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvYWNjZXNzaWJpbGl0eS9BY2Nlc3NpYmlsaXR5
QVJJQUdyaWQuY3BwCkBAIC0xNzIsNSArMTcyLDEzIEBAIEFjY2Vzc2liaWxpdHlUYWJsZUNlbGwq
IEFjY2Vzc2liaWxpdHlBUklBR3JpZDo6Y2VsbEZvckNvbHVtbkFuZFJvdyh1bnNpZ25lZCBjb2x1
CiAKICAgICByZXR1cm4gMDsKIH0KLSAgICAKKworQWNjZXNzaWJpbGl0eVJvbGUgQWNjZXNzaWJp
bGl0eUFSSUFHcmlkOjpyb2xlVmFsdWUoKSBjb25zdAoreworICAgIGlmICghaXNBY2Nlc3NpYmls
aXR5VGFibGUoKSkKKyAgICAgICAgcmV0dXJuIEFjY2Vzc2liaWxpdHlSZW5kZXJPYmplY3Q6OnJv
bGVWYWx1ZSgpOworCisgICAgcmV0dXJuIEdyaWRSb2xlOworfQorCiB9IC8vIG5hbWVzcGFjZSBX
ZWJDb3JlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9hY2Nlc3NpYmlsaXR5L0FjY2Vzc2li
aWxpdHlBUklBR3JpZC5oIGIvU291cmNlL1dlYkNvcmUvYWNjZXNzaWJpbGl0eS9BY2Nlc3NpYmls
aXR5QVJJQUdyaWQuaAppbmRleCAwMmZmNGQwZTQwNGE5OTEwOTQ0ODI5ZjFjYWZhOWQyZGVjOGJj
NWIzLi5iMWYyNzk4ZTJiNDZlMWY5OTNlODc2N2FmZDFlZGI2M2RjNDQwYWEzIDEwMDY0NAotLS0g
YS9Tb3VyY2UvV2ViQ29yZS9hY2Nlc3NpYmlsaXR5L0FjY2Vzc2liaWxpdHlBUklBR3JpZC5oCisr
KyBiL1NvdXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkvQWNjZXNzaWJpbGl0eUFSSUFHcmlkLmgK
QEAgLTQ0LDcgKzQ0LDggQEAgcHJpdmF0ZToKIHB1YmxpYzoKICAgICBzdGF0aWMgUGFzc1JlZlB0
cjxBY2Nlc3NpYmlsaXR5QVJJQUdyaWQ+IGNyZWF0ZShSZW5kZXJPYmplY3QqKTsKICAgICB2aXJ0
dWFsIH5BY2Nlc3NpYmlsaXR5QVJJQUdyaWQoKTsKLSAgICAKKworICAgIHZpcnR1YWwgQWNjZXNz
aWJpbGl0eVJvbGUgcm9sZVZhbHVlKCkgY29uc3Q7CiAgICAgdmlydHVhbCBib29sIGlzQXJpYVRh
YmxlKCkgY29uc3QgeyByZXR1cm4gdHJ1ZTsgfSAgICAKICAgICAKICAgICB2aXJ0dWFsIHZvaWQg
YWRkQ2hpbGRyZW4oKTsKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkv
Z3RrL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVyQXRrLmNwcCBiL1NvdXJjZS9XZWJDb3JlL2Fj
Y2Vzc2liaWxpdHkvZ3RrL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVyQXRrLmNwcAppbmRleCBh
NTU4ZTIyODkwOTU3MDE3ZjNiZTk1ZDdjMDQxYWRkNjFiZDc2ODc4Li40NzZjZjU5ODI4YTIxMGVj
NmQ2OTU4OTBkMGQ0NDgzNWI1NmYwNzBjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9hY2Nl
c3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHAKKysrIGIvU291
cmNlL1dlYkNvcmUvYWNjZXNzaWJpbGl0eS9ndGsvQWNjZXNzaWJpbGl0eU9iamVjdFdyYXBwZXJB
dGsuY3BwCkBAIC00NDcsNyArNDQ3LDcgQEAgc3RhdGljIEF0a1JvbGUgYXRrUm9sZShBY2Nlc3Np
YmlsaXR5Um9sZSByb2xlKQogICAgICAgICByZXR1cm4gQVRLX1JPTEVfU0NST0xMX0JBUjsKICAg
ICBjYXNlIFNjcm9sbEFyZWFSb2xlOgogICAgICAgICByZXR1cm4gQVRLX1JPTEVfU0NST0xMX1BB
TkU7Ci0gICAgY2FzZSBHcmlkUm9sZTogLy8gSXMgdGhpcyByaWdodD8KKyAgICBjYXNlIEdyaWRS
b2xlOgogICAgIGNhc2UgVGFibGVSb2xlOgogICAgICAgICByZXR1cm4gQVRLX1JPTEVfVEFCTEU7
CiAgICAgY2FzZSBBcHBsaWNhdGlvblJvbGU6CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9h
Y2Nlc3NpYmlsaXR5L21hYy9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlci5tbSBiL1NvdXJjZS9X
ZWJDb3JlL2FjY2Vzc2liaWxpdHkvbWFjL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVyLm1tCmlu
ZGV4IGMwZGQ3OTU1OTY1YjQ1MjljMzdjNDFiYjU4M2EyNTdkZDYzMDU1OGUuLmRjY2ZiMDY2NmYz
MmRkOGM3NWYxODY4YmY4ODY2YWU2NzAyOTMzYTEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3Jl
L2FjY2Vzc2liaWxpdHkvbWFjL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVyLm1tCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkvbWFjL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVy
Lm1tCkBAIC0xMjQzLDcgKzEyNDMsNyBAQCBzdGF0aWMgY29uc3QgQWNjZXNzaWJpbGl0eVJvbGVN
YXAmIGNyZWF0ZUFjY2Vzc2liaWxpdHlSb2xlTWFwKCkKICAgICAgICAgeyBMaW5rUm9sZSwgTlNB
Y2Nlc3NpYmlsaXR5TGlua1JvbGUgfSwKICNpZm5kZWYgQlVJTERJTkdfT05fVElHRVIgICAgICAg
IAogICAgICAgICB7IERpc2Nsb3N1cmVUcmlhbmdsZVJvbGUsIE5TQWNjZXNzaWJpbGl0eURpc2Ns
b3N1cmVUcmlhbmdsZVJvbGUgfSwKLSAgICAgICAgeyBHcmlkUm9sZSwgTlNBY2Nlc3NpYmlsaXR5
R3JpZFJvbGUgfSwKKyAgICAgICAgeyBHcmlkUm9sZSwgTlNBY2Nlc3NpYmlsaXR5VGFibGVSb2xl
IH0sCiAjZW5kaWYKICAgICAgICAgeyBXZWJDb3JlTGlua1JvbGUsIE5TQWNjZXNzaWJpbGl0eUxp
bmtSb2xlIH0sIAogICAgICAgICB7IEltYWdlTWFwTGlua1JvbGUsIE5TQWNjZXNzaWJpbGl0eUxp
bmtSb2xlIH0sCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>