78249 2012-02-09 10:26:30 -0800 [Chromium] TiledLayerChromium::protectVisibleTileTextures() should only protect the visible textures 2012-02-09 21:03:13 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 wangxianzhu wangxianzhu cc-bugs enne jamesr vangelis webkit.review.bot oldest_to_newest 553281 0 wangxianzhu 2012-02-09 10:26:30 -0800 For now TiledLayerChromium::protectVisibleTileTextures() protects the whole layer: protectTileTextures(IntRect(IntPoint::zero(), contentBounds())); Should this be changed to protectTileTextures(visibleLayerRect())? 553322 1 enne 2012-02-09 10:57:58 -0800 I think you're right. I'm not sure why it wasn't this way to begin with. 553391 2 126343 wangxianzhu 2012-02-09 12:02:31 -0800 Created attachment 126343 patch 553786 3 126343 webkit.review.bot 2012-02-09 21:03:10 -0800 Comment on attachment 126343 patch Clearing flags on attachment: 126343 Committed r107352: <http://trac.webkit.org/changeset/107352> 553787 4 webkit.review.bot 2012-02-09 21:03:13 -0800 All reviewed patches have been landed. Closing bug. 126343 2012-02-09 12:02:31 -0800 2012-02-09 21:03:09 -0800 patch 78249 text/plain 1327 wangxianzhu SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEwNzI2OSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEzIEBACisyMDEyLTAyLTA5ICBYaWFuemh1 IFdhbmcgIDx3YW5neGlhbnpodUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgW0Nocm9taXVtXSBU aWxlZExheWVyQ2hyb21pdW06OnByb3RlY3RWaXNpYmxlVGlsZVRleHR1cmVzKCkgc2hvdWxkIG9u bHkgcHJvdGVjdCB0aGUgdmlzaWJsZSB0ZXh0dXJlcworICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzgyNDkKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL2Nocm9taXVtL1RpbGVk TGF5ZXJDaHJvbWl1bS5jcHA6CisgICAgICAgIChXZWJDb3JlOjpUaWxlZExheWVyQ2hyb21pdW06 OnByb3RlY3RWaXNpYmxlVGlsZVRleHR1cmVzKToKKwogMjAxMi0wMi0wOSAgU2hlcmlmZiBCb3Qg IDx3ZWJraXQucmV2aWV3LmJvdEBnbWFpbC5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZCwgcm9s bGluZyBvdXQgcjEwNzAzNS4KSW5kZXg6IFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNz L2Nocm9taXVtL1RpbGVkTGF5ZXJDaHJvbWl1bS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dl YkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvY2hyb21pdW0vVGlsZWRMYXllckNocm9taXVtLmNwcAko cmV2aXNpb24gMTA3MjYzKQorKysgU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvY2hy b21pdW0vVGlsZWRMYXllckNocm9taXVtLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMzM3LDcgKzMz Nyw3IEBAIHZvaWQgVGlsZWRMYXllckNocm9taXVtOjppbnZhbGlkYXRlUmVjdCgKIAogdm9pZCBU aWxlZExheWVyQ2hyb21pdW06OnByb3RlY3RWaXNpYmxlVGlsZVRleHR1cmVzKCkKIHsKLSAgICBw cm90ZWN0VGlsZVRleHR1cmVzKEludFJlY3QoSW50UG9pbnQ6Onplcm8oKSwgY29udGVudEJvdW5k cygpKSk7CisgICAgcHJvdGVjdFRpbGVUZXh0dXJlcyh2aXNpYmxlTGF5ZXJSZWN0KCkpOwogfQog CiB2b2lkIFRpbGVkTGF5ZXJDaHJvbWl1bTo6cHJvdGVjdFRpbGVUZXh0dXJlcyhjb25zdCBJbnRS ZWN0JiBsYXllclJlY3QpCg==