<?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>196326</bug_id>
          
          <creation_ts>2019-03-27 16:31:53 -0700</creation_ts>
          <short_desc>[WK2] AccessibilityEventsEnabled setting does not persist after a WebProcess crash</short_desc>
          <delta_ts>2020-10-12 14:12:37 -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>WebKit2</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>ASSIGNED</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Chris Dumez">cdumez</reporter>
          <assigned_to name="Chris Dumez">cdumez</assigned_to>
          <cc>achristensen</cc>
    
    <cc>beidson</cc>
    
    <cc>cfleizach</cc>
    
    <cc>ggaren</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1521907</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2019-03-27 16:31:53 -0700</bug_when>
    <thetext>AccessibilityEventsEnabled setting does not persist after a WebProcess crash.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1521909</commentid>
    <comment_count>1</comment_count>
      <attachid>366128</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2019-03-27 16:34:52 -0700</bug_when>
    <thetext>Created attachment 366128
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1521910</commentid>
    <comment_count>2</comment_count>
      <attachid>366128</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2019-03-27 16:35:56 -0700</bug_when>
    <thetext>Comment on attachment 366128
Patch

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

&gt; Source/WebKit/UIProcess/WebPageProxy.h:2211
&gt; +    bool m_areAccessibilityEventsEnabled { true };

I set it to true by default because it was true by default in WebProcess side:
Source/WebCore/page/Settings.yaml:
accessibilityEventsEnabled:
  initial: true
  conditional: ACCESSIBILITY_EVENTS</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1521911</commentid>
    <comment_count>3</comment_count>
      <attachid>366128</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2019-03-27 16:41:09 -0700</bug_when>
    <thetext>Comment on attachment 366128
Patch

Can we not make a unit test using _killWebContentProcessAndResetState ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1521914</commentid>
    <comment_count>4</comment_count>
      <attachid>366128</attachid>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2019-03-27 16:46:27 -0700</bug_when>
    <thetext>Comment on attachment 366128
Patch

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

&gt; Source/WebKit/Shared/WebPageCreationParameters.cpp:47
&gt; +    encoder &lt;&lt; areAccessibilityEventsEnabled;

not sure what current naming guidelines are, but &quot;are&quot; seems unnecessary.

&quot;accessbilityEventsEnabled&quot; seems like it might be enough

&gt;&gt; Source/WebKit/UIProcess/WebPageProxy.h:2211
&gt;&gt; +    bool m_areAccessibilityEventsEnabled { true };
&gt; 
&gt; I set it to true by default because it was true by default in WebProcess side:
&gt; Source/WebCore/page/Settings.yaml:
&gt; accessibilityEventsEnabled:
&gt;   initial: true
&gt;   conditional: ACCESSIBILITY_EVENTS

correct.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1521915</commentid>
    <comment_count>5</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2019-03-27 16:49:04 -0700</bug_when>
    <thetext>(In reply to chris fleizach from comment #4)
&gt; Comment on attachment 366128 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=366128&amp;action=review
&gt; 
&gt; &gt; Source/WebKit/Shared/WebPageCreationParameters.cpp:47
&gt; &gt; +    encoder &lt;&lt; areAccessibilityEventsEnabled;
&gt; 
&gt; not sure what current naming guidelines are, but &quot;are&quot; seems unnecessary.
&gt; 
&gt; &quot;accessbilityEventsEnabled&quot; seems like it might be enough

This is per WebKit coding style though:
https://webkit.org/code-style-guidelines/#names-bool</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1521916</commentid>
    <comment_count>6</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2019-03-27 16:49:57 -0700</bug_when>
    <thetext>(In reply to Alex Christensen from comment #3)
&gt; Comment on attachment 366128 [details]
&gt; Patch
&gt; 
&gt; Can we not make a unit test using _killWebContentProcessAndResetState ?

What do I observe? I do not know what accessibility events are :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1521918</commentid>
    <comment_count>7</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2019-03-27 16:52:17 -0700</bug_when>
    <thetext>(In reply to Chris Dumez from comment #6)
&gt; (In reply to Alex Christensen from comment #3)
&gt; &gt; Comment on attachment 366128 [details]
&gt; &gt; Patch
&gt; &gt; 
&gt; &gt; Can we not make a unit test using _killWebContentProcessAndResetState ?
&gt; 
&gt; What do I observe? I do not know what accessibility events are :)

Chris F, can you make a recommendation here?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1521920</commentid>
    <comment_count>8</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2019-03-27 16:57:03 -0700</bug_when>
    <thetext>(In reply to Geoffrey Garen from comment #7)
&gt; (In reply to Chris Dumez from comment #6)
&gt; &gt; (In reply to Alex Christensen from comment #3)
&gt; &gt; &gt; Comment on attachment 366128 [details]
&gt; &gt; &gt; Patch
&gt; &gt; &gt; 
&gt; &gt; &gt; Can we not make a unit test using _killWebContentProcessAndResetState ?
&gt; &gt; 
&gt; &gt; What do I observe? I do not know what accessibility events are :)
&gt; 
&gt; Chris F, can you make a recommendation here?

Note that for testing, we would also need an SPI to actually set the state to enabled or disabled. Currently, there is only SPI to ask to update the state and it relies on _AXSWebAccessibilityEventsEnabled() internally, the state is not passed by the client.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1522059</commentid>
    <comment_count>9</comment_count>
    <who name="chris fleizach">cfleizach</who>
    <bug_when>2019-03-28 07:34:57 -0700</bug_when>
    <thetext>(In reply to Chris Dumez from comment #8)
&gt; (In reply to Geoffrey Garen from comment #7)
&gt; &gt; (In reply to Chris Dumez from comment #6)
&gt; &gt; &gt; (In reply to Alex Christensen from comment #3)
&gt; &gt; &gt; &gt; Comment on attachment 366128 [details]
&gt; &gt; &gt; &gt; Patch
&gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; Can we not make a unit test using _killWebContentProcessAndResetState ?
&gt; &gt; &gt; 
&gt; &gt; &gt; What do I observe? I do not know what accessibility events are :)
&gt; &gt; 
&gt; &gt; Chris F, can you make a recommendation here?
&gt; 
&gt; Note that for testing, we would also need an SPI to actually set the state
&gt; to enabled or disabled. Currently, there is only SPI to ask to update the
&gt; state and it relies on _AXSWebAccessibilityEventsEnabled() internally, the
&gt; state is not passed by the client.


Here&apos;s the setter

extern void _AXSSetWebAccessibilityEventsEnabled(Boolean enabled);


and then you can observe kAXSWebAccessibilityEventsEnabledNotification on the local port</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1522193</commentid>
    <comment_count>10</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2019-03-28 13:32:48 -0700</bug_when>
    <thetext>(In reply to chris fleizach from comment #9)
&gt; (In reply to Chris Dumez from comment #8)
&gt; &gt; (In reply to Geoffrey Garen from comment #7)
&gt; &gt; &gt; (In reply to Chris Dumez from comment #6)
&gt; &gt; &gt; &gt; (In reply to Alex Christensen from comment #3)
&gt; &gt; &gt; &gt; &gt; Comment on attachment 366128 [details]
&gt; &gt; &gt; &gt; &gt; Patch
&gt; &gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; &gt; Can we not make a unit test using _killWebContentProcessAndResetState ?
&gt; &gt; &gt; &gt; 
&gt; &gt; &gt; &gt; What do I observe? I do not know what accessibility events are :)
&gt; &gt; &gt; 
&gt; &gt; &gt; Chris F, can you make a recommendation here?
&gt; &gt; 
&gt; &gt; Note that for testing, we would also need an SPI to actually set the state
&gt; &gt; to enabled or disabled. Currently, there is only SPI to ask to update the
&gt; &gt; state and it relies on _AXSWebAccessibilityEventsEnabled() internally, the
&gt; &gt; state is not passed by the client.
&gt; 
&gt; 
&gt; Here&apos;s the setter
&gt; 
&gt; extern void _AXSSetWebAccessibilityEventsEnabled(Boolean enabled);
&gt; 
&gt; 
&gt; and then you can observe kAXSWebAccessibilityEventsEnabledNotification on
&gt; the local port

Ok, this helps with turning the setting on and off from an API test. I am still unclear how I can write an API test for this though. I found the following related layout test:
LayoutTests/accessibility/ios-simulator/accessibility-events-setting.html

Seems like I can set a onaccessibleclick event handler on a button and observe if it gets called upon click or not. So far so good. However, it does not seem to get called for regular clicks.
It seems it only works for &quot;accessible clicks&quot;. How do I do an accessible click from my API test? It seems like the layout test is currently relying on some WebKitTestRunner infra to trigger such a click.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1697020</commentid>
    <comment_count>11</comment_count>
      <attachid>366128</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2020-10-12 14:12:37 -0700</bug_when>
    <thetext>Comment on attachment 366128
Patch

r=me</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>366128</attachid>
            <date>2019-03-27 16:34:52 -0700</date>
            <delta_ts>2020-10-12 14:12:37 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-196326-20190327163447.patch</filename>
            <type>text/plain</type>
            <size>5865</size>
            <attacher name="Chris Dumez">cdumez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQzNTY2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDA2NmMzMGE2OGU1ZmFmNTIz
M2U1YmJlYWUxOGVkMjY2MTlmYTNjZTguLmJkODI2MGU3NjdjYzgxNGE1MmZlMmMzNjc1Y2U1MGU5
NWZiYThlMTQgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjUgQEAKKzIwMTktMDMtMjcgIENocmlzIER1
bWV6ICA8Y2R1bWV6QGFwcGxlLmNvbT4KKworICAgICAgICBbV0syXSBBY2Nlc3NpYmlsaXR5RXZl
bnRzRW5hYmxlZCBzZXR0aW5nIGRvZXMgbm90IHBlcnNpc3QgYWZ0ZXIgYSBXZWJQcm9jZXNzIGNy
YXNoCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xOTYz
MjYKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBBY2Nl
c3NpYmlsaXR5RXZlbnRzRW5hYmxlZCBzZXR0aW5nIGRvZXMgbm90IHBlcnNpc3QgYWZ0ZXIgYSBX
ZWJQcm9jZXNzIGNyYXNoLCBpdCBhbHNvIGRvZXMgbm90IGdldAorICAgICAgICBzZXQgcHJvcGVy
bHkgaWYgdGhlIHBhZ2UgaGFzIG5vdCBsYXVuY2hlZCBpdHMgcHJvY2VzcyB3aGVuIFdlYlBhZ2VQ
cm94eTo6dXBkYXRlQWNjZXNzaWJpbGl0eUV2ZW50c0VuYWJsZWQoKQorICAgICAgICBnZXRzIGNh
bGxlZC4gVGhlIHJlYXNvbiBpcyB0aGF0IFdlYlBhZ2VQcm94eTo6dXBkYXRlQWNjZXNzaWJpbGl0
eUV2ZW50c0VuYWJsZWQoKSB3b3VsZCBzZW5kIElQQyBidXQgbm90CisgICAgICAgIHN0b3JlIHRo
ZSB2YWx1ZSBsb2NhbGx5IGFuZCBwYXNzIGl0IHZpYSB0aGUgV2ViUGFnZUNyZWF0aW9uUGFyYW1l
dGVycy4KKworICAgICAgICAqIFNoYXJlZC9XZWJQYWdlQ3JlYXRpb25QYXJhbWV0ZXJzLmNwcDoK
KyAgICAgICAgKFdlYktpdDo6V2ViUGFnZUNyZWF0aW9uUGFyYW1ldGVyczo6ZW5jb2RlIGNvbnN0
KToKKyAgICAgICAgKFdlYktpdDo6V2ViUGFnZUNyZWF0aW9uUGFyYW1ldGVyczo6ZGVjb2RlKToK
KyAgICAgICAgKiBTaGFyZWQvV2ViUGFnZUNyZWF0aW9uUGFyYW1ldGVycy5oOgorICAgICAgICAq
IFVJUHJvY2Vzcy9XZWJQYWdlUHJveHkuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJQYWdlUHJv
eHk6OnVwZGF0ZUFjY2Vzc2liaWxpdHlFdmVudHNFbmFibGVkKToKKyAgICAgICAgKFdlYktpdDo6
V2ViUGFnZVByb3h5OjpjcmVhdGlvblBhcmFtZXRlcnMpOgorICAgICAgICAqIFVJUHJvY2Vzcy9X
ZWJQYWdlUHJveHkuaDoKKyAgICAgICAgKiBXZWJQcm9jZXNzL1dlYlBhZ2UvV2ViUGFnZS5jcHA6
CisKIDIwMTktMDMtMjcgIEtlaXRoIFJvbGxpbiAgPGtyb2xsaW5AYXBwbGUuY29tPgogCiAgICAg
ICAgIEluY2x1ZGUgdGhlIENGTmV0d29ya2luZyB0YXNrIFVVSUQgdG8gdGhlIE5TRXJyb3Igd2hl
biBhIHJlc291cmNlLWxvYWQgZmFpbHMKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvU2hhcmVk
L1dlYlBhZ2VDcmVhdGlvblBhcmFtZXRlcnMuY3BwIGIvU291cmNlL1dlYktpdC9TaGFyZWQvV2Vi
UGFnZUNyZWF0aW9uUGFyYW1ldGVycy5jcHAKaW5kZXggNTkwNTc0NTUyOWNkNDg5ZjdhYzUxZjE2
M2RkYWFlNjM2MGFhZDQ5Mi4uZTFmN2YxZTY5ODc3YjczNTNkMDEwNjI0MmE1NzY5NDcyZTdiZjY3
NSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9TaGFyZWQvV2ViUGFnZUNyZWF0aW9uUGFyYW1l
dGVycy5jcHAKKysrIGIvU291cmNlL1dlYktpdC9TaGFyZWQvV2ViUGFnZUNyZWF0aW9uUGFyYW1l
dGVycy5jcHAKQEAgLTQzLDYgKzQzLDkgQEAgdm9pZCBXZWJQYWdlQ3JlYXRpb25QYXJhbWV0ZXJz
OjplbmNvZGUoSVBDOjpFbmNvZGVyJiBlbmNvZGVyKSBjb25zdAogICAgIGVuY29kZXIgPDwgdW5k
ZXJsYXlDb2xvcjsKICAgICBlbmNvZGVyIDw8IHVzZUZpeGVkTGF5b3V0OwogICAgIGVuY29kZXIg
PDwgZml4ZWRMYXlvdXRTaXplOworI2lmIEVOQUJMRShBQ0NFU1NJQklMSVRZX0VWRU5UUykKKyAg
ICBlbmNvZGVyIDw8IGFyZUFjY2Vzc2liaWxpdHlFdmVudHNFbmFibGVkOworI2VuZGlmCiAgICAg
ZW5jb2RlciA8PCBhbHdheXNTaG93c0hvcml6b250YWxTY3JvbGxlcjsKICAgICBlbmNvZGVyIDw8
IGFsd2F5c1Nob3dzVmVydGljYWxTY3JvbGxlcjsKICAgICBlbmNvZGVyLmVuY29kZUVudW0ocGFn
aW5hdGlvbk1vZGUpOwpAQCAtMTU3LDYgKzE2MCwxMCBAQCBPcHRpb25hbDxXZWJQYWdlQ3JlYXRp
b25QYXJhbWV0ZXJzPiBXZWJQYWdlQ3JlYXRpb25QYXJhbWV0ZXJzOjpkZWNvZGUoSVBDOjpEZWNv
ZAogICAgICAgICByZXR1cm4gV1RGOjpudWxsb3B0OwogICAgIGlmICghZGVjb2Rlci5kZWNvZGUo
cGFyYW1ldGVycy5maXhlZExheW91dFNpemUpKQogICAgICAgICByZXR1cm4gV1RGOjpudWxsb3B0
OworI2lmIEVOQUJMRShBQ0NFU1NJQklMSVRZX0VWRU5UUykKKyAgICBpZiAoIWRlY29kZXIuZGVj
b2RlKHBhcmFtZXRlcnMuYXJlQWNjZXNzaWJpbGl0eUV2ZW50c0VuYWJsZWQpKQorICAgICAgICBy
ZXR1cm4gV1RGOjpudWxsb3B0OworI2VuZGlmCiAgICAgaWYgKCFkZWNvZGVyLmRlY29kZShwYXJh
bWV0ZXJzLmFsd2F5c1Nob3dzSG9yaXpvbnRhbFNjcm9sbGVyKSkKICAgICAgICAgcmV0dXJuIFdU
Rjo6bnVsbG9wdDsKICAgICBpZiAoIWRlY29kZXIuZGVjb2RlKHBhcmFtZXRlcnMuYWx3YXlzU2hv
d3NWZXJ0aWNhbFNjcm9sbGVyKSkKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvU2hhcmVkL1dl
YlBhZ2VDcmVhdGlvblBhcmFtZXRlcnMuaCBiL1NvdXJjZS9XZWJLaXQvU2hhcmVkL1dlYlBhZ2VD
cmVhdGlvblBhcmFtZXRlcnMuaAppbmRleCA5NzZjODg2ZjQzMzNmYzExNThiNDM5NjJiYTYzYzEz
NGI0ZjZkZWYxLi44YTMwZGYzYTU0NDIzMTMwM2QyNmIyNzU2MTZhNmJkOWUzZmQ5OWI0IDEwMDY0
NAotLS0gYS9Tb3VyY2UvV2ViS2l0L1NoYXJlZC9XZWJQYWdlQ3JlYXRpb25QYXJhbWV0ZXJzLmgK
KysrIGIvU291cmNlL1dlYktpdC9TaGFyZWQvV2ViUGFnZUNyZWF0aW9uUGFyYW1ldGVycy5oCkBA
IC04MSw2ICs4MSwxMCBAQCBzdHJ1Y3QgV2ViUGFnZUNyZWF0aW9uUGFyYW1ldGVycyB7CiAgICAg
Ym9vbCB1c2VGaXhlZExheW91dDsKICAgICBXZWJDb3JlOjpJbnRTaXplIGZpeGVkTGF5b3V0U2l6
ZTsKIAorI2lmIEVOQUJMRShBQ0NFU1NJQklMSVRZX0VWRU5UUykKKyAgICBib29sIGFyZUFjY2Vz
c2liaWxpdHlFdmVudHNFbmFibGVkIHsgdHJ1ZSB9OworI2VuZGlmCisKICAgICBib29sIGFsd2F5
c1Nob3dzSG9yaXpvbnRhbFNjcm9sbGVyOwogICAgIGJvb2wgYWx3YXlzU2hvd3NWZXJ0aWNhbFNj
cm9sbGVyOwogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9XZWJQYWdlUHJv
eHkuY3BwIGIvU291cmNlL1dlYktpdC9VSVByb2Nlc3MvV2ViUGFnZVByb3h5LmNwcAppbmRleCAz
NjJlOTg0MmZlYWI3MGY5MGE0Zjg1YjA3NjMxMjg2ZTVhN2IzNGIzLi5kMGUxMTMyZWM0ZGIzN2Zk
ZDAwNWQ0OTE3ZmM3OWFhZDZlNzFkMjA2IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0L1VJUHJv
Y2Vzcy9XZWJQYWdlUHJveHkuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlBh
Z2VQcm94eS5jcHAKQEAgLTMyNDEsNiArMzI0MSwxMSBAQCB2b2lkIFdlYlBhZ2VQcm94eTo6YWNj
ZXNzaWJpbGl0eVNldHRpbmdzRGlkQ2hhbmdlKCkKICNpZiBFTkFCTEUoQUNDRVNTSUJJTElUWV9F
VkVOVFMpCiB2b2lkIFdlYlBhZ2VQcm94eTo6dXBkYXRlQWNjZXNzaWJpbGl0eUV2ZW50c0VuYWJs
ZWQoYm9vbCBlbmFibGVkKQogeworICAgIGlmIChlbmFibGVkID09IG1fYXJlQWNjZXNzaWJpbGl0
eUV2ZW50c0VuYWJsZWQpCisgICAgICAgIHJldHVybjsKKworICAgIG1fYXJlQWNjZXNzaWJpbGl0
eUV2ZW50c0VuYWJsZWQgPSBlbmFibGVkOworCiAgICAgaWYgKCFoYXNSdW5uaW5nUHJvY2Vzcygp
KQogICAgICAgICByZXR1cm47CiAKQEAgLTY5NzAsNiArNjk3NSw5IEBAIFdlYlBhZ2VDcmVhdGlv
blBhcmFtZXRlcnMgV2ViUGFnZVByb3h5OjpjcmVhdGlvblBhcmFtZXRlcnMoV2ViUHJvY2Vzc1By
b3h5JiBwcm9jCiAgICAgcGFyYW1ldGVycy5pc0VkaXRhYmxlID0gbV9pc0VkaXRhYmxlOwogICAg
IHBhcmFtZXRlcnMudW5kZXJsYXlDb2xvciA9IG1fdW5kZXJsYXlDb2xvcjsKICAgICBwYXJhbWV0
ZXJzLnVzZUZpeGVkTGF5b3V0ID0gbV91c2VGaXhlZExheW91dDsKKyNpZiBFTkFCTEUoQUNDRVNT
SUJJTElUWV9FVkVOVFMpCisgICAgcGFyYW1ldGVycy5hcmVBY2Nlc3NpYmlsaXR5RXZlbnRzRW5h
YmxlZCA9IG1fYXJlQWNjZXNzaWJpbGl0eUV2ZW50c0VuYWJsZWQ7CisjZW5kaWYKICAgICBwYXJh
bWV0ZXJzLmZpeGVkTGF5b3V0U2l6ZSA9IG1fZml4ZWRMYXlvdXRTaXplOwogICAgIHBhcmFtZXRl
cnMuYWx3YXlzU2hvd3NIb3Jpem9udGFsU2Nyb2xsZXIgPSBtX2Fsd2F5c1Nob3dzSG9yaXpvbnRh
bFNjcm9sbGVyOwogICAgIHBhcmFtZXRlcnMuYWx3YXlzU2hvd3NWZXJ0aWNhbFNjcm9sbGVyID0g
bV9hbHdheXNTaG93c1ZlcnRpY2FsU2Nyb2xsZXI7CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0
L1VJUHJvY2Vzcy9XZWJQYWdlUHJveHkuaCBiL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlBh
Z2VQcm94eS5oCmluZGV4IDE0NDczNTE0YTA4MWY3NWU0NmJiOTEyYjc2Nzk4MTI5ZjU5MWViZDUu
LjBkYTRlZGU3YjhiMDhlNDExNGU4OTI5ODVmMmZhYTgyMjQzMGIyNmMgMTAwNjQ0Ci0tLSBhL1Nv
dXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlBhZ2VQcm94eS5oCisrKyBiL1NvdXJjZS9XZWJLaXQv
VUlQcm9jZXNzL1dlYlBhZ2VQcm94eS5oCkBAIC0yMjA3LDYgKzIyMDcsMTAgQEAgcHJpdmF0ZToK
ICAgICBib29sIG1fdXNlRml4ZWRMYXlvdXQgeyBmYWxzZSB9OwogICAgIFdlYkNvcmU6OkludFNp
emUgbV9maXhlZExheW91dFNpemU7CiAKKyNpZiBFTkFCTEUoQUNDRVNTSUJJTElUWV9FVkVOVFMp
CisgICAgYm9vbCBtX2FyZUFjY2Vzc2liaWxpdHlFdmVudHNFbmFibGVkIHsgdHJ1ZSB9OworI2Vu
ZGlmCisKICAgICBib29sIG1fYWx3YXlzU2hvd3NIb3Jpem9udGFsU2Nyb2xsZXIgeyBmYWxzZSB9
OwogICAgIGJvb2wgbV9hbHdheXNTaG93c1ZlcnRpY2FsU2Nyb2xsZXIgeyBmYWxzZSB9OwogCmRp
ZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJQYWdlLmNwcCBi
L1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9XZWJQYWdlL1dlYlBhZ2UuY3BwCmluZGV4IGE3Y2Yx
OGU2ZWVkZDFkYzczNzgyODZlMGUzYzAxZDZmMGJhNTI4NjMuLmZmMzc5MTg2ZTQ4Mzg0Njc3ZTdi
ZjVjMjBiZjBiYzU1Njc0Y2ZhYTggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vz
cy9XZWJQYWdlL1dlYlBhZ2UuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9XZWJQ
YWdlL1dlYlBhZ2UuY3BwCkBAIC01MzgsNiArNTM4LDEwIEBAIFdlYlBhZ2U6OldlYlBhZ2UodWlu
dDY0X3QgcGFnZUlELCBXZWJQYWdlQ3JlYXRpb25QYXJhbWV0ZXJzJiYgcGFyYW1ldGVycykKIAog
ICAgIHNldFVzZUZpeGVkTGF5b3V0KHBhcmFtZXRlcnMudXNlRml4ZWRMYXlvdXQpOwogCisjaWYg
RU5BQkxFKEFDQ0VTU0lCSUxJVFlfRVZFTlRTKQorICAgIHVwZGF0ZUFjY2Vzc2liaWxpdHlFdmVu
dHNFbmFibGVkKHBhcmFtZXRlcnMuYXJlQWNjZXNzaWJpbGl0eUV2ZW50c0VuYWJsZWQpOworI2Vu
ZGlmCisKICAgICBzZXRVbmRlcmxheUNvbG9yKHBhcmFtZXRlcnMudW5kZXJsYXlDb2xvcik7CiAK
ICAgICBzZXRQYWdpbmF0aW9uTW9kZShwYXJhbWV0ZXJzLnBhZ2luYXRpb25Nb2RlKTsK
</data>
<flag name="review"
          id="382550"
          type_id="1"
          status="+"
          setter="ggaren"
    />
          </attachment>
      

    </bug>

</bugzilla>