138083 2014-10-26 08:00:14 -0700 [EFL] Apply adjustScrollPositionWithinRange() to scroll offset for ScrollingCoordinatorCoordinatedGraphics. 2015-08-27 04:14:00 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Other Linux RESOLVED DUPLICATE 146959 P2 Normal --- 1 hyuki.kim webkit-unassigned cmarcelo commit-queue gyuyoung.kim jamesr luiz ryuan.choi tonikitoo oldest_to_newest 1044180 0 hyuki.kim 2014-10-26 08:00:14 -0700 Apply adjustScrollPositionWithinRange() to the new offset on ScrollView's scrollTo() because it can apply the new offset even though the offset value is not in the scrollable range when delegatesScrolling is used and scroll is activated by wheel event. 1044181 1 240465 hyuki.kim 2014-10-26 08:07:56 -0700 Created attachment 240465 Patch 1044370 2 240465 gyuyoung.kim 2014-10-27 17:50:07 -0700 Comment on attachment 240465 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240465&action=review > Source/WebCore/ChangeLog:12 > + No new tests because of no behavior change. I'm not sure this change won't change behavior. At least, this patch should pass WKViewScrollTo.cpp test. 1044374 3 240465 gyuyoung.kim 2014-10-27 17:53:57 -0700 Comment on attachment 240465 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240465&action=review >> Source/WebCore/ChangeLog:12 >> + No new tests because of no behavior change. > > I'm not sure this change won't change behavior. At least, this patch should pass WKViewScrollTo.cpp test. Can't you add new test in order to test this change ? It looks we need to add a test which can handle when the offset value is not within the scrollable range. 1045051 4 hyuki.kim 2014-10-30 10:22:13 -0700 (In reply to comment #3) > Comment on attachment 240465 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=240465&action=review > > >> Source/WebCore/ChangeLog:12 > >> + No new tests because of no behavior change. > > > > I'm not sure this change won't change behavior. At least, this patch should pass WKViewScrollTo.cpp test. > > Can't you add new test in order to test this change ? It looks we need to > add a test which can handle when the offset value is not within the > scrollable range. Ok, let me check the WKViewScrollTo.cpp. However, this is a just bug fix for wheel scroll using mouse. Of course, you can reproduce issue easily using webkit.org. Just go to www.webkit.org and do the wheel scroll to the upper direction 10 or more times, then do the wheel scroll to the down direction. you can notice that it will not scroll down for 10 times even though there is no webview changed. 1066426 5 240465 gyuyoung.kim 2015-02-03 07:38:46 -0800 Comment on attachment 240465 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240465&action=review >>>> Source/WebCore/ChangeLog:12 >>>> + No new tests because of no behavior change. >>> >>> I'm not sure this change won't change behavior. At least, this patch should pass WKViewScrollTo.cpp test. >> >> Can't you add new test in order to test this change ? It looks we need to add a test which can handle when the offset value is not within the scrollable range. > > Ok, let me check the WKViewScrollTo.cpp. > > However, this is a just bug fix for wheel scroll using mouse. > Of course, you can reproduce issue easily using webkit.org. > Just go to www.webkit.org and do the wheel scroll to the upper direction 10 or more times, then do the wheel scroll to the down direction. you can notice that it will not scroll down for 10 times even though there is no webview changed. > Ok, let me check the WKViewScrollTo.cpp. Any update on this check ? 1084872 6 250657 hyuki.kim 2015-04-13 10:49:27 -0700 Created attachment 250657 Patch 1084875 7 hyuki.kim 2015-04-13 10:54:47 -0700 As this patch just touched code enclosed by TILED_BACKING_STORE, it will not effect for both iOS and GTK for now. 1100673 8 254575 hyuki.kim 2015-06-09 09:52:51 -0700 Created attachment 254575 Patch 1100688 9 hyuki.kim 2015-06-09 10:09:02 -0700 * Issue description - go to www.webkit.org by using MiniBrowser - scroll down and repeat scroll up using mouse wheel. - scroll up was applied several times although rendered view was not changed. . you can check it by trying scroll down as scroll down will not change rendered view either. 1100893 10 ryuan.choi 2015-06-09 19:24:51 -0700 (In reply to comment #4) > (In reply to comment #3) > > Comment on attachment 240465 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=240465&action=review > > > > >> Source/WebCore/ChangeLog:12 > > >> + No new tests because of no behavior change. > > > > > > I'm not sure this change won't change behavior. At least, this patch should pass WKViewScrollTo.cpp test. > > > > Can't you add new test in order to test this change ? It looks we need to > > add a test which can handle when the offset value is not within the > > scrollable range. > > Ok, let me check the WKViewScrollTo.cpp. > > However, this is a just bug fix for wheel scroll using mouse. > Of course, you can reproduce issue easily using webkit.org. > Just go to www.webkit.org and do the wheel scroll to the upper direction 10 > or more times, then do the wheel scroll to the down direction. you can > notice that it will not scroll down for 10 times even though there is no > webview changed. Why don't you make the test case like you mentioned? In my two cents, we can make the test case that scroll more than two times to invalid position and scroll few pixel (but valid). I checked some use cases and reproduced this issue with very simple test cases that use windows.scrollTo and/or with fixed elements. 1101033 11 hyuki.kim 2015-06-10 09:03:50 -0700 (In reply to comment #10) > (In reply to comment #4) > > (In reply to comment #3) > > > Comment on attachment 240465 [details] > > > Patch > > > > > > View in context: > > > https://bugs.webkit.org/attachment.cgi?id=240465&action=review > > > > > > >> Source/WebCore/ChangeLog:12 > > > >> + No new tests because of no behavior change. > > > > > > > > I'm not sure this change won't change behavior. At least, this patch should pass WKViewScrollTo.cpp test. > > > > > > Can't you add new test in order to test this change ? It looks we need to > > > add a test which can handle when the offset value is not within the > > > scrollable range. > > > > Ok, let me check the WKViewScrollTo.cpp. > > > > However, this is a just bug fix for wheel scroll using mouse. > > Of course, you can reproduce issue easily using webkit.org. > > Just go to www.webkit.org and do the wheel scroll to the upper direction 10 > > or more times, then do the wheel scroll to the down direction. you can > > notice that it will not scroll down for 10 times even though there is no > > webview changed. > > Why don't you make the test case like you mentioned? > > In my two cents, > we can make the test case that scroll more than two times to invalid > position and scroll few pixel (but valid). > > I checked some use cases and reproduced this issue with very simple test > cases that use windows.scrollTo and/or with fixed elements. Ok, let me try. :) 1110073 12 254575 gyuyoung.kim 2015-07-16 20:18:20 -0700 Comment on attachment 254575 Patch Clearing r? until new patch is uploaded. 1121281 13 hyuki.kim 2015-08-27 04:14:00 -0700 Confirmed that fixed by 146959. *** This bug has been marked as a duplicate of bug 146959 *** 240465 2014-10-26 08:07:56 -0700 2015-04-13 10:49:19 -0700 Patch bug-138083-20141027000803.patch text/plain 1736 hyuki.kim U3VidmVyc2lvbiBSZXZpc2lvbjogMTc1MjA5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggODg0NTY1NGY3NzVjMTEx YjBhOGJmNDBhNjc4ZmM1ODMxODY5NmNiYi4uNWYxZGZjNWEwNzU2ZWM1OTg2MjIzYzEwZjhmNTE2 NmExYWYyMDU1ZSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5IEBACisyMDE0LTEwLTI2ICBLd2Fu Z0h5dWsgS2ltICA8aHl1a2kua2ltQHNhbXN1bmcuY29tPgorCisgICAgICAgIFtFRkxdIEFwcGx5 IGFkanVzdFNjcm9sbFBvc2l0aW9uV2l0aGluUmFuZ2UoKSB0byB0aGUgbmV3IG9mZnNldCBvbiBT Y3JvbGxWaWV3J3Mgc2Nyb2xsVG8oKS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTEzODA4MworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q UyEpLgorCisgICAgICAgIEFwcGx5IGFkanVzdFNjcm9sbFBvc2l0aW9uV2l0aGluUmFuZ2UoKSB0 byB0aGUgbmV3IG9mZnNldCBvbiBTY3JvbGxWaWV3J3Mgc2Nyb2xsVG8oKQorICAgICAgICBiZWNh dXNlIGl0IGNhbiBhcHBseSB0aGUgbmV3IG9mZnNldCBldmVuIHRob3VnaCB0aGUgb2Zmc2V0IHZh bHVlIGlzIG5vdCBpbiB0aGUgc2Nyb2xsYWJsZSByYW5nZQorICAgICAgICB3aGVuIGRlbGVnYXRl c1Njcm9sbGluZyBpcyB1c2VkIGFuZCBzY3JvbGwgaXMgYWN0aXZhdGVkIGJ5IHdoZWVsIGV2ZW50 LgorCisgICAgICAgIE5vIG5ldyB0ZXN0cyBiZWNhdXNlIG9mIG5vIGJlaGF2aW9yIGNoYW5nZS4K KworICAgICAgICAqIHBsYXRmb3JtL1Njcm9sbFZpZXcuY3BwOgorICAgICAgICAoV2ViQ29yZTo6 U2Nyb2xsVmlldzo6c2Nyb2xsVG8pOgorCiAyMDE0LTEwLTI1ICBCZW5qYW1pbiBQb3VsYWluICA8 YmVuamFtaW5Ad2Via2l0Lm9yZz4KIAogICAgICAgICBSZW1vdmUgYSB1c2VsZXNzIHZhcmlhYmxl IGZyb20gU2VsZWN0b3JDb2RlR2VuZXJhdG9yOjpnZW5lcmF0ZUVsZW1lbnRNYXRjaGVzU2VsZWN0 b3JMaXN0KCkKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL1Njcm9sbFZpZXcu Y3BwIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2Nyb2xsVmlldy5jcHAKaW5kZXggNDZkMWI2 YmJkYzMwMGU1NDcxMzc4NTgwMmJkZjVhN2FhOTY5NzcyMC4uNTNjZWM0YWYwOWQ4Zjg0YmRiY2Fj ZTkwZDI1ZGQ1ZjU3MTQwZDRmYSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0v U2Nyb2xsVmlldy5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2Nyb2xsVmlldy5j cHAKQEAgLTQ2OSw3ICs0NjksOCBAQCB2b2lkIFNjcm9sbFZpZXc6OnNjcm9sbFRvKGNvbnN0IElu dFNpemUmIG5ld09mZnNldCkKIAogI2lmIFVTRShUSUxFRF9CQUNLSU5HX1NUT1JFKQogICAgIGlm IChkZWxlZ2F0ZXNTY3JvbGxpbmcoKSkgewotICAgICAgICByZXF1ZXN0U2Nyb2xsUG9zaXRpb25V cGRhdGUoSW50UG9pbnQobmV3T2Zmc2V0KSk7CisgICAgICAgIEludFBvaW50IG5ld1Bvc2l0aW9u ID0gYWRqdXN0U2Nyb2xsUG9zaXRpb25XaXRoaW5SYW5nZShJbnRQb2ludChuZXdPZmZzZXQpKTsK KyAgICAgICAgcmVxdWVzdFNjcm9sbFBvc2l0aW9uVXBkYXRlKG5ld1Bvc2l0aW9uKTsKICAgICAg ICAgcmV0dXJuOwogICAgIH0KICNlbmRpZgo= 250657 2015-04-13 10:49:27 -0700 2015-06-09 09:52:43 -0700 Patch bug-138083-20150414024832.patch text/plain 1684 hyuki.kim U3VidmVyc2lvbiBSZXZpc2lvbjogMTgyNzI2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZDYyMThlM2YxNzY0NDli ZWY5MWVlZWIzNzI3ODMyZmY4YTBkM2JiZS4uMzEyY2I5YTE1NGFjYjI4MjY0OTc1MGU1MWNlM2Ew ODRiNzcxNDJlZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5IEBACisyMDE1LTA0LTEzICBLd2Fu Z0h5dWsgS2ltICA8aHl1a2kua2ltQHNhbXN1bmcuY29tPgorCisgICAgICAgIFtFRkxdIEFwcGx5 IGFkanVzdFNjcm9sbFBvc2l0aW9uV2l0aGluUmFuZ2UoKSB0byB0aGUgbmV3IG9mZnNldCBvbiBT Y3JvbGxWaWV3J3Mgc2Nyb2xsVG8oKS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTEzODA4MworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q UyEpLgorCisgICAgICAgIEFwcGx5IGFkanVzdFNjcm9sbFBvc2l0aW9uV2l0aGluUmFuZ2UoKSB0 byB0aGUgbmV3IG9mZnNldCBvbiBTY3JvbGxWaWV3J3Mgc2Nyb2xsVG8oKQorICAgICAgICBiZWNh dXNlIGl0IGNhbiBhcHBseSB0aGUgbmV3IG9mZnNldCBldmVuIHRob3VnaCB0aGUgb2Zmc2V0IHZh bHVlIGlzIG5vdCBpbiB0aGUgc2Nyb2xsYWJsZSByYW5nZQorICAgICAgICB3aGVuIGRlbGVnYXRl c1Njcm9sbGluZyBpcyB1c2VkIGFuZCBzY3JvbGwgaXMgYWN0aXZhdGVkIGJ5IHdoZWVsIGV2ZW50 LgorCisgICAgICAgIE5vIG5ldyB0ZXN0cyBiZWNhdXNlIG9mIG5vIGJlaGF2aW9yIGNoYW5nZS4K KworICAgICAgICAqIHBsYXRmb3JtL1Njcm9sbFZpZXcuY3BwOgorICAgICAgICAoV2ViQ29yZTo6 U2Nyb2xsVmlldzo6c2Nyb2xsVG8pOgorCiAyMDE1LTA0LTEzICBTZXJnaW8gVmlsbGFyIFNlbmlu ICA8c3ZpbGxhckBpZ2FsaWEuY29tPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIGJ1aWxkIGZpeCBh ZnRlciByMTgyNzA0LgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2Nyb2xs Vmlldy5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TY3JvbGxWaWV3LmNwcAppbmRleCAz YzgyNjdhNDM1NWZkN2RlYjE3MmI3MTg1ZWRjZjU5MDhiNzVmOGE4Li44MDVmMzQyZjc4YzE0ZDJh MWU2NTAwZTI1ZDE4M2Y5YWY1YTU5ZWE0IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0 Zm9ybS9TY3JvbGxWaWV3LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TY3JvbGxW aWV3LmNwcApAQCAtNTA1LDcgKzUwNSw4IEBAIHZvaWQgU2Nyb2xsVmlldzo6c2Nyb2xsVG8oY29u c3QgSW50U2l6ZSYgbmV3T2Zmc2V0KQogCiAjaWYgVVNFKFRJTEVEX0JBQ0tJTkdfU1RPUkUpCiAg ICAgaWYgKGRlbGVnYXRlc1Njcm9sbGluZygpKSB7Ci0gICAgICAgIHJlcXVlc3RTY3JvbGxQb3Np dGlvblVwZGF0ZShJbnRQb2ludChuZXdPZmZzZXQpKTsKKyAgICAgICAgSW50UG9pbnQgbmV3UG9z aXRpb24gPSBhZGp1c3RTY3JvbGxQb3NpdGlvbldpdGhpblJhbmdlKEludFBvaW50KG5ld09mZnNl dCkpOworICAgICAgICByZXF1ZXN0U2Nyb2xsUG9zaXRpb25VcGRhdGUobmV3UG9zaXRpb24pOwog ICAgICAgICByZXR1cm47CiAgICAgfQogI2VuZGlmCg== 254575 2015-06-09 09:52:51 -0700 2015-07-16 20:18:20 -0700 Patch bug-138083-20150610015224.patch text/plain 2362 hyuki.kim U3VidmVyc2lvbiBSZXZpc2lvbjogMTg1MzYzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNWRhYTRkNDRjZWU2Zjhl ZDBhNmZkNmE1MzRiMGNlM2U5MzhmNmI1NC4uYTdkNWYwZDNiMDc0Y2Q3M2YyMzg0ZTMwZGY5MmQ4 MmFiZGU0NzY0MCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIxIEBACisyMDE1LTA2LTA5ICBLd2Fu Z0h5dWsgS2ltICA8aHl1a2kua2ltQHNhbXN1bmcuY29tPgorCisgICAgICAgIHNjcm9sbCBwb3Np dGlvbiBhZGp1c3RtZW50CisKKyAgICAgICAgW0VGTF0gQXBwbHkgYWRqdXN0U2Nyb2xsUG9zaXRp b25XaXRoaW5SYW5nZSgpIHRvIHNjcm9sbCBvZmZzZXQgZm9yIFNjcm9sbGluZ0Nvb3JkaW5hdG9y Q29vcmRpbmF0ZWRHcmFwaGljcy4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv d19idWcuY2dpP2lkPTEzODA4MworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp LgorCisgICAgICAgIEFwcGx5IGFkanVzdFNjcm9sbFBvc2l0aW9uV2l0aGluUmFuZ2UoKSB0byB0 aGUgc2Nyb2xsIG9mZnNldCBvbiBTY3JvbGxpbmdDb29yZGluYXRvckNvb3JkaW5hdGVkR3JhcGhp Y3MKKyAgICAgICAgYmVjYXVzZSBpdCBqdXN0IGFwcGx5IHRoZSBuZXcgc2Nyb2xsIG9mZnNldCBl dmVuIHRob3VnaCB0aGUgb2Zmc2V0IHZhbHVlIGlzIG5vdCBpbiB0aGUgc2Nyb2xsYWJsZSByYW5n ZQorICAgICAgICB3aGVuIGRlbGVnYXRlc1Njcm9sbGluZyBpcyB1c2VkIGZvciBDT09SRElOQVRF RF9HUkFQSElDUyBhbmQgc2Nyb2xsIGlzIGFjdGl2YXRlZCBieSB3aGVlbCBldmVudC4KKworICAg ICAgICBObyBuZXcgdGVzdHMsIG5vIGJlaGF2aW9yIGNoYW5nZSBhcyBzY3JvbGwgb2Zmc2V0IGFk anVzdG1lbnQgd2FzIGp1c3QgYXBwbGllZC4KKworICAgICAgICAqIHBhZ2Uvc2Nyb2xsaW5nL2Nv b3JkaW5hdGVkZ3JhcGhpY3MvU2Nyb2xsaW5nQ29vcmRpbmF0b3JDb29yZGluYXRlZEdyYXBoaWNz LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlNjcm9sbGluZ0Nvb3JkaW5hdG9yQ29vcmRpbmF0ZWRH cmFwaGljczo6cmVxdWVzdFNjcm9sbFBvc2l0aW9uVXBkYXRlKToKKwogMjAxNS0wNi0wOSAgQ3Nh YmEgT3N6dHJvZ29uw6FjICA8b3NzeUB3ZWJraXQub3JnPgogCiAgICAgICAgIFtjbWFrZV0gRml4 IHRoZSBzdHlsZSBpc3N1ZXMgaW4gY21ha2UgcHJvamVjdCBmaWxlcwpkaWZmIC0tZ2l0IGEvU291 cmNlL1dlYkNvcmUvcGFnZS9zY3JvbGxpbmcvY29vcmRpbmF0ZWRncmFwaGljcy9TY3JvbGxpbmdD b29yZGluYXRvckNvb3JkaW5hdGVkR3JhcGhpY3MuY3BwIGIvU291cmNlL1dlYkNvcmUvcGFnZS9z Y3JvbGxpbmcvY29vcmRpbmF0ZWRncmFwaGljcy9TY3JvbGxpbmdDb29yZGluYXRvckNvb3JkaW5h dGVkR3JhcGhpY3MuY3BwCmluZGV4IDMxNWI0MjM2ZjljZGViOTFjMjEzZmFkYWI2MDA0NTZiMjc2 ODM3ZmQuLjlmMWE1YjI0ZjQyYTQ3Mzg0ZDY3NjdlYWU4M2FhZDdkODEwOTRiYzIgMTAwNjQ0Ci0t LSBhL1NvdXJjZS9XZWJDb3JlL3BhZ2Uvc2Nyb2xsaW5nL2Nvb3JkaW5hdGVkZ3JhcGhpY3MvU2Ny b2xsaW5nQ29vcmRpbmF0b3JDb29yZGluYXRlZEdyYXBoaWNzLmNwcAorKysgYi9Tb3VyY2UvV2Vi Q29yZS9wYWdlL3Njcm9sbGluZy9jb29yZGluYXRlZGdyYXBoaWNzL1Njcm9sbGluZ0Nvb3JkaW5h dG9yQ29vcmRpbmF0ZWRHcmFwaGljcy5jcHAKQEAgLTExNyw3ICsxMTcsOCBAQCBib29sIFNjcm9s bGluZ0Nvb3JkaW5hdG9yQ29vcmRpbmF0ZWRHcmFwaGljczo6cmVxdWVzdFNjcm9sbFBvc2l0aW9u VXBkYXRlKEZyYW1lVgogICAgIGlmICghZnJhbWVWaWV3LmRlbGVnYXRlc1Njcm9sbGluZygpKQog ICAgICAgICByZXR1cm4gZmFsc2U7CiAKLSAgICBmcmFtZVZpZXcuc2V0Rml4ZWRWaXNpYmxlQ29u dGVudFJlY3QoSW50UmVjdChzY3JvbGxQb3NpdGlvbiwgZnJhbWVWaWV3LnZpc2libGVDb250ZW50 UmVjdCgpLnNpemUoKSkpOworICAgIEludFBvaW50IGFkanVzdGVkUG9zaXRpb24gPSBmcmFtZVZp ZXcuYWRqdXN0U2Nyb2xsUG9zaXRpb25XaXRoaW5SYW5nZShzY3JvbGxQb3NpdGlvbik7CisgICAg ZnJhbWVWaWV3LnNldEZpeGVkVmlzaWJsZUNvbnRlbnRSZWN0KEludFJlY3QoYWRqdXN0ZWRQb3Np dGlvbiwgZnJhbWVWaWV3LnZpc2libGVDb250ZW50UmVjdCgpLnNpemUoKSkpOwogICAgIGZyYW1l Vmlldy5ob3N0V2luZG93KCktPmRlbGVnYXRlZFNjcm9sbFJlcXVlc3RlZChzY3JvbGxQb3NpdGlv bik7CiAgICAgcmV0dXJuIHRydWU7CiB9Cg==