88057 2012-06-01 00:43:46 -0700 Overflow scrolling doesn't need to create a stacking context is the overflow is hidden 2012-06-01 15:54:16 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 66687 1 abarth abarth cmarcelo jamesr jchaffraix macpherson menard satish simon.fraser skyostil tonikitoo webkit.review.bot oldest_to_newest 639053 0 abarth 2012-06-01 00:43:46 -0700 Overflow scrolling doesn't need to create a stacking context is the overflow is hidden 639054 1 145230 abarth 2012-06-01 00:44:14 -0700 Created attachment 145230 Patch 639055 2 abarth 2012-06-01 00:45:02 -0700 This patch is in the chromium-android branch, but I'm unsure whether it's correct. If it's correct, I'd appreciate any pointers towards how to create a test for this change. Thanks! 639058 3 webkit.review.bot 2012-06-01 00:47:25 -0700 Attachment 145230 did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style. 639450 4 skyostil 2012-06-01 09:36:54 -0700 I think the change makes sense, but we should check that it matches the iOS5 implementation. 639456 5 abarth 2012-06-01 09:42:53 -0700 (In reply to comment #4) > I think the change makes sense, but we should check that it matches the iOS5 implementation. Is there an easy way I can check that with my iPad? 639465 6 145230 jamesr 2012-06-01 09:51:10 -0700 Comment on attachment 145230 Patch http://opensource.apple.com/source/WebCore/WebCore-1298.39/css/CSSStyleSelector.cpp 639524 7 abarth 2012-06-01 10:56:44 -0700 Thanks James. 639681 8 145372 abarth 2012-06-01 13:41:39 -0700 Created attachment 145372 Patch for landing 639807 9 145372 webkit.review.bot 2012-06-01 15:54:11 -0700 Comment on attachment 145372 Patch for landing Clearing flags on attachment: 145372 Committed r119294: <http://trac.webkit.org/changeset/119294> 639808 10 webkit.review.bot 2012-06-01 15:54:16 -0700 All reviewed patches have been landed. Closing bug. 145230 2012-06-01 00:44:14 -0700 2012-06-01 13:41:34 -0700 Patch bug-88057-20120601004413.patch text/plain 1306 abarth SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDExOTE5NikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDEyLTA2LTAxICBBZGFtIEJh cnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CisKKyAgICAgICAgT3ZlcmZsb3cgc2Nyb2xsaW5nIGRv ZXNuJ3QgbmVlZCB0byBjcmVhdGUgYSBzdGFja2luZyBjb250ZXh0IGlzIHRoZSBvdmVyZmxvdyBp cyBoaWRkZW4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk PTg4MDU3CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg Tm8gbmV3IHRlc3RzLiAoT09QUyEpCisKKyAgICAgICAgKiBjc3MvU3R5bGVSZXNvbHZlci5jcHA6 CisgICAgICAgIChXZWJDb3JlOjpTdHlsZVJlc29sdmVyOjpjb2xsZWN0TWF0Y2hpbmdSdWxlc0Zv ckxpc3QpOgorCiAyMDEyLTA1LTMxICBEb21pbmljIENvb25leSAgPGRvbWluaWNjQGNocm9taXVt Lm9yZz4KIAogICAgICAgICBbVjhdIEV4cGFuZG8gcHJvcGVydGllcyBvbiBhdHRyaWJ1dGUgbm9k ZXMgZGlzYXBwZWFyCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9jc3MvU3R5bGVSZXNvbHZlci5jcHAK PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvY3NzL1N0eWxlUmVzb2x2ZXIuY3BwCShyZXZp c2lvbiAxMTkxODcpCisrKyBTb3VyY2UvV2ViQ29yZS9jc3MvU3R5bGVSZXNvbHZlci5jcHAJKHdv cmtpbmcgY29weSkKQEAgLTIwNzEsNyArMjA3MSw3IEBAIHZvaWQgU3R5bGVSZXNvbHZlcjo6YWRq dXN0UmVuZGVyU3R5bGUoUmUKICNlbmRpZgogI2lmIEVOQUJMRShPVkVSRkxPV19TQ1JPTExJTkcp CiAgICAgICAgIC8vIFRvdWNoIG92ZXJmbG93IHNjcm9sbGluZyBjcmVhdGVzIGEgc3RhY2tpbmcg Y29udGV4dC4KLSAgICAgICAgfHwgc3R5bGUtPnVzZVRvdWNoT3ZlcmZsb3dTY3JvbGxpbmcoKQor ICAgICAgICB8fCAoKHN0eWxlLT5vdmVyZmxvd1goKSAhPSBPSElEREVOIHx8IHN0eWxlLT5vdmVy Zmxvd1koKSAhPSBPSElEREVOKSAmJiBzdHlsZS0+dXNlVG91Y2hPdmVyZmxvd1Njcm9sbGluZygp KQogI2VuZGlmCiAgICAgICAgICkpCiAgICAgICAgIHN0eWxlLT5zZXRaSW5kZXgoMCk7Cg== 145372 2012-06-01 13:41:39 -0700 2012-06-01 15:54:11 -0700 Patch for landing bug-88057-20120601134138.patch text/plain 1334 abarth SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDExOTI3NikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDEyLTA2LTAxICBBZGFtIEJh cnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CisKKyAgICAgICAgT3ZlcmZsb3cgc2Nyb2xsaW5nIGRv ZXNuJ3QgbmVlZCB0byBjcmVhdGUgYSBzdGFja2luZyBjb250ZXh0IGlzIHRoZSBvdmVyZmxvdyBp cyBoaWRkZW4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk PTg4MDU3CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgSmFtZXMgUm9iaW5zb24uCisKKyAgICAgICAg VGhpcyBwYXRjaCBjYXVzZXMgdXMgdG8gbWF0Y2ggdGhlIGlPUyBpbXBsZW1lbnRhdGlvbiBvZiB0 aGlzIGZlYXR1cmUuCisKKyAgICAgICAgKiBjc3MvU3R5bGVSZXNvbHZlci5jcHA6CisgICAgICAg IChXZWJDb3JlOjpTdHlsZVJlc29sdmVyOjpjb2xsZWN0TWF0Y2hpbmdSdWxlc0Zvckxpc3QpOgor CiAyMDEyLTA2LTAxICBNaWhhaSBQYXJwYXJpdGEgIDxtaWhhaXBAY2hyb21pdW0ub3JnPgogCiAg ICAgICAgIEltcHJvdmUgc3luY2hyb25vdXMgWEhSIGRpc2FibGluZwpJbmRleDogU291cmNlL1dl YkNvcmUvY3NzL1N0eWxlUmVzb2x2ZXIuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3Jl L2Nzcy9TdHlsZVJlc29sdmVyLmNwcAkocmV2aXNpb24gMTE5Mjc2KQorKysgU291cmNlL1dlYkNv cmUvY3NzL1N0eWxlUmVzb2x2ZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yMDcxLDcgKzIwNzEs NyBAQCB2b2lkIFN0eWxlUmVzb2x2ZXI6OmFkanVzdFJlbmRlclN0eWxlKFJlCiAjZW5kaWYKICNp ZiBFTkFCTEUoT1ZFUkZMT1dfU0NST0xMSU5HKQogICAgICAgICAvLyBUb3VjaCBvdmVyZmxvdyBz Y3JvbGxpbmcgY3JlYXRlcyBhIHN0YWNraW5nIGNvbnRleHQuCi0gICAgICAgIHx8IHN0eWxlLT51 c2VUb3VjaE92ZXJmbG93U2Nyb2xsaW5nKCkKKyAgICAgICAgfHwgKChzdHlsZS0+b3ZlcmZsb3dY KCkgIT0gT0hJRERFTiB8fCBzdHlsZS0+b3ZlcmZsb3dZKCkgIT0gT0hJRERFTikgJiYgc3R5bGUt PnVzZVRvdWNoT3ZlcmZsb3dTY3JvbGxpbmcoKSkKICNlbmRpZgogICAgICAgICApKQogICAgICAg ICBzdHlsZS0+c2V0WkluZGV4KDApOwo=