<?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>91911</bug_id>
          
          <creation_ts>2012-07-20 17:16:41 -0700</creation_ts>
          <short_desc>WebKit should expose @title as label (AXTitle or AXDescription) sometimes instead of AXHelp, according to the ARIA text alt computation</short_desc>
          <delta_ts>2012-07-27 10:39:43 -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>Accessibility</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="chris fleizach">cfleizach</reporter>
          <assigned_to name="chris fleizach">cfleizach</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>675105</commentid>
    <comment_count>0</comment_count>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2012-07-20 17:16:41 -0700</bug_when>
    <thetext>WebKit should expose @title as label (AXTitle or AXDescription) sometimes instead of AXHelp, according to the ARIA text alt computation

http://www.w3.org/WAI/PF/aria/complete#textalternativecomputation

Logic should be:
if AXDescription &amp;&amp; AXTitle == empty, then 
	use title attribute as AXDescription
	do not return title attribute as AXHelp
For AXHelp
	- use aria-help first, then use title attribute, unless being used for description</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675107</commentid>
    <comment_count>1</comment_count>
      <attachid>153629</attachid>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2012-07-20 17:18:02 -0700</bug_when>
    <thetext>Created attachment 153629
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>679474</commentid>
    <comment_count>2</comment_count>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2012-07-26 10:24:10 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/123767</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>679902</commentid>
    <comment_count>3</comment_count>
      <attachid>153629</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2012-07-26 16:18:51 -0700</bug_when>
    <thetext>Comment on attachment 153629
patch

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

&gt; Source/WebCore/accessibility/AccessibilityRenderObject.cpp:1032
&gt; +    String descriptiveText = accessibilityDescription();

If the function is named description, then I suggest naming the local variable description too. If they should be named descriptive text, then both should use that phrase instead of description.

&gt; Source/WebCore/accessibility/AccessibilityRenderObject.cpp:1039
&gt; +            // The title attribute should be used as help text, unless it is already being used as descriptive text.

I think you should omit the comma.

This comment says what the code does, but not why.

&gt; Source/WebCore/accessibility/AccessibilityRenderObject.cpp:1501
&gt; +    // The description should fall back to the title attribute as a last resort.
&gt; +    if (title().isEmpty())
&gt; +        return getAttribute(titleAttr);
&gt; +    
&gt;      return String();

I don’t understand the if statement. If title() returns a non-empty string, we return a null accessibility description. Why is that what we want? Probably needs a why comment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>679933</commentid>
    <comment_count>4</comment_count>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2012-07-26 17:11:04 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; (From update of attachment 153629 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=153629&amp;action=review
&gt; 
&gt; &gt; Source/WebCore/accessibility/AccessibilityRenderObject.cpp:1032
&gt; &gt; +    String descriptiveText = accessibilityDescription();
&gt; 
&gt; If the function is named description, then I suggest naming the local variable description too. If they should be named descriptive text, then both should use that phrase instead of description.
&gt; 
&gt; &gt; Source/WebCore/accessibility/AccessibilityRenderObject.cpp:1039
&gt; &gt; +            // The title attribute should be used as help text, unless it is already being used as descriptive text.
&gt; 
&gt; I think you should omit the comma.
&gt; 
&gt; This comment says what the code does, but not why.
&gt; 
&gt; &gt; Source/WebCore/accessibility/AccessibilityRenderObject.cpp:1501
&gt; &gt; +    // The description should fall back to the title attribute as a last resort.
&gt; &gt; +    if (title().isEmpty())
&gt; &gt; +        return getAttribute(titleAttr);
&gt; &gt; +    
&gt; &gt;      return String();
&gt; 
&gt; I don’t understand the if statement. If title() returns a non-empty string, we return a null accessibility description. Why is that what we want? Probably needs a why comment.

Thanks for the feedback. I will make all these changes. As to your final question, the comment I will add to explain it is the following: 

+    // An elements descriptive text is comprised of title() (what&apos;s visible on the screen) and accessibilityDescription() (other descriptive text).
+    // Both are used to generate what a screen reader speaks.
+    // If this point is reached (i.e. there&apos;s no accessibilityDescription) and there&apos;s no title(), we should fallback to using the title attribute.
+    // The title attribute is normally used as help text (because it is a tooltip), but if there is nothing else available, this should be used (according to ARIA).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>680789</commentid>
    <comment_count>5</comment_count>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2012-07-27 10:39:43 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/123884
in response to Darin&apos;s comments</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>153629</attachid>
            <date>2012-07-20 17:18:02 -0700</date>
            <delta_ts>2012-07-26 16:18:51 -0700</delta_ts>
            <desc>patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>7544</size>
            <attacher name="chris fleizach">cfleizach</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEyMzI3NykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE4IEBACisyMDEyLTA3LTIwICBDaHJpcyBG
bGVpemFjaCAgPGNmbGVpemFjaEBhcHBsZS5jb20+CisKKyAgICAgICAgV2ViS2l0IHNob3VsZCBl
eHBvc2UgQHRpdGxlIGFzIGxhYmVsIChBWFRpdGxlIG9yIEFYRGVzY3JpcHRpb24pIHNvbWV0aW1l
cyBpbnN0ZWFkIG9mIEFYSGVscCwgYWNjb3JkaW5nIHRvIHRoZSBBUklBIHRleHQgYWx0IGNvbXB1
dGF0aW9uCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05
MTkxMQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRo
ZSBBUklBIHNwZWMgc3RhdGVzIHRoYXQgdGhlIHRpdGxlIGF0dHJpYnV0ZSBzaG91bGQgYmUgdXNl
ZCBhcyB0aGUgbGFzdCByZXNvcnQgZm9yIGEgZGVzY3JpcHRpb24gb2YgYW4gZWxlbWVudC4gV2Vi
S2l0IG5lZWRzIHRvIGhvbm9yIHRoYXQuCisKKyAgICAgICAgVGVzdDogcGxhdGZvcm0vbWFjL2Fj
Y2Vzc2liaWxpdHkvdGl0bGUtYXR0cmlidXRlLXVzZWQtYXMtbGFzdC1yZXNvcnQuaHRtbAorCisg
ICAgICAgICogYWNjZXNzaWJpbGl0eS9BY2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0LmNwcDoKKyAg
ICAgICAgKFdlYkNvcmU6OkFjY2Vzc2liaWxpdHlSZW5kZXJPYmplY3Q6OmhlbHBUZXh0KToKKyAg
ICAgICAgKFdlYkNvcmU6OkFjY2Vzc2liaWxpdHlSZW5kZXJPYmplY3Q6OmFjY2Vzc2liaWxpdHlE
ZXNjcmlwdGlvbik6CisKIDIwMTItMDctMjAgIEpvc2h1YSBCZWxsICA8anNiZWxsQGNocm9taXVt
Lm9yZz4KIAogICAgICAgICBJbmRleGVkREI6IFNpbXBsaWZ5IGJhY2tlbmQgaW50ZXJmYWNlIGNs
YXNzZXMKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL2FjY2Vzc2liaWxpdHkvQWNjZXNzaWJpbGl0eVJl
bmRlck9iamVjdC5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvYWNjZXNzaWJpbGl0
eS9BY2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0LmNwcAkocmV2aXNpb24gMTIzMjIyKQorKysgU291
cmNlL1dlYkNvcmUvYWNjZXNzaWJpbGl0eS9BY2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0LmNwcAko
d29ya2luZyBjb3B5KQpAQCAtMTAyOSwxMyArMTAyOSwxNiBAQAogICAgIGlmICghZGVzY3JpYmVk
QnkuaXNFbXB0eSgpKQogICAgICAgICByZXR1cm4gZGVzY3JpYmVkQnk7CiAgICAgCisgICAgU3Ry
aW5nIGRlc2NyaXB0aXZlVGV4dCA9IGFjY2Vzc2liaWxpdHlEZXNjcmlwdGlvbigpOwogICAgIGZv
ciAoUmVuZGVyT2JqZWN0KiBjdXJyID0gbV9yZW5kZXJlcjsgY3VycjsgY3VyciA9IGN1cnItPnBh
cmVudCgpKSB7CiAgICAgICAgIGlmIChjdXJyLT5ub2RlKCkgJiYgY3Vyci0+bm9kZSgpLT5pc0hU
TUxFbGVtZW50KCkpIHsKICAgICAgICAgICAgIGNvbnN0IEF0b21pY1N0cmluZyYgc3VtbWFyeSA9
IHN0YXRpY19jYXN0PEVsZW1lbnQqPihjdXJyLT5ub2RlKCkpLT5nZXRBdHRyaWJ1dGUoc3VtbWFy
eUF0dHIpOwogICAgICAgICAgICAgaWYgKCFzdW1tYXJ5LmlzRW1wdHkoKSkKICAgICAgICAgICAg
ICAgICByZXR1cm4gc3VtbWFyeTsKKyAgICAgICAgICAgIAorICAgICAgICAgICAgLy8gVGhlIHRp
dGxlIGF0dHJpYnV0ZSBzaG91bGQgYmUgdXNlZCBhcyBoZWxwIHRleHQsIHVubGVzcyBpdCBpcyBh
bHJlYWR5IGJlaW5nIHVzZWQgYXMgZGVzY3JpcHRpdmUgdGV4dC4KICAgICAgICAgICAgIGNvbnN0
IEF0b21pY1N0cmluZyYgdGl0bGUgPSBzdGF0aWNfY2FzdDxFbGVtZW50Kj4oY3Vyci0+bm9kZSgp
KS0+Z2V0QXR0cmlidXRlKHRpdGxlQXR0cik7Ci0gICAgICAgICAgICBpZiAoIXRpdGxlLmlzRW1w
dHkoKSkKKyAgICAgICAgICAgIGlmICghdGl0bGUuaXNFbXB0eSgpICYmIGRlc2NyaXB0aXZlVGV4
dCAhPSB0aXRsZSkKICAgICAgICAgICAgICAgICByZXR1cm4gdGl0bGU7CiAgICAgICAgIH0KICAg
ICAgICAgCkBAIC0xNDkxLDYgKzE0OTQsMTAgQEAKICAgICBpZiAoaXNXZWJBcmVhKCkpCiAgICAg
ICAgIHJldHVybiB3ZWJBcmVhQWNjZXNzaWJpbGl0eURlc2NyaXB0aW9uKCk7CiAgICAgCisgICAg
Ly8gVGhlIGRlc2NyaXB0aW9uIHNob3VsZCBmYWxsIGJhY2sgdG8gdGhlIHRpdGxlIGF0dHJpYnV0
ZSBhcyBhIGxhc3QgcmVzb3J0LgorICAgIGlmICh0aXRsZSgpLmlzRW1wdHkoKSkKKyAgICAgICAg
cmV0dXJuIGdldEF0dHJpYnV0ZSh0aXRsZUF0dHIpOworICAgIAogICAgIHJldHVybiBTdHJpbmco
KTsKIH0KIApJbmRleDogTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91
dFRlc3RzL0NoYW5nZUxvZwkocmV2aXNpb24gMTIzMjc3KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTItMDctMjAgIENocmlzIEZs
ZWl6YWNoICA8Y2ZsZWl6YWNoQGFwcGxlLmNvbT4KKworICAgICAgICBXZWJLaXQgc2hvdWxkIGV4
cG9zZSBAdGl0bGUgYXMgbGFiZWwgKEFYVGl0bGUgb3IgQVhEZXNjcmlwdGlvbikgc29tZXRpbWVz
IGluc3RlYWQgb2YgQVhIZWxwLCBhY2NvcmRpbmcgdG8gdGhlIEFSSUEgdGV4dCBhbHQgY29tcHV0
YXRpb24KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTkx
OTExCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBw
bGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS90aXRsZS1hdHRyaWJ1dGUtdXNlZC1hcy1sYXN0LXJl
c29ydC1leHBlY3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAqIHBsYXRmb3JtL21hYy9hY2Nlc3Np
YmlsaXR5L3RpdGxlLWF0dHJpYnV0ZS11c2VkLWFzLWxhc3QtcmVzb3J0Lmh0bWw6IEFkZGVkLgor
CiAyMDEyLTA3LTIwICBDYWlvIE1hcmNlbG8gZGUgT2xpdmVpcmEgRmlsaG8gIDxjYWlvLm9saXZl
aXJhQG9wZW5ib3NzYS5vcmc+CiAKICAgICAgICAgW1F0XSBzdmcvdGV4dCByZWJhc2VsaW5lIGFm
dGVyIG5ldyB0ZXN0IGZvbnRzIChwYXJ0IDQpCkluZGV4OiBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9t
YWMvYWNjZXNzaWJpbGl0eS90aXRsZS1hdHRyaWJ1dGUtdXNlZC1hcy1sYXN0LXJlc29ydC1leHBl
Y3RlZC50eHQKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjL2FjY2Vzc2li
aWxpdHkvdGl0bGUtYXR0cmlidXRlLXVzZWQtYXMtbGFzdC1yZXNvcnQtZXhwZWN0ZWQudHh0CShy
ZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjL2FjY2Vzc2liaWxpdHkvdGl0
bGUtYXR0cmlidXRlLXVzZWQtYXMtbGFzdC1yZXNvcnQtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAw
KQpAQCAtMCwwICsxLDIwIEBACit0ZXh0IHRleHQKK1RoaXMgdGVzdHMgdGhhdCB0aGUgdGl0bGUg
YXR0cmlidXRlIGlzIHVzZWQgYXMgdGhlIGxhc3QgcmVzb3J0IGZvciBkZXNjcmlwdGlvbiB3aGVu
IHRoZXJlIGlzIG5vIG90aGVyIG9wdGlvbiBhdmFpbGFibGUsIGJ1dCBvdGhlcndpc2UgdXNlZCBh
cyBoZWxwIHRleHQuCisKK09uIHN1Y2Nlc3MsIHlvdSB3aWxsIHNlZSBhIHNlcmllcyBvZiAiUEFT
UyIgbWVzc2FnZXMsIGZvbGxvd2VkIGJ5ICJURVNUIENPTVBMRVRFIi4KKworCitQQVNTIGFjY2Vz
c2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50LnRpdGxlIGlzICdBWFRpdGxlOiB0ZXh0
JworUEFTUyBhY2Nlc3NpYmlsaXR5Q29udHJvbGxlci5mb2N1c2VkRWxlbWVudC5oZWxwVGV4dCBp
cyAnQVhIZWxwOiBUSVRMRScKK1BBU1MgYWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIuZm9jdXNlZEVs
ZW1lbnQudGl0bGUgaXMgJ0FYVGl0bGU6IHRleHQnCitQQVNTIGFjY2Vzc2liaWxpdHlDb250cm9s
bGVyLmZvY3VzZWRFbGVtZW50LmhlbHBUZXh0IGlzICdBWEhlbHA6IEhFTFAnCitQQVNTIGFjY2Vz
c2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50LmRlc2NyaXB0aW9uIGlzICdBWERlc2Ny
aXB0aW9uOiBUSVRMRScKK1BBU1MgYWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIuZm9jdXNlZEVsZW1l
bnQuaGVscFRleHQgaXMgJ0FYSGVscDogJworUEFTUyBhY2Nlc3NpYmlsaXR5Q29udHJvbGxlci5m
b2N1c2VkRWxlbWVudC5kZXNjcmlwdGlvbiBpcyAnQVhEZXNjcmlwdGlvbjogVElUTEUnCitQQVNT
IGFjY2Vzc2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50LmhlbHBUZXh0IGlzICdBWEhl
bHA6IEhFTFAnCitQQVNTIGFjY2Vzc2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50LmRl
c2NyaXB0aW9uIGlzICdBWERlc2NyaXB0aW9uOiBMQUJFTCcKK1BBU1MgYWNjZXNzaWJpbGl0eUNv
bnRyb2xsZXIuZm9jdXNlZEVsZW1lbnQuaGVscFRleHQgaXMgJ0FYSGVscDogSEVMUCcKK1BBU1Mg
c3VjY2Vzc2Z1bGx5UGFyc2VkIGlzIHRydWUKKworVEVTVCBDT01QTEVURQorCkluZGV4OiBMYXlv
dXRUZXN0cy9wbGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS90aXRsZS1hdHRyaWJ1dGUtdXNlZC1h
cy1sYXN0LXJlc29ydC5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL3BsYXRmb3JtL21h
Yy9hY2Nlc3NpYmlsaXR5L3RpdGxlLWF0dHJpYnV0ZS11c2VkLWFzLWxhc3QtcmVzb3J0Lmh0bWwJ
KHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS90
aXRsZS1hdHRyaWJ1dGUtdXNlZC1hcy1sYXN0LXJlc29ydC5odG1sCShyZXZpc2lvbiAwKQpAQCAt
MCwwICsxLDU0IEBACis8IURPQ1RZUEUgSFRNTCBQVUJMSUMgIi0vL0lFVEYvL0RURCBIVE1MLy9F
TiI+Cis8aHRtbD4KKzxoZWFkPgorPHNjcmlwdCBzcmM9Ii4uLy4uLy4uL2Zhc3QvanMvcmVzb3Vy
Y2VzL2pzLXRlc3QtcHJlLmpzIj48L3NjcmlwdD4KKzwvaGVhZD4KKzxib2R5IGlkPSJib2R5Ij4K
KworPGEgaWQ9ImxpbmsxIiB0YWJpbmRleD0iMCIgaHJlZj0iIyIgdGl0bGU9IlRJVExFIj50ZXh0
PC9hPgorPGEgaWQ9ImxpbmsyIiB0YWJpbmRleD0iMCIgaHJlZj0iIyIgYXJpYS1oZWxwPSJIRUxQ
IiB0aXRsZT0iVElUTEUiPnRleHQ8L2E+Cis8YSBpZD0ibGluazMiIHRhYmluZGV4PSIwIiBocmVm
PSIjIiB0aXRsZT0iVElUTEUiPjwvYT4KKzxhIGlkPSJsaW5rNCIgdGFiaW5kZXg9IjAiIGhyZWY9
IiMiIGFyaWEtaGVscD0iSEVMUCIgdGl0bGU9IlRJVExFIj48L2E+Cis8YSBpZD0ibGluazUiIHRh
YmluZGV4PSIwIiBocmVmPSIjIiBhcmlhLWhlbHA9IkhFTFAiIGFyaWEtbGFiZWw9IkxBQkVMIiB0
aXRsZT0iVElUTEUiPjwvYT4KKworPHAgaWQ9ImRlc2NyaXB0aW9uIj48L3A+Cis8ZGl2IGlkPSJj
b25zb2xlIj48L2Rpdj4KKzxkaXYgaWQ9Im5vdGlmaWNhdGlvbnMiPjwvZGl2PgorCis8c2NyaXB0
PgorCisgICAgZGVzY3JpcHRpb24oIlRoaXMgdGVzdHMgdGhhdCB0aGUgdGl0bGUgYXR0cmlidXRl
IGlzIHVzZWQgYXMgdGhlIGxhc3QgcmVzb3J0IGZvciBkZXNjcmlwdGlvbiB3aGVuIHRoZXJlIGlz
IG5vIG90aGVyIG9wdGlvbiBhdmFpbGFibGUsIGJ1dCBvdGhlcndpc2UgdXNlZCBhcyBoZWxwIHRl
eHQuIik7CisKKyAgICBpZiAod2luZG93LmFjY2Vzc2liaWxpdHlDb250cm9sbGVyKSB7CisKKyAg
ICAgICAgLy8gQ2FzZSAxOiB0aXRsZSBhdHRyaWJ1dGUsIHdpdGggdGV4dCBpbnNpZGUgZWxlbWVu
dC4KKyAgICAgICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImxpbmsxIikuZm9jdXMoKTsKKyAg
ICAgICAgc2hvdWxkQmUoImFjY2Vzc2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50LnRp
dGxlIiwgIidBWFRpdGxlOiB0ZXh0JyIpOworICAgICAgICBzaG91bGRCZSgiYWNjZXNzaWJpbGl0
eUNvbnRyb2xsZXIuZm9jdXNlZEVsZW1lbnQuaGVscFRleHQiLCAiJ0FYSGVscDogVElUTEUnIik7
CisKKyAgICAgICAgLy8gQ2FzZSAyOiB0aXRsZSBhdHRyaWJ1dGUgYW5kIGFyaWEtaGVscCBhdHRy
aWJ1dGUsIHdpdGggdGV4dCBpbnNpZGUgZWxlbWVudC4KKyAgICAgICAgZG9jdW1lbnQuZ2V0RWxl
bWVudEJ5SWQoImxpbmsyIikuZm9jdXMoKTsKKyAgICAgICAgc2hvdWxkQmUoImFjY2Vzc2liaWxp
dHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50LnRpdGxlIiwgIidBWFRpdGxlOiB0ZXh0JyIpOwor
ICAgICAgICBzaG91bGRCZSgiYWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIuZm9jdXNlZEVsZW1lbnQu
aGVscFRleHQiLCAiJ0FYSGVscDogSEVMUCciKTsKKworICAgICAgICAvLyBDYXNlIDM6IHRpdGxl
IGF0dHJpYnV0ZSBpcyBvbmx5IHRleHQgYXZhaWxhYmxlLgorICAgICAgICBkb2N1bWVudC5nZXRF
bGVtZW50QnlJZCgibGluazMiKS5mb2N1cygpOworICAgICAgICBzaG91bGRCZSgiYWNjZXNzaWJp
bGl0eUNvbnRyb2xsZXIuZm9jdXNlZEVsZW1lbnQuZGVzY3JpcHRpb24iLCAiJ0FYRGVzY3JpcHRp
b246IFRJVExFJyIpOworICAgICAgICBzaG91bGRCZSgiYWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIu
Zm9jdXNlZEVsZW1lbnQuaGVscFRleHQiLCAiJ0FYSGVscDogJyIpOworCisgICAgICAgIC8vIENh
c2UgNDogdGl0bGUgYXR0cmlidXRlIGFuZCBhcmlhLWhlbHAgYXJlIGF2YWlsYWJsZS4KKyAgICAg
ICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImxpbms0IikuZm9jdXMoKTsKKyAgICAgICAgc2hv
dWxkQmUoImFjY2Vzc2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50LmRlc2NyaXB0aW9u
IiwgIidBWERlc2NyaXB0aW9uOiBUSVRMRSciKTsKKyAgICAgICAgc2hvdWxkQmUoImFjY2Vzc2li
aWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVtZW50LmhlbHBUZXh0IiwgIidBWEhlbHA6IEhFTFAn
Iik7CisKKyAgICAgICAgLy8gQ2FzZSA1OiBhcmlhLWhlbHAsIHRpdGxlIGFuZCBhcmlhLWxhYmVs
IGFyZSBhdmFpbGFibGUuCisgICAgICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJsaW5rNSIp
LmZvY3VzKCk7CisgICAgICAgIHNob3VsZEJlKCJhY2Nlc3NpYmlsaXR5Q29udHJvbGxlci5mb2N1
c2VkRWxlbWVudC5kZXNjcmlwdGlvbiIsICInQVhEZXNjcmlwdGlvbjogTEFCRUwnIik7CisgICAg
ICAgIHNob3VsZEJlKCJhY2Nlc3NpYmlsaXR5Q29udHJvbGxlci5mb2N1c2VkRWxlbWVudC5oZWxw
VGV4dCIsICInQVhIZWxwOiBIRUxQJyIpOworICAgIH0KKworPC9zY3JpcHQ+CisKKzxzY3JpcHQg
c3JjPSIuLi8uLi8uLi9mYXN0L2pzL3Jlc291cmNlcy9qcy10ZXN0LXBvc3QuanMiPjwvc2NyaXB0
PgorPC9ib2R5PgorPC9odG1sPgo=
</data>
<flag name="review"
          id="162636"
          type_id="1"
          status="+"
          setter="andersca"
    />
          </attachment>
      

    </bug>

</bugzilla>