43872 2010-08-11 13:16:03 -0700 AX: isNativeCheckbox does not work as advertised 2010-08-13 11:51:03 -0700 1 1 1 Unclassified WebKit Accessibility 528+ (Nightly build) PC OS X 10.5 RESOLVED FIXED P2 Normal --- 43965 1 cfleizach cfleizach bdakin ddkilzer japhet yutak oldest_to_newest 263423 0 cfleizach 2010-08-11 13:16:03 -0700 this method will return true for any InputElement. it does not account for someone doing something like <input type='image' role="checkbox"> which would return true because it's an InputElement, but in reality is false illustrated with this webpage http://www.paciellogroup.com/blog/misc/ARIA/tristatecheck.html 263441 1 64158 cfleizach 2010-08-11 13:56:58 -0700 Created attachment 64158 Patch 263553 2 cfleizach 2010-08-11 18:24:37 -0700 http://trac.webkit.org/changeset/65204 264298 3 yutak 2010-08-13 05:16:53 -0700 Hello, I've found your r65204 produces a lot of test crashes (90+ tests) when it is merged in Chromium. Change: http://src.chromium.org/viewvc/chrome?view=rev&revision=56014 Bot result: http://buildbot.jail.google.com/buildbot/chromium/builders/Webkit/builds/26081/steps/webkit_tests/logs/stdio (Please see the very bottom of the output) To tell the truth, I really don't understand why these tests are crashing. I could not reproduce in my local environment, and the stack trace was totally unusable. Your patch may not be the fault. However, we cannot leave this change as we don't want to see 90+ crashes when it is merged to Chromium. So, if there is no objections, I want to roll out r65204 in next hour or so. Thank you for understanding. If you have any questions, please ask me. 264382 4 cfleizach 2010-08-13 08:37:39 -0700 (In reply to comment #3) > Hello, > > I've found your r65204 produces a lot of test crashes (90+ tests) when it is merged in Chromium. > > Change: http://src.chromium.org/viewvc/chrome?view=rev&revision=56014 > Bot result: http://buildbot.jail.google.com/buildbot/chromium/builders/Webkit/builds/26081/steps/webkit_tests/logs/stdio > (Please see the very bottom of the output) > > To tell the truth, I really don't understand why these tests are crashing. I could not reproduce in my local environment, and the stack trace was totally unusable. Your patch may not be the fault. > > However, we cannot leave this change as we don't want to see 90+ crashes when it is merged to Chromium. So, if there is no objections, I want to roll out r65204 in next hour or so. > > Thank you for understanding. If you have any questions, please ask me. As you can imagine, I am quite confused. Asking if there will be objections at 5AM obviously won't find too many objectors. This URL gives me an error page as not found http://buildbot.jail.google.com/buildbot/chromium/builders/Webkit/builds/26081/steps/webkit_tests/logs/stdio and this page looks like the change just merged in a DEPS file http://src.chromium.org/viewvc/chrome?view=rev&revision=56014 can you attach some actual crash logs. did these issues reproduce on the chromium build bots? i didn't hear anything about any problems all day long on the 12th. did this actually fix your crashing issues? I don't see how this could affect anything besides accessibility related code. are you seeing crashes in accessibility code? i plan to roll this change back in at my earliest convenience. please include more evidence in the future. 264401 5 yutak 2010-08-13 09:19:10 -0700 (In reply to comment #4) > (In reply to comment #3) > > Hello, > > > > I've found your r65204 produces a lot of test crashes (90+ tests) when it is merged in Chromium. > > > > Change: http://src.chromium.org/viewvc/chrome?view=rev&revision=56014 > > Bot result: http://buildbot.jail.google.com/buildbot/chromium/builders/Webkit/builds/26081/steps/webkit_tests/logs/stdio > > (Please see the very bottom of the output) > > > > To tell the truth, I really don't understand why these tests are crashing. I could not reproduce in my local environment, and the stack trace was totally unusable. Your patch may not be the fault. > > > > However, we cannot leave this change as we don't want to see 90+ crashes when it is merged to Chromium. So, if there is no objections, I want to roll out r65204 in next hour or so. > > > > Thank you for understanding. If you have any questions, please ask me. > > As you can imagine, I am quite confused. Asking if there will be objections at 5AM obviously won't find too many objectors. > I used almost all day of Friday (in Japan time) to track down this revision. My time was almost running out when I've figure out the revision. I hope you understand my situation. Our repository was out of sync with WebKit for more than 24 hours (with 2 people working in different timezones) just because of this mysterious crashes... > This URL gives me an error page as not found > http://buildbot.jail.google.com/buildbot/chromium/builders/Webkit/builds/26081/steps/webkit_tests/logs/stdio > > and this page looks like the change just merged in a DEPS file > http://src.chromium.org/viewvc/chrome?view=rev&revision=56014 > > can you attach some actual crash logs. Sorry, I pasted the wrong address. The correct one is: http://build.chromium.org/buildbot/waterfall/builders/Webkit/builds/26081/steps/webkit_tests/logs/stdio However, it is not very useful, because stack trace is corrupted. Many media or video-related tests are crashing. A few CSS tests also crashes. It seems the set of crashing tests does not change. > > did these issues reproduce on the chromium build bots? > i didn't hear anything about any problems all day long on the 12th. > > did this actually fix your crashing issues? At least on our buildbots, it reliably reproduces. All the tries of WebKit merge beyond r65204 caused the same set of crashes. Bad news is that I could not reproduce the issue in my local environment. > > I don't see how this could affect anything besides accessibility related code. are you seeing crashes in accessibility code? > > i plan to roll this change back in at my earliest convenience. please include more evidence in the future. I understand your confusion. I'm sorry for the fact I can't provide a concrete evidence. WebKit out-of-sync was pretty serious and there was no other way to avoid the crashes... 264454 6 cfleizach 2010-08-13 11:50:57 -0700 I've re-added this patch. I believe it was rolled out erroneously. There's nothing in this change that would cause so many random failures. If the issue persists, I think someone needs to debug the build server (especially because the issue doesn't occur locally). http://trac.webkit.org/changeset/65335 64158 2010-08-11 13:56:58 -0700 2010-08-11 16:46:40 -0700 Patch bug-43872-20100811135656.patch text/plain 6933 cfleizach SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA2NTE4MykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMjEgQEAKKzIwMTAtMDgtMTEgIENocmlzIEZsZWl6YWNoICA8Y2ZsZWl6YWNoQGFw cGxlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg ICBBWDogaXNOYXRpdmVDaGVja2JveCBkb2VzIG5vdCB3b3JrIGFzIGFkdmVydGlzZWQKKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTQzODcyCisKKyAgICAg ICAgVGVzdDogcGxhdGZvcm0vbWFjL2FjY2Vzc2liaWxpdHkvbmF0aXZlLXZzLW5vbm5hdGl2ZS1j aGVja2JveGVzLmh0bWwKKworICAgICAgICAqIGFjY2Vzc2liaWxpdHkvQWNjZXNzaWJpbGl0eVJl bmRlck9iamVjdC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpBY2Nlc3NpYmlsaXR5UmVuZGVyT2Jq ZWN0Ojppc0lucHV0SW1hZ2UpOgorICAgICAgICAgICAgT25seSBzYXkgaXQncyBhbiBpbnB1dCBp bWFnZSBpZiBpdHMgcm9sZSBoYXMgbm90IGJlZW4gb3ZlcnJpZGVuCisgICAgICAgIChXZWJDb3Jl OjpBY2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0Ojppc05hdGl2ZUNoZWNrYm94T3JSYWRpbyk6Cisg ICAgICAgICogZG9tL0lucHV0RWxlbWVudC5oOgorICAgICAgICAgICAgQWRkIG1ldGhvZHMgdG8g cXVlcnkgaWYgaXQncyBhIGNoZWNrYm94IG9yIHJhZGlvIGJ1dHRvbiB0eXBlIGVsZW1lbnQuCisg ICAgICAgICogaHRtbC9IVE1MSW5wdXRFbGVtZW50Lmg6CisgICAgICAgIChXZWJDb3JlOjpIVE1M SW5wdXRFbGVtZW50Ojppc0NoZWNrYm94KToKKwogMjAxMC0wOC0xMCAgQWFyb24gQm9vZG1hbiAg PGFhQGNocm9taXVtLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXZpZCBIeWF0dC4KSW5k ZXg6IFdlYkNvcmUvYWNjZXNzaWJpbGl0eS9BY2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0LmNwcAo9 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09Ci0tLSBXZWJDb3JlL2FjY2Vzc2liaWxpdHkvQWNjZXNzaWJpbGl0eVJlbmRlck9i amVjdC5jcHAJKHJldmlzaW9uIDY1MTc3KQorKysgV2ViQ29yZS9hY2Nlc3NpYmlsaXR5L0FjY2Vz c2liaWxpdHlSZW5kZXJPYmplY3QuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC01MDgsOCArNTA4LDkg QEAgYm9vbCBBY2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0Ojppc0ZpbGVVcAogICAgIAogYm9vbCBB Y2Nlc3NpYmlsaXR5UmVuZGVyT2JqZWN0Ojppc0lucHV0SW1hZ2UoKSBjb25zdAogewotICAgIGlm IChtX3JlbmRlcmVyICYmIG1fcmVuZGVyZXItPm5vZGUoKSAmJiBtX3JlbmRlcmVyLT5ub2RlKCkt Pmhhc1RhZ05hbWUoaW5wdXRUYWcpKSB7Ci0gICAgICAgIEhUTUxJbnB1dEVsZW1lbnQqIGlucHV0 ID0gc3RhdGljX2Nhc3Q8SFRNTElucHV0RWxlbWVudCo+KG1fcmVuZGVyZXItPm5vZGUoKSk7Cisg ICAgTm9kZSogZWxlbWVudE5vZGUgPSBub2RlKCk7CisgICAgaWYgKHJvbGVWYWx1ZSgpID09IEJ1 dHRvblJvbGUgJiYgZWxlbWVudE5vZGUgJiYgZWxlbWVudE5vZGUtPmhhc1RhZ05hbWUoaW5wdXRU YWcpKSB7CisgICAgICAgIEhUTUxJbnB1dEVsZW1lbnQqIGlucHV0ID0gc3RhdGljX2Nhc3Q8SFRN TElucHV0RWxlbWVudCo+KGVsZW1lbnROb2RlKTsKICAgICAgICAgcmV0dXJuIGlucHV0LT5pbnB1 dFR5cGUoKSA9PSBIVE1MSW5wdXRFbGVtZW50OjpJTUFHRTsKICAgICB9CiAgICAgCkBAIC01OTEs OCArNTkyLDExIEBAIGJvb2wgQWNjZXNzaWJpbGl0eVJlbmRlck9iamVjdDo6aXNJbmRldGUKIGJv b2wgQWNjZXNzaWJpbGl0eVJlbmRlck9iamVjdDo6aXNOYXRpdmVDaGVja2JveE9yUmFkaW8oKSBj b25zdAogewogICAgIE5vZGUqIGVsZW1lbnROb2RlID0gbm9kZSgpOwotICAgIGlmIChlbGVtZW50 Tm9kZSAmJiBlbGVtZW50Tm9kZS0+aXNFbGVtZW50Tm9kZSgpKQotICAgICAgICByZXR1cm4gdG9J bnB1dEVsZW1lbnQoc3RhdGljX2Nhc3Q8RWxlbWVudCo+KGVsZW1lbnROb2RlKSk7CisgICAgaWYg KGVsZW1lbnROb2RlICYmIGVsZW1lbnROb2RlLT5pc0VsZW1lbnROb2RlKCkpIHsKKyAgICAgICAg SW5wdXRFbGVtZW50KiBpbnB1dCA9IHRvSW5wdXRFbGVtZW50KHN0YXRpY19jYXN0PEVsZW1lbnQq PihlbGVtZW50Tm9kZSkpOworICAgICAgICBpZiAoaW5wdXQpCisgICAgICAgICAgICByZXR1cm4g aW5wdXQtPmlzQ2hlY2tib3goKSB8fCBpbnB1dC0+aXNSYWRpb0J1dHRvbigpOworICAgIH0KICAg ICAKICAgICByZXR1cm4gZmFsc2U7CiB9CkluZGV4OiBXZWJDb3JlL2RvbS9JbnB1dEVsZW1lbnQu aAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2RvbS9JbnB1dEVsZW1lbnQuaAkocmV2aXNpb24gNjUx NzcpCisrKyBXZWJDb3JlL2RvbS9JbnB1dEVsZW1lbnQuaAkod29ya2luZyBjb3B5KQpAQCAtNDMs NiArNDMsOSBAQCBwdWJsaWM6CiAgICAgdmlydHVhbCBib29sIGlzUGFzc3dvcmRGaWVsZCgpIGNv bnN0ID0gMDsKICAgICB2aXJ0dWFsIGJvb2wgaXNTZWFyY2hGaWVsZCgpIGNvbnN0ID0gMDsKICAg ICB2aXJ0dWFsIGJvb2wgaXNUZXh0RmllbGQoKSBjb25zdCA9IDA7CisgICAgdmlydHVhbCBib29s IGlzUmFkaW9CdXR0b24oKSBjb25zdCA9IDA7CisgICAgdmlydHVhbCBib29sIGlzQ2hlY2tib3go KSBjb25zdCA9IDA7CisgICAgCiAgICAgdmlydHVhbCBib29sIHN1cHBvcnRzTWF4TGVuZ3RoKCkg Y29uc3QgPSAwOwogICAgIHZpcnR1YWwgYm9vbCBoYXNTcGluQnV0dG9uKCkgY29uc3QgeyByZXR1 cm4gZmFsc2U7IH0KICNpZiBFTkFCTEUoSU5QVVRfU1BFRUNIKQpJbmRleDogV2ViQ29yZS9odG1s L0hUTUxJbnB1dEVsZW1lbnQuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2h0bWwvSFRNTElucHV0 RWxlbWVudC5oCShyZXZpc2lvbiA2NTE3NykKKysrIFdlYkNvcmUvaHRtbC9IVE1MSW5wdXRFbGVt ZW50LmgJKHdvcmtpbmcgY29weSkKQEAgLTEwMSw2ICsxMDEsNyBAQCBwdWJsaWM6CiAgICAgdmly dHVhbCBib29sIGlzU2VhcmNoRmllbGQoKSBjb25zdCB7IHJldHVybiBtX3R5cGUgPT0gU0VBUkNI OyB9CiAgICAgdmlydHVhbCBib29sIGlzSW5wdXRUeXBlSGlkZGVuKCkgY29uc3QgeyByZXR1cm4g bV90eXBlID09IEhJRERFTjsgfQogICAgIHZpcnR1YWwgYm9vbCBpc1Bhc3N3b3JkRmllbGQoKSBj b25zdCB7IHJldHVybiBtX3R5cGUgPT0gUEFTU1dPUkQ7IH0KKyAgICB2aXJ0dWFsIGJvb2wgaXND aGVja2JveCgpIGNvbnN0IHsgcmV0dXJuIG1fdHlwZSA9PSBDSEVDS0JPWDsgfQogICAgIGJvb2wg aXNUZWxlcGhvbmVGaWVsZCgpIGNvbnN0IHsgcmV0dXJuIG1fdHlwZSA9PSBURUxFUEhPTkU7IH0K ICAgICBib29sIGlzTnVtYmVyRmllbGQoKSBjb25zdCB7IHJldHVybiBtX3R5cGUgPT0gTlVNQkVS OyB9CiAgICAgYm9vbCBpc0VtYWlsRmllbGQoKSBjb25zdCB7IHJldHVybiBtX3R5cGUgPT0gRU1B SUw7IH0KSW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU ZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9uIDY1MTgzKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9n CSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTAtMDgtMTEgIENocmlzIEZsZWl6 YWNoICA8Y2ZsZWl6YWNoQGFwcGxlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg KE9PUFMhKS4KKworICAgICAgICBBWDogaXNOYXRpdmVDaGVja2JveCBkb2VzIG5vdCB3b3JrIGFz IGFkdmVydGlzZWQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp P2lkPTQzODcyCisKKyAgICAgICAgKiBwbGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS9uYXRpdmUt dnMtbm9ubmF0aXZlLWNoZWNrYm94ZXMtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBw bGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS9uYXRpdmUtdnMtbm9ubmF0aXZlLWNoZWNrYm94ZXMu aHRtbDogQWRkZWQuCisKIDIwMTAtMDgtMTEgIE5hdGUgQ2hhcGluICA8amFwaGV0QGNocm9taXVt Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEaW1pdHJpIEdsYXprb3YuCkluZGV4OiBMYXlv dXRUZXN0cy9wbGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS9uYXRpdmUtdnMtbm9ubmF0aXZlLWNo ZWNrYm94ZXMtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL3BsYXRmb3Jt L21hYy9hY2Nlc3NpYmlsaXR5L25hdGl2ZS12cy1ub25uYXRpdmUtY2hlY2tib3hlcy1leHBlY3Rl ZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvYWNjZXNzaWJp bGl0eS9uYXRpdmUtdnMtbm9ubmF0aXZlLWNoZWNrYm94ZXMtZXhwZWN0ZWQudHh0CShyZXZpc2lv biAwKQpAQCAtMCwwICsxLDE1IEBACitjaGVjazEKK2NoZWNrMgorY2hlY2syCitUaGlzIHRlc3Rz IHRoYXQgaW5wdXQgdHlwZXMgbWFzcXVlcmFkaW5nIGFzIGNoZWNrYm94ZXMgd2lsbCB1c2UgdGhl IGFyaWEtY2hlY2tlZCB2YWx1ZXMgZm9yIGl0cyB2YWx1ZS4KKworT24gc3VjY2VzcywgeW91IHdp bGwgc2VlIGEgc2VyaWVzIG9mICJQQVNTIiBtZXNzYWdlcywgZm9sbG93ZWQgYnkgIlRFU1QgQ09N UExFVEUiLgorCisKK1BBU1MgYWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIuZm9jdXNlZEVsZW1lbnQu aW50VmFsdWUgaXMgMQorUEFTUyBhY2Nlc3NpYmlsaXR5Q29udHJvbGxlci5mb2N1c2VkRWxlbWVu dC5pbnRWYWx1ZSBpcyAwCitQQVNTIGFjY2Vzc2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRFbGVt ZW50LmludFZhbHVlIGlzIDIKK1BBU1Mgc3VjY2Vzc2Z1bGx5UGFyc2VkIGlzIHRydWUKKworVEVT VCBDT01QTEVURQorCkluZGV4OiBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0 eS9uYXRpdmUtdnMtbm9ubmF0aXZlLWNoZWNrYm94ZXMuaHRtbAo9PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlv dXRUZXN0cy9wbGF0Zm9ybS9tYWMvYWNjZXNzaWJpbGl0eS9uYXRpdmUtdnMtbm9ubmF0aXZlLWNo ZWNrYm94ZXMuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL3BsYXRmb3JtL21hYy9h Y2Nlc3NpYmlsaXR5L25hdGl2ZS12cy1ub25uYXRpdmUtY2hlY2tib3hlcy5odG1sCShyZXZpc2lv biAwKQpAQCAtMCwwICsxLDQ0IEBACis8IURPQ1RZUEUgSFRNTCBQVUJMSUMgIi0vL0lFVEYvL0RU RCBIVE1MLy9FTiI+Cis8aHRtbD4KKzxoZWFkPgorPGxpbmsgcmVsPSJzdHlsZXNoZWV0IiBocmVm PSIuLi8uLi8uLi9mYXN0L2pzL3Jlc291cmNlcy9qcy10ZXN0LXN0eWxlLmNzcyI+Cis8c2NyaXB0 PgordmFyIHN1Y2Nlc3NmdWxseVBhcnNlZCA9IGZhbHNlOworPC9zY3JpcHQ+Cis8c2NyaXB0IHNy Yz0iLi4vLi4vLi4vZmFzdC9qcy9yZXNvdXJjZXMvanMtdGVzdC1wcmUuanMiPjwvc2NyaXB0Pgor PC9oZWFkPgorPGJvZHkgaWQ9ImJvZHkiPgorCis8aW5wdXQgdHlwZT0iaW1hZ2UiIHJvbGU9ImNo ZWNrYm94IiBhcmlhLWNoZWNrZWQ9InRydWUiIGlkPSJjaGVjazEiPmNoZWNrMTxCUj4KKzxpbnB1 dCB0eXBlPSJpbWFnZSIgcm9sZT0iY2hlY2tib3giIGFyaWEtY2hlY2tlZD0ibWl4ZWQiIGlkPSJj aGVjazIiPmNoZWNrMjxCUj4KKzxpbnB1dCB0eXBlPSJpbWFnZSIgcm9sZT0iY2hlY2tib3giIGlk PSJjaGVjazMiPmNoZWNrMjxCUj4KKworCis8cCBpZD0iZGVzY3JpcHRpb24iPjwvcD4KKzxkaXYg aWQ9ImNvbnNvbGUiPjwvZGl2PgorCis8c2NyaXB0PgorCisgICAgZGVzY3JpcHRpb24oIlRoaXMg dGVzdHMgdGhhdCBpbnB1dCB0eXBlcyBtYXNxdWVyYWRpbmcgYXMgY2hlY2tib3hlcyB3aWxsIHVz ZSB0aGUgYXJpYS1jaGVja2VkIHZhbHVlcyBmb3IgaXRzIHZhbHVlLiIpOworCisgICAgaWYgKHdp bmRvdy5hY2Nlc3NpYmlsaXR5Q29udHJvbGxlcikgeworCisgICAgICAgICAgLy8gYXJpYS1zdGF0 ZT10cnVlCisgICAgICAgICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImNoZWNrMSIpLmZvY3Vz KCk7CisgICAgICAgICAgc2hvdWxkQmUoImFjY2Vzc2liaWxpdHlDb250cm9sbGVyLmZvY3VzZWRF bGVtZW50LmludFZhbHVlIiwgIjEiKTsKKworICAgICAgICAgIC8vIGFyaWEtc3RhdGU9ZmFsc2UK KyAgICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgiY2hlY2szIikuZm9jdXMoKTsKKyAg ICAgICAgICBzaG91bGRCZSgiYWNjZXNzaWJpbGl0eUNvbnRyb2xsZXIuZm9jdXNlZEVsZW1lbnQu aW50VmFsdWUiLCAiMCIpOworCisgICAgICAgICAgLy8gYXJpYS1zdGF0ZT1taXhlZAorICAgICAg ICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJjaGVjazIiKS5mb2N1cygpOworICAgICAgICAg IHNob3VsZEJlKCJhY2Nlc3NpYmlsaXR5Q29udHJvbGxlci5mb2N1c2VkRWxlbWVudC5pbnRWYWx1 ZSIsICIyIik7CisgICAgfQorCisgICAgc3VjY2Vzc2Z1bGx5UGFyc2VkID0gdHJ1ZTsKKzwvc2Ny aXB0PgorCis8c2NyaXB0IHNyYz0iLi4vLi4vLi4vZmFzdC9qcy9yZXNvdXJjZXMvanMtdGVzdC1w b3N0LmpzIj48L3NjcmlwdD4KKzwvYm9keT4KKzwvaHRtbD4K