88762 2012-06-11 00:24:37 -0700 Invalidation rects are not scaled for position:fixed elements 2012-10-17 13:13:12 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified ASSIGNED P2 Normal --- 66687 1 abarth abarth andersca ap bdakin hclam jamesr klobag simon.fraser skyostil skyostil tonikitoo oldest_to_newest 645782 0 abarth 2012-06-11 00:24:37 -0700 Invalidation rects are not scaled for position:fixed elements 645784 1 146797 abarth 2012-06-11 00:25:20 -0700 Created attachment 146797 Patch (needs tests) 645786 2 abarth 2012-06-11 00:27:12 -0700 This patch exists in the chromium-android branch, but I'm somewhat out of my area here. Do you know if this patch is correct? If so, how do we write a test for it? Thanks! 645965 3 tonikitoo 2012-06-11 06:57:08 -0700 is this possible to come with a regression test? 645983 4 skyostil 2012-06-11 07:10:49 -0700 I think it should be possible to test this with window.internals.settings.setPageScaleFactor(), window.scrollBy() and changing the contents of the fixed position element. 646074 5 abarth 2012-06-11 10:28:04 -0700 Thanks Sami. I'll give that a try. 646513 6 146985 abarth 2012-06-11 18:14:41 -0700 Created attachment 146985 Attempt at a test 646515 7 abarth 2012-06-11 18:15:21 -0700 This test seems to pass without the patch. Any ideas what I'm doing wrong? Thanks! 646520 8 abarth 2012-06-11 18:21:16 -0700 Note: This test does note appear to execute the lines changed by this patch. 652466 9 skyostil 2012-06-19 08:23:47 -0700 Looks like calling scrollTo() in an inline <script> tag is a bit too early, so FrameView skips the fast blit scroll path and we don't end up in the changed code path. Moving the scrollTo() to repaintTest() makes it work. I tried the patch both on the master and android branches, and on the former it does the right thing, but on master updateRect ends up getting scaled twice. That is, m_repaintRect for the fixed RenderBox already includes page scale. I'm not sure where the discrepancy comes from, so this needs a little more investigation. By the way, I was a bit off on how to reproduce this: you only need to have a (non-layer) fixed position element and then apply a scroll. There's no need to invalidate the fixed position element itself, because this bug is about painting the stuff behind the fixed position element. 654833 10 148873 abarth 2012-06-21 13:32:55 -0700 Created attachment 148873 updated test 654834 11 148874 abarth 2012-06-21 13:33:26 -0700 Created attachment 148874 updated test 654837 12 abarth 2012-06-21 13:35:26 -0700 > Looks like calling scrollTo() in an inline <script> tag is a bit too early, so FrameView skips the fast blit scroll path and we don't end up in the changed code path. Moving the scrollTo() to repaintTest() makes it work. Ok. I've updated the test to do that. However, I can't get the bug to manifest. > I tried the patch both on the master and android branches, and on the former it does the right thing, but on master updateRect ends up getting scaled twice. That is, m_repaintRect for the fixed RenderBox already includes page scale. Have you successfully replicated the bug on trunk? > I'm not sure where the discrepancy comes from, so this needs a little more investigation. > > By the way, I was a bit off on how to reproduce this: you only need to have a (non-layer) fixed position element and then apply a scroll. There's no need to invalidate the fixed position element itself, because this bug is about painting the stuff behind the fixed position element. Ok. Let me try to update the test again. 654838 13 148876 abarth 2012-06-21 13:37:32 -0700 Created attachment 148876 updated test 654839 14 148874 abarth 2012-06-21 13:38:18 -0700 Comment on attachment 148874 updated test Ok. I stil can't reproduce the issue. If you're able to reproduce the issue, would you be willing to post the test case that reproduces it? 655465 15 skyostil 2012-06-22 08:02:52 -0700 Ok, I figured out what was causing the test to not show the problem: we need to be composited mode. As soon as that happens, the invalidation rects lose their page scale and the result is corrupted. Coincidentally the fix we have in the Android branch is wrong, because it invalidations in non-composited mode. Some more work is still needed, but at least we have a test now :) 655467 16 149031 skyostil 2012-06-22 08:03:42 -0700 Created attachment 149031 Working test 655549 17 abarth 2012-06-22 10:15:35 -0700 Ah, thanks for the test! 656191 18 149190 abarth 2012-06-24 02:21:49 -0700 Created attachment 149190 test with expected results 656192 19 abarth 2012-06-24 02:24:37 -0700 Unfortunately, that means this bug is in an area of the code I'm not familiar with. Sami, do you know who would be a good person to work on a proper fix? One possibility is hclam, who made the original change to the chromium-android branch. 661354 20 skyostil 2012-07-02 02:51:38 -0700 Sure, I can try to come up with fix. 744539 21 abarth 2012-10-17 13:13:12 -0700 I've posted a change to remove this diff from the downstream branch. 146797 2012-06-11 00:25:20 -0700 2012-06-24 02:16:29 -0700 Patch (needs tests) bug-88762-20120611002519.patch text/plain 1891 abarth SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDExOTk1NykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDI0IEBACisyMDEyLTA2LTExICBBZGFtIEJh cnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CisKKyAgICAgICAgSW52YWxpZGF0aW9uIHJlY3RzIGFy ZSBub3Qgc2NhbGVkIGZvciBwb3NpdGlvbjpmaXhlZCBlbGVtZW50cworICAgICAgICBodHRwczov L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODg3NjIKKworICAgICAgICBSZXZpZXdl ZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXaGVuIHBvc2l0aW9uOmZpeGVkIGVsZW1l bnRzIGRvbid0IGhhdmUgYSBzZXBhcmF0ZSBsYXllciBzY3JvbGxpbmcgdGhlCisgICAgICAgIHBh Z2Ugd2lsbCBpbnZhbGlkYXRlIHRoZSByb290IGxheWVyIGVhY2ggcmVjdGFuZ2xlIG9mIHRoZXNl IGVsZW1lbnRzLgorICAgICAgICBIb3dldmVyIHRoZSByZWN0YW5nbGVzIHdlcmUgbm90IHNjYWxl ZCBhY2NvcmRpbmcgdG8gdGhlIHBhZ2Ugc2NhbGUKKyAgICAgICAgZmFjdG9yLgorCisgICAgICAg IFRoaXMgZml4ZXMgdGhlIHJlbmRlcmluZyBwcm9ibGVtIGZvciBhIHNpbXBsZSBjYXNlIGxpa2Ug dGhpczoKKworICAgICAgICA8Ym9keSBzdHlsZT0id2lkdGg6MjAwMHB4O2hlaWdodDo0MDAwcHg7 Ij4KKyAgICAgICAgICA8ZGl2IHN0eWxlPSJwb3NpdGlvbjpmaXhlZDtsZWZ0OjIwcHg7dG9wOjIw cHg7Ij5GaXhlZDwvZGl2PgorICAgICAgICA8L2JvZHk+CisKKyAgICAgICAgKiBwYWdlL0ZyYW1l Vmlldy5jcHA6CisgICAgICAgIChXZWJDb3JlOjpGcmFtZVZpZXc6OnNjcm9sbENvbnRlbnRzRmFz dFBhdGgpOgorCiAyMDEyLTA2LTEwICBTaGVyaWZmIEJvdCAgPHdlYmtpdC5yZXZpZXcuYm90QGdt YWlsLmNvbT4KIAogICAgICAgICBVbnJldmlld2VkLCByb2xsaW5nIG91dCByMTE5OTU1LgpJbmRl eDogU291cmNlL1dlYkNvcmUvcGFnZS9GcmFtZVZpZXcuY3BwCj09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJj ZS9XZWJDb3JlL3BhZ2UvRnJhbWVWaWV3LmNwcAkocmV2aXNpb24gMTE5OTExKQorKysgU291cmNl L1dlYkNvcmUvcGFnZS9GcmFtZVZpZXcuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNDUzLDYgKzE0 NTMsMTAgQEAgYm9vbCBGcmFtZVZpZXc6OnNjcm9sbENvbnRlbnRzRmFzdFBhdGgoYwogI2VuZGlm CiAgICAgICAgIEludFJlY3QgdXBkYXRlUmVjdCA9IHBpeGVsU25hcHBlZEludFJlY3QocmVuZGVy Qm94LT5sYXllcigpLT5yZXBhaW50UmVjdEluY2x1ZGluZ05vbkNvbXBvc2l0aW5nRGVzY2VuZGFu dHMoKSk7CiAgICAgICAgIHVwZGF0ZVJlY3QgPSBjb250ZW50c1RvUm9vdFZpZXcodXBkYXRlUmVj dCk7CisgICAgICAgIEZsb2F0UmVjdCBzY2FsZWRSZWN0ID0gdXBkYXRlUmVjdDsKKyAgICAgICAg c2NhbGVkUmVjdC5zY2FsZShtX2ZyYW1lLT5mcmFtZVNjYWxlRmFjdG9yKCkpOworICAgICAgICB1 cGRhdGVSZWN0ID0gZW5jbG9zaW5nSW50UmVjdChzY2FsZWRSZWN0KTsKKyAgICAgICAgdXBkYXRl UmVjdCA9IGNvbnRlbnRzVG9XaW5kb3codXBkYXRlUmVjdCk7CiAgICAgICAgIGlmICghaXNDb21w b3NpdGVkQ29udGVudExheWVyICYmIGNsaXBzUmVwYWludHMoKSkKICAgICAgICAgICAgIHVwZGF0 ZVJlY3QuaW50ZXJzZWN0KHJlY3RUb1Njcm9sbCk7CiAgICAgICAgIGlmICghdXBkYXRlUmVjdC5p c0VtcHR5KCkpCg== 146985 2012-06-11 18:14:41 -0700 2012-06-21 13:33:39 -0700 Attempt at a test bug-88762-20120611181440.patch text/plain 943 abarth SW5kZXg6IExheW91dFRlc3RzL2Zhc3QvcmVwYWludC9mb28uaHRtbAo9PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBM YXlvdXRUZXN0cy9mYXN0L3JlcGFpbnQvZm9vLmh0bWwJKHJldmlzaW9uIDApCisrKyBMYXlvdXRU ZXN0cy9mYXN0L3JlcGFpbnQvZm9vLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMjQgQEAK KzxodG1sPgorPGhlYWQ+CisgICAgPGxpbmsgcmVsPSJzdHlsZXNoZWV0IiBocmVmPSJyZXNvdXJj ZXMvZGVmYXVsdC5jc3MiPgorICAgIDxzY3JpcHQgc3JjPSJyZXNvdXJjZXMvcmVwYWludC5qcyI+ PC9zY3JpcHQ+CisgICAgPHNjcmlwdD4KKyAgICAgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29u dHJvbGxlcikgeworICAgICAgICAgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCh0 cnVlKTsKKyAgICAgICAgICAgIGludGVybmFscy5zZXR0aW5ncy5zZXRQYWdlU2NhbGVGYWN0b3Io MTAsIDAsIDApOworICAgICAgICB9CisKKyAgICAgICAgZnVuY3Rpb24gcmVwYWludFRlc3QoKQor ICAgICAgICB7CisgICAgICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50c0J5VGFnTmFtZSgiZGl2 IilbMF0uc3R5bGUuYmFja2dyb3VuZENvbG9yID0gImdyZWVuIjsKKyAgICAgICAgfQorICAgIDwv c2NyaXB0PgorPC9oZWFkPgorCis8Ym9keSBzdHlsZT0iaGVpZ2h0OjIwMDBweDsiIG9ubG9hZD0i cnVuUmVwYWludFRlc3QoKSI+CisgICAgPGRpdiBzdHlsZT0icG9zaXRpb246Zml4ZWQ7bGVmdDoy MHB4O3RvcDoyMHB4O2JhY2tncm91bmQtY29sb3I6cmVkO2hlaWdodDoxMHB4O3dpZHRoOjEwcHgi PjwvZGl2PgorICAgIDxzY3JpcHQ+CisgICAgICAgIHdpbmRvdy5zY3JvbGxUbygwLCAxMCk7Cisg ICAgPC9zY3JpcHQ+Cis8L2JvZHk+Cis8L2h0bWw+Cg== 148873 2012-06-21 13:32:55 -0700 2012-06-21 13:33:48 -0700 updated test bug-88762-20120621133254.patch text/plain 28 abarth U3VidmVyc2lvbiBSZXZpc2lvbjogMTIwODYyCg== 148874 2012-06-21 13:33:26 -0700 2012-06-21 13:38:18 -0700 updated test bug-88762-20120621133326.patch text/plain 1145 abarth U3VidmVyc2lvbiBSZXZpc2lvbjogMTIwODYyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0 L3JlcGFpbnQvZm9vLmh0bWwgYi9MYXlvdXRUZXN0cy9mYXN0L3JlcGFpbnQvZm9vLmh0bWwKbmV3 IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw MDAwMDAwMC4uNmMxNWE2ZDA3MTAzN2Y2MzQ3OGFiNzNhZjg1N2NmODVjZTVjNmIwMgotLS0gL2Rl di9udWxsCisrKyBiL0xheW91dFRlc3RzL2Zhc3QvcmVwYWludC9mb28uaHRtbApAQCAtMCwwICsx LDMwIEBACis8aHRtbD4KKzxoZWFkPgorPHNjcmlwdD4KK2lmICghd2luZG93LmxheW91dFRlc3RD b250cm9sbGVyKQorICAgIGFsZXJ0KCJUaGlzIHRlc3QgcmVxdWlyZXMgdGhlIExheW91dFRlc3RD b250cm9sbGVyLiIpOworCitsYXlvdXRUZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KHRydWUpOwor bGF5b3V0VGVzdENvbnRyb2xsZXIud2FpdFVudGlsRG9uZSgpOworaW50ZXJuYWxzLnNldHRpbmdz LnNldFBhZ2VTY2FsZUZhY3RvcigxMCwgMCwgMCk7CisKK3dpbmRvdy5hZGRFdmVudExpc3RlbmVy KCJsb2FkIiwgZnVuY3Rpb24oKSB7CisgICAgZG9jdW1lbnQuYm9keS5vZmZzZXRUb3A7IC8vIEZv cmNlIGxheW91dC4KKyAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5kaXNwbGF5KCk7CisKKyAgICB3 aW5kb3cuc2Nyb2xsVG8oMCwgMTApOworfSwgZmFsc2UpOworCit3aW5kb3cuYWRkRXZlbnRMaXN0 ZW5lcigic2Nyb2xsIiwgZnVuY3Rpb24oKSB7CisgICAgc2V0VGltZW91dChmdW5jdGlvbigpIHsK KyAgICAgICAgZG9jdW1lbnQuZ2V0RWxlbWVudHNCeVRhZ05hbWUoImRpdiIpWzBdLnN0eWxlLmJh Y2tncm91bmRDb2xvciA9ICJncmVlbiI7CisgICAgICAgIHNldFRpbWVvdXQoZnVuY3Rpb24oKSB7 IGxheW91dFRlc3RDb250cm9sbGVyLm5vdGlmeURvbmUoKSB9LCAwKTsKKyAgICB9LCAwKTsKK30p OworPC9zY3JpcHQ+Cis8L2hlYWQ+CisKKzxib2R5IHN0eWxlPSJoZWlnaHQ6MjAwMHB4OyI+Cisg ICAgPGRpdiBzdHlsZT0icG9zaXRpb246Zml4ZWQ7bGVmdDoyMHB4O3RvcDoyMHB4O2JhY2tncm91 bmQtY29sb3I6cmVkO2hlaWdodDoxMHB4O3dpZHRoOjEwcHgiPjwvZGl2PgorPC9ib2R5PgorPC9o dG1sPgo= 148876 2012-06-21 13:37:32 -0700 2012-06-22 08:03:42 -0700 updated test bug-88762-20120621133732.patch text/plain 1042 abarth U3VidmVyc2lvbiBSZXZpc2lvbjogMTIwODYyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0 L3JlcGFpbnQvZm9vLmh0bWwgYi9MYXlvdXRUZXN0cy9mYXN0L3JlcGFpbnQvZm9vLmh0bWwKbmV3 IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw MDAwMDAwMC4uYmI5NDcyMjVhNzI0NTIwYWIzNmU1ZGFlOGM0MTk3YzdlMjI4MTFlNwotLS0gL2Rl di9udWxsCisrKyBiL0xheW91dFRlc3RzL2Zhc3QvcmVwYWludC9mb28uaHRtbApAQCAtMCwwICsx LDI3IEBACis8aHRtbD4KKzxoZWFkPgorPHNjcmlwdD4KK2lmICghd2luZG93LmxheW91dFRlc3RD b250cm9sbGVyKQorICAgIGFsZXJ0KCJUaGlzIHRlc3QgcmVxdWlyZXMgdGhlIExheW91dFRlc3RD b250cm9sbGVyLiIpOworCitsYXlvdXRUZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KHRydWUpOwor bGF5b3V0VGVzdENvbnRyb2xsZXIud2FpdFVudGlsRG9uZSgpOworaW50ZXJuYWxzLnNldHRpbmdz LnNldFBhZ2VTY2FsZUZhY3RvcigxMCwgMCwgMCk7CisKK3dpbmRvdy5hZGRFdmVudExpc3RlbmVy KCJsb2FkIiwgZnVuY3Rpb24oKSB7CisgICAgZG9jdW1lbnQuYm9keS5vZmZzZXRUb3A7IC8vIEZv cmNlIGxheW91dC4KKyAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5kaXNwbGF5KCk7CisKKyAgICB3 aW5kb3cuc2Nyb2xsVG8oMCwgMTUpOworfSwgZmFsc2UpOworCit3aW5kb3cuYWRkRXZlbnRMaXN0 ZW5lcigic2Nyb2xsIiwgZnVuY3Rpb24oKSB7CisgICAgc2V0VGltZW91dChmdW5jdGlvbigpIHsg bGF5b3V0VGVzdENvbnRyb2xsZXIubm90aWZ5RG9uZSgpIH0sIDApOworfSk7Cis8L3NjcmlwdD4K KzwvaGVhZD4KKworPGJvZHkgc3R5bGU9ImhlaWdodDoyMDAwcHg7IGJhY2tncm91bmQtY29sb3I6 Ymx1ZSI+CisgICAgPGRpdiBzdHlsZT0icG9zaXRpb246Zml4ZWQ7bGVmdDoyMHB4O3RvcDoyMHB4 O2JhY2tncm91bmQtY29sb3I6Z3JlZW47aGVpZ2h0OjEwcHg7d2lkdGg6MTBweCI+PC9kaXY+Cis8 L2JvZHk+Cis8L2h0bWw+Cg== 149031 2012-06-22 08:03:42 -0700 2012-06-24 02:21:44 -0700 Working test 0001-Working-test.patch text/plain 1573 skyostil RnJvbSA5YjllY2FkZTI3ZTdmZGUwOTdjNDVkMmIwNWJjZmU4MjczZDBkYzNhIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBTYW1pIEt5b3N0aWxhIDxza3lvc3RpbEBjaHJvbWl1bS5vcmc+ CkRhdGU6IEZyaSwgMjIgSnVuIDIwMTIgMTU6NTk6NTggKzAxMDAKU3ViamVjdDogW1BBVENIXSBX b3JraW5nIHRlc3QKCi0tLQogTGF5b3V0VGVzdHMvZmFzdC9yZXBhaW50L2Zvby5odG1sIHwgICA1 MiArKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrCiAxIGZpbGVzIGNoYW5nZWQs IDUyIGluc2VydGlvbnMoKyksIDAgZGVsZXRpb25zKC0pCiBjcmVhdGUgbW9kZSAxMDA2NDQgTGF5 b3V0VGVzdHMvZmFzdC9yZXBhaW50L2Zvby5odG1sCgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMv ZmFzdC9yZXBhaW50L2Zvby5odG1sIGIvTGF5b3V0VGVzdHMvZmFzdC9yZXBhaW50L2Zvby5odG1s Cm5ldyBmaWxlIG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAuLmUzNDRmMGMKLS0tIC9kZXYvbnVs bAorKysgYi9MYXlvdXRUZXN0cy9mYXN0L3JlcGFpbnQvZm9vLmh0bWwKQEAgLTAsMCArMSw1MiBA QAorPCFET0NUWVBFIGh0bWw+Cis8aHRtbD4KKzxoZWFkPgorCis8c3R5bGU+CisgIC50ZXN0IHsK KyAgICBwb3NpdGlvbjogZml4ZWQ7CisgICAgbGVmdDogMjBweDsKKyAgICB0b3A6IDIwcHg7Cisg ICAgd2lkdGg6IDQwcHg7CisgICAgaGVpZ2h0OiA0MHB4OworICAgIGJhY2tncm91bmQtY29sb3I6 IGdyZWVuOworICB9CisKKyAgLnRyaWdnZXItY29tcG9zaXRpb24geworICAgIC13ZWJraXQtdHJh bnNmb3JtOiB0cmFuc2xhdGVaKDApOworICB9CisKKyAgYm9keSB7CisgICAgaGVpZ2h0OiAyMDAw cHg7CisgIH0KKzwvc3R5bGU+CisKKzxzY3JpcHQ+CisgIGlmICghd2luZG93LmxheW91dFRlc3RD b250cm9sbGVyKQorICAgIGFsZXJ0KCJUaGlzIHRlc3QgcmVxdWlyZXMgdGhlIExheW91dFRlc3RD b250cm9sbGVyLiIpOworCisgIGxheW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQodHJ1ZSk7 CisgIGxheW91dFRlc3RDb250cm9sbGVyLndhaXRVbnRpbERvbmUoKTsKKyAgaW50ZXJuYWxzLnNl dHRpbmdzLnNldFBhZ2VTY2FsZUZhY3Rvcig0LCAwLCAwKTsKKworICB3aW5kb3cuYWRkRXZlbnRM aXN0ZW5lcigibG9hZCIsIGZ1bmN0aW9uKCkgeworICAgIGRvY3VtZW50LmJvZHkub2Zmc2V0VG9w OyAvLyBGb3JjZSBsYXlvdXQuCisgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIuZGlzcGxheSgpOwor CisgICAgd2luZG93LnNjcm9sbFRvKDAsIDI1KTsKKyAgfSwgZmFsc2UpOworCisgIHdpbmRvdy5h ZGRFdmVudExpc3RlbmVyKCJzY3JvbGwiLCBmdW5jdGlvbigpIHsKKyAgICBzZXRUaW1lb3V0KGZ1 bmN0aW9uKCkgeworICAgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIubm90aWZ5RG9uZSgpCisgICAg fSwgMCk7CisgIH0pOworPC9zY3JpcHQ+Cis8L2hlYWQ+CisKKzxib2R5PgorICA8ZGl2IGNsYXNz PSJ0ZXN0Ij48L2Rpdj4KKyAgPGRpdiBjbGFzcz0idHJpZ2dlci1jb21wb3NpdGlvbiI+PC9kaXY+ Cis8L2JvZHk+CisKKzwvaHRtbD4KLS0gCjEuNy43LjMKCg== 149190 2012-06-24 02:21:49 -0700 2012-06-24 02:21:49 -0700 test with expected results bug-88762-20120624022150.patch text/plain 3297 abarth U3VidmVyc2lvbiBSZXZpc2lvbjogMTIxMTEyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0 L3JlcGFpbnQvc2Nyb2xsLWZpeGVkLXBvc2l0aW9uLXdpdGgtcGFnZS1zY2FsZS1leHBlY3RlZC5w bmcgYi9MYXlvdXRUZXN0cy9mYXN0L3JlcGFpbnQvc2Nyb2xsLWZpeGVkLXBvc2l0aW9uLXdpdGgt cGFnZS1zY2FsZS1leHBlY3RlZC5wbmcKbmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAw MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMC4uMGMxYWMxNmE1MjRmMDJiMjVjNzA4 OTkwMzA5MDMzMjllMmVkMzZiMQpHSVQgYmluYXJ5IHBhdGNoCmxpdGVyYWwgMzg0Ngp6Y21lQVNA Tj8ob2xIeWB1VkJxIWlhMHl+eVV7K3ZZVjJhPmkxQiVRbFlicFJ6bmtCOWdDQ000NyQ9U3VGeGVW ckoKem1Qc2lfV3tLdCYkJSU9e0RXKD50aERuSlFzZ354K2k2LVYyTmhUSlJSeSsoMGReMD48OTc4 SlJ5dUl0X0ZPbl9SCnpATm9VRSolfT9ZOGFEX2B8TUNCU1UoVkRTMHp5SnBqdyQkX1BrTHJtd0Fi N0BxMns7TmBPUG1LTFJIaDlGWD1EVwp6R0ZeTjleO2UrZkl+RzVGe0o1fjd6I3doJiVeQXh3YjtQ PDJrNGIjflM+dG9yPGUwPkxKRHtTN3dQREZ5cWs1a2YKelMqQShQbVNTTTJDfUg3WFYzOzVYXm9O NjYxSkRSYk1rV1NeNnMmP2B2Km15YWV0ZUNNaXxnWjt4OWQ0Uy1AZjFRCnoqcDBoKVchdX1ebGRH JExVI20wJm5zY0M4WFpxPkZuaT9CREotdVVEVGN1V0MmNURWTT0+ZldJVn5rJEBUeHx5aQp6aEtC MGRYVlZAKGQ2RShuQU9GQUwmOTdmb2QrUHQhYENSXlR8TVRnR0s8V0dFPCtzUCllTDkxSWslMilq V295KmsKenBKMWpENkFKXncyQFV9VTFfZm8xRExvK2hLYDFEZlZtSlJqI0wwM2BaZjtJNXdeSVhx ZGlUenFsQXhmPCZWSnBWCno5MzM2fl9gYD5TSFA+SX4/LXZGS1RUWHN7fEtpMXFLUiFDelo4LW5A QGZ0Z017ckcoez8oWGklPkZMan0/ZDt8fgp6M3lpOGladlEkYVdYP0ohQCVaLWhAMztGNmIzaTZm S25zWStpZUsxJm8hQE1fTDlvN1I4OF5jTmBTV1o7QmpnNVEKeldSNXklQE1rIUIjTD0wZyVnbGRY VTM9fjgmJW5VMnBIRih3bXFQUmlsKStqOGtOU055KH1GNi0oZXliKW5vT2VeCnpifk1BMW9uZHxR QDkmR3Z1TDhEP0E2KTxWP0NrOTdffXZ5MzdDLUYhZkwpbW4tXkRpSGpLQU0lPktrbXNtMjw4dAp6 U0NFNHRBbVdkdTM5eVV3PElCcy1LWW8wQzREMXcxT0wxRVBKNCFiN3w3emE2ZClHSTZfSDVzTldg Kztadi1SfSMKel95MndVe25AaiRKPU5keT1GRlgmVGNtY1FXQUVPKkFBZnkpe2pvZ1NrJThnZytu MU1AZjhZMSp3KXQ5TSF+RXNUCnpsfExSX3c2ZUNBVWNQKkpvOz9mMGs0fn1BS21YazNmQnlYVzxL T2NpWm5VKHdtdFA7Wl90cUIpVV5gLU1eIWJMRQp6KngxfHhrc0V0ZnVoc14oRGpNZCR4T1ZvUHwy JihVUzdiaSlLZ2I3cT5pTXAlajVseEptUjQ2K216SDVTJWcpYVMKendkV3RHMUJFUnVFRl55YXs9 S2leO3xwTFhAN3dxOT9jJjh1NDFaIVNVdGI/elUlQD40ZCY3cXJIVUgmK2pnXjFmCnprMHVxcSZ0 NEFmUX5ZX1Z5JiVgSSFlPFlzRVc/OSRhLVooSklXJiEjZm5rTnR3OWArUXIzMEhjYiZBJjMpPDdT ewpZMk9EbClTVUxDcUhsUEBUcj5tZEtJO1ZzdDA4OGwoc1E+QH4KCmxpdGVyYWwgMApIY21WP2Qw MDAwMQoKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3QvcmVwYWludC9zY3JvbGwtZml4ZWQt cG9zaXRpb24td2l0aC1wYWdlLXNjYWxlLWV4cGVjdGVkLnR4dCBiL0xheW91dFRlc3RzL2Zhc3Qv cmVwYWludC9zY3JvbGwtZml4ZWQtcG9zaXRpb24td2l0aC1wYWdlLXNjYWxlLWV4cGVjdGVkLnR4 dApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw MDAwMDAwMDAwMDAwLi44YjEzNzg5MTc5MWZlOTY5MjdhZDc4ZTY0YjBhYWQ3YmRlZDA4YmRjCi0t LSAvZGV2L251bGwKKysrIGIvTGF5b3V0VGVzdHMvZmFzdC9yZXBhaW50L3Njcm9sbC1maXhlZC1w b3NpdGlvbi13aXRoLXBhZ2Utc2NhbGUtZXhwZWN0ZWQudHh0CkBAIC0wLDAgKzEgQEAKKwpkaWZm IC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFzdC9yZXBhaW50L3Njcm9sbC1maXhlZC1wb3NpdGlvbi13 aXRoLXBhZ2Utc2NhbGUuaHRtbCBiL0xheW91dFRlc3RzL2Zhc3QvcmVwYWludC9zY3JvbGwtZml4 ZWQtcG9zaXRpb24td2l0aC1wYWdlLXNjYWxlLmh0bWwKbmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5k ZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMC4uOTZiNDRmZmJiOWEw YWUyNGJlOTVlMDg3NmYyNDFkOGUwYmYyZGIyMwotLS0gL2Rldi9udWxsCisrKyBiL0xheW91dFRl c3RzL2Zhc3QvcmVwYWludC9zY3JvbGwtZml4ZWQtcG9zaXRpb24td2l0aC1wYWdlLXNjYWxlLmh0 bWwKQEAgLTAsMCArMSw1MSBAQAorPCFET0NUWVBFIGh0bWw+Cis8aHRtbD4KKzxoZWFkPgorCis8 c3R5bGU+CisgIC50ZXN0IHsKKyAgICBwb3NpdGlvbjogZml4ZWQ7CisgICAgbGVmdDogMjBweDsK KyAgICB0b3A6IDIwcHg7CisgICAgd2lkdGg6IDQwcHg7CisgICAgaGVpZ2h0OiA0MHB4OworICAg IGJhY2tncm91bmQtY29sb3I6IGJsdWU7CisgIH0KKworICAudHJpZ2dlci1jb21wb3NpdGlvbiB7 CisgICAgLXdlYmtpdC10cmFuc2Zvcm06IHRyYW5zbGF0ZVooMCk7CisgIH0KKworICBib2R5IHsK KyAgICBoZWlnaHQ6IDIwMDBweDsKKyAgfQorPC9zdHlsZT4KKworPHNjcmlwdD4KKyAgaWYgKCF3 aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCisgICAgYWxlcnQoIlRoaXMgdGVzdCByZXF1aXJl cyB0aGUgTGF5b3V0VGVzdENvbnRyb2xsZXIuIik7CisKKyAgbGF5b3V0VGVzdENvbnRyb2xsZXIu ZHVtcEFzVGV4dCh0cnVlKTsKKyAgbGF5b3V0VGVzdENvbnRyb2xsZXIud2FpdFVudGlsRG9uZSgp OworICBpbnRlcm5hbHMuc2V0dGluZ3Muc2V0UGFnZVNjYWxlRmFjdG9yKDQsIDAsIDApOworCisg IHdpbmRvdy5hZGRFdmVudExpc3RlbmVyKCJsb2FkIiwgZnVuY3Rpb24oKSB7CisgICAgbGF5b3V0 VGVzdENvbnRyb2xsZXIuZGlzcGxheSgpOworCisgICAgd2luZG93LnNjcm9sbFRvKDAsIDI1KTsK KyAgfSwgZmFsc2UpOworCisgIHdpbmRvdy5hZGRFdmVudExpc3RlbmVyKCJzY3JvbGwiLCBmdW5j dGlvbigpIHsKKyAgICBzZXRUaW1lb3V0KGZ1bmN0aW9uKCkgeworICAgICAgbGF5b3V0VGVzdENv bnRyb2xsZXIubm90aWZ5RG9uZSgpCisgICAgfSwgMCk7CisgIH0pOworPC9zY3JpcHQ+Cis8L2hl YWQ+CisKKzxib2R5PgorICA8ZGl2IGNsYXNzPSJ0ZXN0Ij48L2Rpdj4KKyAgPGRpdiBjbGFzcz0i dHJpZ2dlci1jb21wb3NpdGlvbiI+PC9kaXY+Cis8L2JvZHk+CisKKzwvaHRtbD4K