<?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>30964</bug_id>
          
          <creation_ts>2009-10-30 14:30:10 -0700</creation_ts>
          <short_desc>[Gtk] Implement AtkDocument</short_desc>
          <delta_ts>2009-11-01 13:57:35 -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>Accessibility</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</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>
          
          <blocked>25531</blocked>
    
    <blocked>25831</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Joanmarie Diggs">jdiggs</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>apinheiro</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>jmalonzo</cc>
    
    <cc>walker.willie</cc>
    
    <cc>xan.lopez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>159446</commentid>
    <comment_count>0</comment_count>
    <who name="Joanmarie Diggs">jdiggs</who>
    <bug_when>2009-10-30 14:30:10 -0700</bug_when>
    <thetext>We need to implement the AtkDocument interface.
http://library.gnome.org/devel/atk/unstable/AtkDocument.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159551</commentid>
    <comment_count>1</comment_count>
      <attachid>42252</attachid>
    <who name="Joanmarie Diggs">jdiggs</who>
    <bug_when>2009-10-31 05:11:24 -0700</bug_when>
    <thetext>Created attachment 42252
Part 1 - bare bones implementation</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159552</commentid>
    <comment_count>2</comment_count>
      <attachid>42253</attachid>
    <who name="Joanmarie Diggs">jdiggs</who>
    <bug_when>2009-10-31 05:12:29 -0700</bug_when>
    <thetext>Created attachment 42253
Part 2 - Attributes (requires part 1)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159553</commentid>
    <comment_count>3</comment_count>
      <attachid>42254</attachid>
    <who name="Joanmarie Diggs">jdiggs</who>
    <bug_when>2009-10-31 05:14:39 -0700</bug_when>
    <thetext>Created attachment 42254
Part 3 - language/locale (requires part 1)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159586</commentid>
    <comment_count>4</comment_count>
      <attachid>42252</attachid>
    <who name="Jan Alonzo">jmalonzo</who>
    <bug_when>2009-10-31 15:38:25 -0700</bug_when>
    <thetext>Comment on attachment 42252
Part 1 - bare bones implementation

Looks fine. r=me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159588</commentid>
    <comment_count>5</comment_count>
    <who name="Jan Alonzo">jmalonzo</who>
    <bug_when>2009-10-31 15:40:34 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; Created an attachment (id=42253) [details]
&gt; Part 2 - Attributes (requires part 1)

I think we should start writing layout tests for these. Would that be possible today or there&apos;s a plan to write layout tests for these changes soon?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159592</commentid>
    <comment_count>6</comment_count>
    <who name="Joanmarie Diggs">jdiggs</who>
    <bug_when>2009-10-31 15:52:29 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; (In reply to comment #2)
&gt; &gt; Created an attachment (id=42253) [details] [details]
&gt; &gt; Part 2 - Attributes (requires part 1)
&gt; 
&gt; I think we should start writing layout tests for these. Would that be possible
&gt; today or there&apos;s a plan to write layout tests for these changes soon?

Learning about (and then writing) layout tests was on my to-do list, but not my today-/this-weekend list. :-) However, I will re-adjust my list. Last time I looked at merely running the tests (once they were running) I kept getting failures, so I&apos;ve got a few things to work on before I can start generating tests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159597</commentid>
    <comment_count>7</comment_count>
    <who name="Xan Lopez">xan.lopez</who>
    <bug_when>2009-10-31 16:06:12 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; (In reply to comment #2)
&gt; &gt; Created an attachment (id=42253) [details] [details]
&gt; &gt; Part 2 - Attributes (requires part 1)
&gt; 
&gt; I think we should start writing layout tests for these. Would that be possible
&gt; today or there&apos;s a plan to write layout tests for these changes soon?

Mmm, do you mean unit tests here? Or how could we test ATK APIs from layout tests in JavaScript?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159598</commentid>
    <comment_count>8</comment_count>
    <who name="Jan Alonzo">jmalonzo</who>
    <bug_when>2009-10-31 16:20:12 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; (In reply to comment #5)
&gt; &gt; (In reply to comment #2)
&gt; &gt; &gt; Created an attachment (id=42253) [details] [details] [details]
&gt; &gt; &gt; Part 2 - Attributes (requires part 1)
&gt; &gt; 
&gt; &gt; I think we should start writing layout tests for these. Would that be possible
&gt; &gt; today or there&apos;s a plan to write layout tests for these changes soon?
&gt; 
&gt; Mmm, do you mean unit tests here? Or how could we test ATK APIs from layout
&gt; tests in JavaScript?

Layout tests. We&apos;re not testing ATK APIs but testing the accessible characteristics of AX objects. AccessibilityController exposes some of this attributes that we can test in JavaScript. It&apos;s still missing a few implementations for Gtk though, but shouldn&apos;t be difficult to implement those.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159616</commentid>
    <comment_count>9</comment_count>
    <who name="Xan Lopez">xan.lopez</who>
    <bug_when>2009-11-01 02:13:14 -0800</bug_when>
    <thetext>(In reply to comment #8)
&gt; &gt; Mmm, do you mean unit tests here? Or how could we test ATK APIs from layout
&gt; &gt; tests in JavaScript?
&gt; 
&gt; Layout tests. We&apos;re not testing ATK APIs but testing the accessible
&gt; characteristics of AX objects. AccessibilityController exposes some of this
&gt; attributes that we can test in JavaScript. It&apos;s still missing a few
&gt; implementations for Gtk though, but shouldn&apos;t be difficult to implement those.

OK. Since most of what we are doing here is implementing the ATK APIs I assumed you were referring to that. In any case, testing that all those APIs work as expected is, at least, as important as doing layout tests for any core AX functionality we are using or adding.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159657</commentid>
    <comment_count>10</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-11-01 13:19:08 -0800</bug_when>
    <thetext>bugzilla-tool is not smart enough to leave bugs open after landing due to bug 28230.  So this bug will need to be re-opened after the commit-queue lands patch #1.  Either someone (which can be me), needs to fix bug 28230, or better yet, we should follow the one-change-per-bug style that is more common in WebKit. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159658</commentid>
    <comment_count>11</comment_count>
      <attachid>42252</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2009-11-01 13:22:42 -0800</bug_when>
    <thetext>Comment on attachment 42252
Part 1 - bare bones implementation

Clearing flags on attachment: 42252

Committed r50393: &lt;http://trac.webkit.org/changeset/50393&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159659</commentid>
    <comment_count>12</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2009-11-01 13:22:50 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159665</commentid>
    <comment_count>13</comment_count>
      <attachid>42253</attachid>
    <who name="Joanmarie Diggs">jdiggs</who>
    <bug_when>2009-11-01 13:41:31 -0800</bug_when>
    <thetext>Comment on attachment 42253
Part 2 - Attributes (requires part 1)

Obsoleting this patch and clearing the review flag.

Bug 30997 has been spun off so that we only have one change per bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159667</commentid>
    <comment_count>14</comment_count>
      <attachid>42254</attachid>
    <who name="Joanmarie Diggs">jdiggs</who>
    <bug_when>2009-11-01 13:44:06 -0800</bug_when>
    <thetext>Comment on attachment 42254
Part 3 - language/locale (requires part 1)

Obsoleting this patch and clearing the review flag.

Bug 30998 has been spun off so that we only have one change per bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159669</commentid>
    <comment_count>15</comment_count>
    <who name="Joanmarie Diggs">jdiggs</who>
    <bug_when>2009-11-01 13:51:36 -0800</bug_when>
    <thetext>(In reply to comment #10)
&gt; bugzilla-tool is not smart enough to leave bugs open after landing due to bug
&gt; 28230.  So this bug will need to be re-opened after the commit-queue lands
&gt; patch #1.  Either someone (which can be me), needs to fix bug 28230, or better
&gt; yet, we should follow the one-change-per-bug style that is more common in
&gt; WebKit. :)

Sorry. New bugs spun off for each patch.

Should I also file new bugs for the tests which I will (hopefully) write for each of those bugs, or do tests associated with a fix constitute part of a single change?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>159671</commentid>
    <comment_count>16</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-11-01 13:57:35 -0800</bug_when>
    <thetext>(In reply to comment #15)
&gt; (In reply to comment #10)
&gt; &gt; bugzilla-tool is not smart enough to leave bugs open after landing due to bug
&gt; &gt; 28230.  So this bug will need to be re-opened after the commit-queue lands
&gt; &gt; patch #1.  Either someone (which can be me), needs to fix bug 28230, or better
&gt; &gt; yet, we should follow the one-change-per-bug style that is more common in
&gt; &gt; WebKit. :)
&gt; 
&gt; Sorry. New bugs spun off for each patch.
&gt; 
&gt; Should I also file new bugs for the tests which I will (hopefully) write for
&gt; each of those bugs, or do tests associated with a fix constitute part of a
&gt; single change?

Certainly nothing to apologize for!  :)   I just wanted you to know that our tools don&apos;t handle multiple-patches-per-bug very well.

Tests should go along with fixes.  When I review patches often seeing and understanding what the tests are doing is more important than the code itself.  If we have a test, we&apos;ll never break the code in the same way again. :)</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>42252</attachid>
            <date>2009-10-31 05:11:24 -0700</date>
            <delta_ts>2009-11-01 13:22:42 -0800</delta_ts>
            <desc>Part 1 - bare bones implementation</desc>
            <filename>document-iface-1.patch</filename>
            <type>text/plain</type>
            <size>4642</size>
            <attacher name="Joanmarie Diggs">jdiggs</attacher>
            
              <data encoding="base64">RnJvbSAyNTZjY2ZlNjE3YWM4ZTliYjM4Yjc2ZmYyMzk0YmJkZjJmOGEyNzBmIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBKb2FubWFyaWUgRGlnZ3MgPGpvYW5tYXJpZS5kaWdnc0BnbWFp
bC5jb20+CkRhdGU6IEZyaSwgMzAgT2N0IDIwMDkgMTg6MTg6MzUgLTA0MDAKU3ViamVjdDogW1BB
VENIIDEvM10gUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCgpodHRwczovL2J1Z3Mud2Via2l0
Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzA5NjQKW0d0a10gSW1wbGVtbWVudCBBdGtEb2N1bWVudAoK
SW1wbGVtZW50cyB3aGF0IGhhcyBiZWVuIGltcGxlbWVudGVkIGluIEFULVNQSQoKKiBhY2Nlc3Np
YmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHA6CihHZXRBdGtJbnRl
cmZhY2VUeXBlRnJvbVdBSVR5cGUpOgooZ2V0SW50ZXJmYWNlTWFza0Zyb21PYmplY3QpOgooYXRr
X2RvY3VtZW50X2ludGVyZmFjZV9pbml0KToKKHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dl
dF9kb2N1bWVudF9hdHRyaWJ1dGVfdmFsdWUpOgood2Via2l0X2FjY2Vzc2libGVfZG9jdW1lbnRf
Z2V0X2RvY3VtZW50X2F0dHJpYnV0ZXMpOgood2Via2l0X2FjY2Vzc2libGVfZG9jdW1lbnRfZ2V0
X2xvY2FsZSk6Ci0tLQogV2ViQ29yZS9DaGFuZ2VMb2cgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgfCAgIDE3ICsrKysrKysrCiAuLi4vZ3RrL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFw
cGVyQXRrLmNwcCAgICAgICAgICB8ICAgNDQgKysrKysrKysrKysrKysrKysrKy0KIDIgZmlsZXMg
Y2hhbmdlZCwgNjAgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbnMoLSkKCmRpZmYgLS1naXQgYS9X
ZWJDb3JlL0NoYW5nZUxvZyBiL1dlYkNvcmUvQ2hhbmdlTG9nCmluZGV4IDlkMWY4YmMuLjczOGU5
MzcgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1dlYkNvcmUvQ2hhbmdlTG9n
CkBAIC0xLDMgKzEsMjAgQEAKKzIwMDktMTAtMzAgIEpvYW5tYXJpZSBEaWdncyAgPGpvYW5tYXJp
ZS5kaWdnc0BnbWFpbC5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTMwOTY0
CisgICAgICAgIFtHdGtdIEltcGxlbW1lbnQgQXRrRG9jdW1lbnQKKworICAgICAgICBJbXBsZW1l
bnRzIHdoYXQgaGFzIGJlZW4gaW1wbGVtZW50ZWQgaW4gQVQtU1BJLgorCisgICAgICAgICogYWNj
ZXNzaWJpbGl0eS9ndGsvQWNjZXNzaWJpbGl0eU9iamVjdFdyYXBwZXJBdGsuY3BwOgorICAgICAg
ICAoR2V0QXRrSW50ZXJmYWNlVHlwZUZyb21XQUlUeXBlKToKKyAgICAgICAgKGdldEludGVyZmFj
ZU1hc2tGcm9tT2JqZWN0KToKKyAgICAgICAgKGF0a19kb2N1bWVudF9pbnRlcmZhY2VfaW5pdCk6
CisgICAgICAgICh3ZWJraXRfYWNjZXNzaWJsZV9kb2N1bWVudF9nZXRfZG9jdW1lbnRfYXR0cmli
dXRlX3ZhbHVlKToKKyAgICAgICAgKHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dldF9kb2N1
bWVudF9hdHRyaWJ1dGVzKToKKyAgICAgICAgKHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dl
dF9sb2NhbGUpOgorCiAyMDA5LTEwLTMwICBEbWl0cnkgVGl0b3YgIDxkaW1pY2hAY2hyb21pdW0u
b3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERpbWl0cmkgR2xhemtvdi4KZGlmZiAtLWdpdCBh
L1dlYkNvcmUvYWNjZXNzaWJpbGl0eS9ndGsvQWNjZXNzaWJpbGl0eU9iamVjdFdyYXBwZXJBdGsu
Y3BwIGIvV2ViQ29yZS9hY2Nlc3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBl
ckF0ay5jcHAKaW5kZXggMmVhODMzNi4uODE5NTkyNiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9hY2Nl
c3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHAKKysrIGIvV2Vi
Q29yZS9hY2Nlc3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHAK
QEAgLTEzNyw2ICsxMzcsMTEgQEAgc3RhdGljIEFjY2Vzc2liaWxpdHlPYmplY3QqIGNvcmUoQXRr
VGFibGUqIHRhYmxlKQogICAgIHJldHVybiBjb3JlKEFUS19PQkpFQ1QodGFibGUpKTsKIH0KIAor
c3RhdGljIEFjY2Vzc2liaWxpdHlPYmplY3QqIGNvcmUoQXRrRG9jdW1lbnQqIGRvY3VtZW50KQor
eworICAgIHJldHVybiBjb3JlKEFUS19PQkpFQ1QoZG9jdW1lbnQpKTsKK30KKwogc3RhdGljIGNv
bnN0IGdjaGFyKiBuYW1lRnJvbUNoaWxkcmVuKEFjY2Vzc2liaWxpdHlPYmplY3QqIG9iamVjdCkK
IHsKICAgICBpZiAoIW9iamVjdCkKQEAgLTE0MDYsNiArMTQxMSwzNCBAQCBzdGF0aWMgdm9pZCBh
dGtfdGFibGVfaW50ZXJmYWNlX2luaXQoQXRrVGFibGVJZmFjZSogaWZhY2UpCiAgICAgaWZhY2Ut
PmdldF9yb3dfZGVzY3JpcHRpb24gPSB3ZWJraXRfYWNjZXNzaWJsZV90YWJsZV9nZXRfcm93X2Rl
c2NyaXB0aW9uOwogfQogCitzdGF0aWMgY29uc3QgZ2NoYXIqIHdlYmtpdF9hY2Nlc3NpYmxlX2Rv
Y3VtZW50X2dldF9hdHRyaWJ1dGVfdmFsdWUoQXRrRG9jdW1lbnQqIGRvY3VtZW50LCBjb25zdCBn
Y2hhciogYXR0cmlidXRlKQoreworICAgIC8vIEZJWE1FOiBUaGlzIG5lZWRzIHRvIGJlIGltcGxl
bWVudGVkLgorICAgIG5vdEltcGxlbWVudGVkKCk7CisgICAgcmV0dXJuIDA7Cit9CisKK3N0YXRp
YyBBdGtBdHRyaWJ1dGVTZXQqIHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dldF9hdHRyaWJ1
dGVzKEF0a0RvY3VtZW50KiBkb2N1bWVudCkKK3sKKyAgICAvLyBGSVhNRTogVGhpcyBuZWVkcyB0
byBiZSBpbXBsZW1lbnRlZC4KKyAgICBub3RJbXBsZW1lbnRlZCgpOworICAgIHJldHVybiAwOwor
fQorCitzdGF0aWMgY29uc3QgZ2NoYXIqIHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dldF9s
b2NhbGUoQXRrRG9jdW1lbnQqIGRvY3VtZW50KQoreworICAgIC8vIEZJWE1FOiBUaGlzIG5lZWRz
IHRvIGJlIGltcGxlbWVudGVkLgorICAgIG5vdEltcGxlbWVudGVkKCk7CisgICAgcmV0dXJuIDA7
Cit9CisKK3N0YXRpYyB2b2lkIGF0a19kb2N1bWVudF9pbnRlcmZhY2VfaW5pdChBdGtEb2N1bWVu
dElmYWNlKiBpZmFjZSkKK3sKKyAgICBpZmFjZS0+Z2V0X2RvY3VtZW50X2F0dHJpYnV0ZV92YWx1
ZSA9IHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dldF9hdHRyaWJ1dGVfdmFsdWU7CisgICAg
aWZhY2UtPmdldF9kb2N1bWVudF9hdHRyaWJ1dGVzID0gd2Via2l0X2FjY2Vzc2libGVfZG9jdW1l
bnRfZ2V0X2F0dHJpYnV0ZXM7CisgICAgaWZhY2UtPmdldF9kb2N1bWVudF9sb2NhbGUgPSB3ZWJr
aXRfYWNjZXNzaWJsZV9kb2N1bWVudF9nZXRfbG9jYWxlOworfQorCiBzdGF0aWMgY29uc3QgR0lu
dGVyZmFjZUluZm8gQXRrSW50ZXJmYWNlc0luaXRGdW5jdGlvbnNbXSA9IHsKICAgICB7KEdJbnRl
cmZhY2VJbml0RnVuYylhdGtfYWN0aW9uX2ludGVyZmFjZV9pbml0LAogICAgICAoR0ludGVyZmFj
ZUZpbmFsaXplRnVuYykgTlVMTCwgTlVMTH0sCkBAIC0xNDIwLDYgKzE0NTMsOCBAQCBzdGF0aWMg
Y29uc3QgR0ludGVyZmFjZUluZm8gQXRrSW50ZXJmYWNlc0luaXRGdW5jdGlvbnNbXSA9IHsKICAg
ICB7KEdJbnRlcmZhY2VJbml0RnVuYylhdGtfaW1hZ2VfaW50ZXJmYWNlX2luaXQsCiAgICAgIChH
SW50ZXJmYWNlRmluYWxpemVGdW5jKSBOVUxMLCBOVUxMfSwKICAgICB7KEdJbnRlcmZhY2VJbml0
RnVuYylhdGtfdGFibGVfaW50ZXJmYWNlX2luaXQsCisgICAgIChHSW50ZXJmYWNlRmluYWxpemVG
dW5jKSBOVUxMLCBOVUxMfSwKKyAgICB7KEdJbnRlcmZhY2VJbml0RnVuYylhdGtfZG9jdW1lbnRf
aW50ZXJmYWNlX2luaXQsCiAgICAgIChHSW50ZXJmYWNlRmluYWxpemVGdW5jKSBOVUxMLCBOVUxM
fQogfTsKIApAQCAtMTQzMCw3ICsxNDY1LDggQEAgZW51bSBXQUlUeXBlIHsKICAgICBXQUlfVEVY
VCwKICAgICBXQUlfQ09NUE9ORU5ULAogICAgIFdBSV9JTUFHRSwKLSAgICBXQUlfVEFCTEUKKyAg
ICBXQUlfVEFCTEUsCisgICAgV0FJX0RPQ1VNRU5UCiB9OwogCiBzdGF0aWMgR1R5cGUgR2V0QXRr
SW50ZXJmYWNlVHlwZUZyb21XQUlUeXBlKFdBSVR5cGUgdHlwZSkKQEAgLTE0NTAsNiArMTQ4Niw4
IEBAIHN0YXRpYyBHVHlwZSBHZXRBdGtJbnRlcmZhY2VUeXBlRnJvbVdBSVR5cGUoV0FJVHlwZSB0
eXBlKQogICAgICAgcmV0dXJuIEFUS19UWVBFX0lNQUdFOwogICBjYXNlIFdBSV9UQUJMRToKICAg
ICAgIHJldHVybiBBVEtfVFlQRV9UQUJMRTsKKyAgY2FzZSBXQUlfRE9DVU1FTlQ6CisgICAgICBy
ZXR1cm4gQVRLX1RZUEVfRE9DVU1FTlQ7CiAgIH0KIAogICByZXR1cm4gR19UWVBFX0lOVkFMSUQ7
CkBAIC0xNDg5LDYgKzE1MjcsMTAgQEAgc3RhdGljIGd1aW50MTYgZ2V0SW50ZXJmYWNlTWFza0Zy
b21PYmplY3QoQWNjZXNzaWJpbGl0eU9iamVjdCogY29yZU9iamVjdCkKICAgICBpZiAocm9sZSA9
PSBUYWJsZVJvbGUpCiAgICAgICAgIGludGVyZmFjZU1hc2sgfD0gMSA8PCBXQUlfVEFCTEU7CiAK
KyAgICAvLyBEb2N1bWVudAorICAgIGlmIChyb2xlID09IFdlYkFyZWFSb2xlKQorICAgICAgICBp
bnRlcmZhY2VNYXNrIHw9IDEgPDwgV0FJX0RPQ1VNRU5UOworCiAgICAgcmV0dXJuIGludGVyZmFj
ZU1hc2s7CiB9CiAKLS0gCjEuNi4zLjMKCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>42253</attachid>
            <date>2009-10-31 05:12:29 -0700</date>
            <delta_ts>2009-11-01 13:41:31 -0800</delta_ts>
            <desc>Part 2 - Attributes (requires part 1)</desc>
            <filename>document-iface-2.patch</filename>
            <type>text/plain</type>
            <size>3895</size>
            <attacher name="Joanmarie Diggs">jdiggs</attacher>
            
              <data encoding="base64">RnJvbSBmNzZmNjYwYzkzNjk1N2RlNTMwNzA3OWFiYWUxMDQwMmMxMWJhOWRlIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBKb2FubWFyaWUgRGlnZ3MgPGpvYW5tYXJpZS5kaWdnc0BnbWFp
bC5jb20+CkRhdGU6IFNhdCwgMzEgT2N0IDIwMDkgMDE6NTI6NTMgLTA0MDAKU3ViamVjdDogW1BB
VENIIDIvM10gUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCgpodHRwczovL2J1Z3Mud2Via2l0
Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzA5NjQKW0d0a10gSW1wbGVtbWVudCBBdGtEb2N1bWVudAoK
UHJvdmlkZXMgYWNjZXNzIHRvIGV4cGVjdGVkIGRvY3VtZW50IGF0dHJpYnV0ZXMuCgoqIGFjY2Vz
c2liaWxpdHkvZ3RrL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVyQXRrLmNwcDoKKGRvY3VtZW50
QXR0cmlidXRlVmFsdWUpOgood2Via2l0X2FjY2Vzc2libGVfZG9jdW1lbnRfZ2V0X2RvY3VtZW50
X2F0dHJpYnV0ZV92YWx1ZSk6Cih3ZWJraXRfYWNjZXNzaWJsZV9kb2N1bWVudF9nZXRfZG9jdW1l
bnRfYXR0cmlidXRlcyk6Ci0tLQogV2ViQ29yZS9DaGFuZ2VMb2cgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgfCAgIDE0ICsrKysrKysKIC4uLi9ndGsvQWNjZXNzaWJpbGl0eU9iamVj
dFdyYXBwZXJBdGsuY3BwICAgICAgICAgIHwgICAzNyArKysrKysrKysrKysrKysrLS0tCiAyIGZp
bGVzIGNoYW5nZWQsIDQ1IGluc2VydGlvbnMoKyksIDYgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0
IGEvV2ViQ29yZS9DaGFuZ2VMb2cgYi9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCA3MzhlOTM3Li5l
MTRjOTRiIDEwMDY0NAotLS0gYS9XZWJDb3JlL0NoYW5nZUxvZworKysgYi9XZWJDb3JlL0NoYW5n
ZUxvZwpAQCAtNSw2ICs1LDIwIEBACiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD0zMDk2NAogICAgICAgICBbR3RrXSBJbXBsZW1tZW50IEF0a0RvY3VtZW50
CiAKKyAgICAgICAgUHJvdmlkZXMgYWNjZXNzIHRvIGV4cGVjdGVkIGRvY3VtZW50IGF0dHJpYnV0
ZXMuCisKKyAgICAgICAgKiBhY2Nlc3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3Jh
cHBlckF0ay5jcHA6CisgICAgICAgIChkb2N1bWVudEF0dHJpYnV0ZVZhbHVlKToKKyAgICAgICAg
KHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dldF9kb2N1bWVudF9hdHRyaWJ1dGVfdmFsdWUp
OgorICAgICAgICAod2Via2l0X2FjY2Vzc2libGVfZG9jdW1lbnRfZ2V0X2RvY3VtZW50X2F0dHJp
YnV0ZXMpOgorCisyMDA5LTEwLTMwICBKb2FubWFyaWUgRGlnZ3MgIDxqb2FubWFyaWUuZGlnZ3NA
Z21haWwuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zMDk2NAorICAgICAg
ICBbR3RrXSBJbXBsZW1tZW50IEF0a0RvY3VtZW50CisKICAgICAgICAgSW1wbGVtZW50cyB3aGF0
IGhhcyBiZWVuIGltcGxlbWVudGVkIGluIEFULVNQSS4KIAogICAgICAgICAqIGFjY2Vzc2liaWxp
dHkvZ3RrL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVyQXRrLmNwcDoKZGlmZiAtLWdpdCBhL1dl
YkNvcmUvYWNjZXNzaWJpbGl0eS9ndGsvQWNjZXNzaWJpbGl0eU9iamVjdFdyYXBwZXJBdGsuY3Bw
IGIvV2ViQ29yZS9hY2Nlc3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0
ay5jcHAKaW5kZXggODE5NTkyNi4uOTdkY2QwYiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9hY2Nlc3Np
YmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHAKKysrIGIvV2ViQ29y
ZS9hY2Nlc3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHAKQEAg
LTQ0LDYgKzQ0LDcgQEAKICNpbmNsdWRlICJBdG9taWNTdHJpbmcuaCIKICNpbmNsdWRlICJDU3Ry
aW5nLmgiCiAjaW5jbHVkZSAiRG9jdW1lbnQuaCIKKyNpbmNsdWRlICJEb2N1bWVudFR5cGUuaCIK
ICNpbmNsdWRlICJFZGl0b3IuaCIKICNpbmNsdWRlICJGcmFtZS5oIgogI2luY2x1ZGUgIkZyYW1l
Vmlldy5oIgpAQCAtMTQxMSwxOCArMTQxMiw0MiBAQCBzdGF0aWMgdm9pZCBhdGtfdGFibGVfaW50
ZXJmYWNlX2luaXQoQXRrVGFibGVJZmFjZSogaWZhY2UpCiAgICAgaWZhY2UtPmdldF9yb3dfZGVz
Y3JpcHRpb24gPSB3ZWJraXRfYWNjZXNzaWJsZV90YWJsZV9nZXRfcm93X2Rlc2NyaXB0aW9uOwog
fQogCi1zdGF0aWMgY29uc3QgZ2NoYXIqIHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dldF9h
dHRyaWJ1dGVfdmFsdWUoQXRrRG9jdW1lbnQqIGRvY3VtZW50LCBjb25zdCBnY2hhciogYXR0cmli
dXRlKQorc3RhdGljIGNvbnN0IGdjaGFyKiBkb2N1bWVudEF0dHJpYnV0ZVZhbHVlKEF0a0RvY3Vt
ZW50KiBkb2N1bWVudCwgY29uc3QgZ2NoYXIqIGF0dHJpYnV0ZSkKIHsKLSAgICAvLyBGSVhNRTog
VGhpcyBuZWVkcyB0byBiZSBpbXBsZW1lbnRlZC4KLSAgICBub3RJbXBsZW1lbnRlZCgpOworICAg
IERvY3VtZW50KiBjb3JlRG9jdW1lbnQgPSBjb3JlKGRvY3VtZW50KS0+ZG9jdW1lbnQoKTsKKyAg
ICBpZiAoIWNvcmVEb2N1bWVudCkKKyAgICAgICAgcmV0dXJuIDA7CisKKyAgICBTdHJpbmcgdmFs
dWUgPSBTdHJpbmcoKTsKKyAgICBpZiAoIWdfYXNjaWlfc3RyY2FzZWNtcChhdHRyaWJ1dGUsICJE
b2NUeXBlIikgJiYgY29yZURvY3VtZW50LT5kb2N0eXBlKCkpCisgICAgICAgIHZhbHVlID0gY29y
ZURvY3VtZW50LT5kb2N0eXBlKCktPm5hbWUoKTsKKyAgICBlbHNlIGlmICghZ19hc2NpaV9zdHJj
YXNlY21wKGF0dHJpYnV0ZSwgIkVuY29kaW5nIikpCisgICAgICAgIHZhbHVlID0gY29yZURvY3Vt
ZW50LT5jaGFyc2V0KCk7CisgICAgZWxzZSBpZiAoIWdfYXNjaWlfc3RyY2FzZWNtcChhdHRyaWJ1
dGUsICJVUkkiKSkKKyAgICAgICAgdmFsdWUgPSBjb3JlRG9jdW1lbnQtPmRvY3VtZW50VVJJKCk7
CisgICAgaWYgKCF2YWx1ZS5pc0VtcHR5KCkpCisgICAgICAgIHJldHVybiByZXR1cm5TdHJpbmco
dmFsdWUpOworCiAgICAgcmV0dXJuIDA7CiB9CiAKK3N0YXRpYyBjb25zdCBnY2hhciogd2Via2l0
X2FjY2Vzc2libGVfZG9jdW1lbnRfZ2V0X2F0dHJpYnV0ZV92YWx1ZShBdGtEb2N1bWVudCogZG9j
dW1lbnQsIGNvbnN0IGdjaGFyKiBhdHRyaWJ1dGUpCit7CisgICAgcmV0dXJuIGRvY3VtZW50QXR0
cmlidXRlVmFsdWUoZG9jdW1lbnQsIGF0dHJpYnV0ZSk7Cit9CisKIHN0YXRpYyBBdGtBdHRyaWJ1
dGVTZXQqIHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dldF9hdHRyaWJ1dGVzKEF0a0RvY3Vt
ZW50KiBkb2N1bWVudCkKIHsKLSAgICAvLyBGSVhNRTogVGhpcyBuZWVkcyB0byBiZSBpbXBsZW1l
bnRlZC4KLSAgICBub3RJbXBsZW1lbnRlZCgpOwotICAgIHJldHVybiAwOworICAgIEF0a0F0dHJp
YnV0ZVNldCogYXR0cmlidXRlU2V0ID0gMDsKKyAgICBjb25zdCBnY2hhciogYXR0cmlidXRlcyBb
XSA9IHsiRG9jVHlwZSIsICJFbmNvZGluZyIsICJVUkkifTsKKworICAgIGZvciAodW5zaWduZWQg
aSA9IDA7IGkgPCBHX05fRUxFTUVOVFMoYXR0cmlidXRlcyk7IGkrKykgeworICAgICAgICBjb25z
dCBnY2hhciogdmFsdWUgPSBkb2N1bWVudEF0dHJpYnV0ZVZhbHVlKGRvY3VtZW50LCBhdHRyaWJ1
dGVzW2ldKTsKKyAgICAgICAgaWYgKHZhbHVlKQorICAgICAgICAgICAgYXR0cmlidXRlU2V0ID0g
YWRkQXR0cmlidXRlVG9TZXQoYXR0cmlidXRlU2V0LCBhdHRyaWJ1dGVzW2ldLCB2YWx1ZSk7Cisg
ICAgfQorCisgICAgcmV0dXJuIGF0dHJpYnV0ZVNldDsKIH0KIAogc3RhdGljIGNvbnN0IGdjaGFy
KiB3ZWJraXRfYWNjZXNzaWJsZV9kb2N1bWVudF9nZXRfbG9jYWxlKEF0a0RvY3VtZW50KiBkb2N1
bWVudCkKLS0gCjEuNi4zLjMKCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>42254</attachid>
            <date>2009-10-31 05:14:39 -0700</date>
            <delta_ts>2009-11-01 13:44:17 -0800</delta_ts>
            <desc>Part 3 - language/locale (requires part 1)</desc>
            <filename>document-iface-3.patch</filename>
            <type>text/plain</type>
            <size>2083</size>
            <attacher name="Joanmarie Diggs">jdiggs</attacher>
            
              <data encoding="base64">RnJvbSBmNDAyZjI0MWRmMWQ2ZmUxOWU4MDhjZDg2M2RkNzFiMDU5YWNkNDhiIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBKb2FubWFyaWUgRGlnZ3MgPGpvYW5tYXJpZS5kaWdnc0BnbWFp
bC5jb20+CkRhdGU6IFNhdCwgMzEgT2N0IDIwMDkgMDg6MDI6MjUgLTA0MDAKU3ViamVjdDogW1BB
VENIIDMvM10gUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCgpodHRwczovL2J1Z3Mud2Via2l0
Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzA5NjQKW0d0a10gSW1wbGVtbWVudCBBdGtEb2N1bWVudAoK
UHJvdmlkZXMgYWNjZXNzIHRvIHRoZSByZXBvcnRlZCBjb250ZW50IGxhbmd1YWdlLgoKKiBhY2Nl
c3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHA6Cih3ZWJraXRf
YWNjZXNzaWJsZV9kb2N1bWVudF9nZXRfbG9jYWxlKToKLS0tCiBXZWJDb3JlL0NoYW5nZUxvZyAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB8ICAgMTIgKysrKysrKysrKysrCiAuLi4v
Z3RrL0FjY2Vzc2liaWxpdHlPYmplY3RXcmFwcGVyQXRrLmNwcCAgICAgICAgICB8ICAgIDggKysr
KysrLS0KIDIgZmlsZXMgY2hhbmdlZCwgMTggaW5zZXJ0aW9ucygrKSwgMiBkZWxldGlvbnMoLSkK
CmRpZmYgLS1naXQgYS9XZWJDb3JlL0NoYW5nZUxvZyBiL1dlYkNvcmUvQ2hhbmdlTG9nCmluZGV4
IGUxNGM5NGIuLjllOWY3YjYgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1dl
YkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMDktMTAtMzEgIEpvYW5tYXJpZSBE
aWdncyAgPGpvYW5tYXJpZS5kaWdnc0BnbWFpbC5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkg
Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTMwOTY0CisgICAgICAgIFtHdGtdIEltcGxlbW1lbnQgQXRrRG9jdW1lbnQKKwor
ICAgICAgICBQcm92aWRlcyBhY2Nlc3MgdG8gdGhlIHJlcG9ydGVkIGNvbnRlbnQgbGFuZ3VhZ2Uu
CisKKyAgICAgICAgKiBhY2Nlc3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBl
ckF0ay5jcHA6CisgICAgICAgICh3ZWJraXRfYWNjZXNzaWJsZV9kb2N1bWVudF9nZXRfbG9jYWxl
KToKKwogMjAwOS0xMC0zMCAgSm9hbm1hcmllIERpZ2dzICA8am9hbm1hcmllLmRpZ2dzQGdtYWls
LmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KZGlmZiAtLWdpdCBh
L1dlYkNvcmUvYWNjZXNzaWJpbGl0eS9ndGsvQWNjZXNzaWJpbGl0eU9iamVjdFdyYXBwZXJBdGsu
Y3BwIGIvV2ViQ29yZS9hY2Nlc3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBl
ckF0ay5jcHAKaW5kZXggOTdkY2QwYi4uODBiNWEzZCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9hY2Nl
c3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHAKKysrIGIvV2Vi
Q29yZS9hY2Nlc3NpYmlsaXR5L2d0ay9BY2Nlc3NpYmlsaXR5T2JqZWN0V3JhcHBlckF0ay5jcHAK
QEAgLTE0NTIsOCArMTQ1MiwxMiBAQCBzdGF0aWMgQXRrQXR0cmlidXRlU2V0KiB3ZWJraXRfYWNj
ZXNzaWJsZV9kb2N1bWVudF9nZXRfYXR0cmlidXRlcyhBdGtEb2N1bWVudCogZAogCiBzdGF0aWMg
Y29uc3QgZ2NoYXIqIHdlYmtpdF9hY2Nlc3NpYmxlX2RvY3VtZW50X2dldF9sb2NhbGUoQXRrRG9j
dW1lbnQqIGRvY3VtZW50KQogewotICAgIC8vIEZJWE1FOiBUaGlzIG5lZWRzIHRvIGJlIGltcGxl
bWVudGVkLgotICAgIG5vdEltcGxlbWVudGVkKCk7CisKKyAgICAvLyBUT0RPOiBTaG91bGQgd2Ug
ZmFsbCBiYWNrIG9uIGxhbmcgeG1sOmxhbmcgd2hlbiB0aGUgZm9sbG93aW5nIGNvbWVzIHVwIGVt
cHR5PworICAgIFN0cmluZyBsYW5ndWFnZSA9IHN0YXRpY19jYXN0PEFjY2Vzc2liaWxpdHlSZW5k
ZXJPYmplY3QqPihjb3JlKGRvY3VtZW50KSktPmxhbmd1YWdlKCk7CisgICAgaWYgKCFsYW5ndWFn
ZS5pc0VtcHR5KCkpCisgICAgICAgIHJldHVybiByZXR1cm5TdHJpbmcobGFuZ3VhZ2UpOworCiAg
ICAgcmV0dXJuIDA7CiB9CiAKLS0gCjEuNi4zLjMKCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>