131828 2014-04-17 16:15:45 -0700 [iOS] REGRESSION (r166975): WKPDFView is broken 2014-04-17 16:30:20 -0700 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 thorton thorton aestes benjamin simon.fraser oldest_to_newest 1001742 0 thorton 2014-04-17 16:15:45 -0700 We're getting layer tree updates while the custom content view is installed, this is causing trouble. The assert should be an early return, because we never want to apply changes to the scroll view while it's being managed by the custom content view, and we'll get any changes again in the next commit after removing the custom content view (and painting the new page) anyway. <rdar://problem/16631282> 1001744 1 229591 thorton 2014-04-17 16:17:14 -0700 Created attachment 229591 patch 1001750 2 thorton 2014-04-17 16:30:20 -0700 http://trac.webkit.org/changeset/167465 229591 2014-04-17 16:17:14 -0700 2014-04-17 16:18:05 -0700 patch pdf.diff text/plain 1540 thorton ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCA0MDMwY2E0Li5jNzM2N2JjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcg QEAKKzIwMTQtMDQtMTcgIFRpbSBIb3J0b24gIDx0aW1vdGh5X2hvcnRvbkBhcHBsZS5jb20+CisK KyAgICAgICAgW2lPU10gUkVHUkVTU0lPTiAocjE2Njk3NSk6IFdLUERGVmlldyBpcyBicm9rZW4K KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEzMTgyOAor CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogVUlQcm9j ZXNzL0FQSS9Db2NvYS9XS1dlYlZpZXcubW06CisgICAgICAgICgtW1dLV2ViVmlldyBfZGlkQ29t bWl0TGF5ZXJUcmVlOldlYktpdDo6XSk6CisgICAgICAgIEJhaWwgZnJvbSB1cGRhdGluZyB0aGUg c2Nyb2xsIHZpZXcgcGFyYW1ldGVycyBmcm9tIHRoZSBsYXllciB0cmVlIHVwZGF0ZQorICAgICAg ICB3aGlsZSBpdCdzIGJlaW5nIG1hbmFnZWQgYnkgYSBjdXN0b20gY29udGVudCB2aWV3LgorICAg ICAgICBXZSdsbCBnZXQgYW55IGNoYW5nZXMgYWdhaW4gaW4gdGhlIG5leHQgY29tbWl0IGFmdGVy IHJlbW92aW5nIHRoZSBjdXN0b20KKyAgICAgICAgY29udGVudCB2aWV3IChhbmQgcGFpbnRpbmcg dGhlIG5ldyBwYWdlKSBhbnl3YXksIHNvIHRoaXMgc2hvdWxkIGJlIE9LLgorCiAyMDE0LTA0LTE3 ICBEYXJpbiBBZGxlciAgPGRhcmluQGFwcGxlLmNvbT4KIAogICAgICAgICBSZW1vdmUgdXNlIG9m IGRlcHJlY2F0ZWREZWxldGVBbGxWYWx1ZXMgaW4gTlBSZW1vdGVPYmplY3RNYXA6OnBsdWdpbkRl c3Ryb3llZApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9Db2NvYS9X S1dlYlZpZXcubW0gYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL0NvY29hL1dLV2ViVmll dy5tbQppbmRleCBhMDM3OGM5Li44NzczZjU1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9V SVByb2Nlc3MvQVBJL0NvY29hL1dLV2ViVmlldy5tbQorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVBy b2Nlc3MvQVBJL0NvY29hL1dLV2ViVmlldy5tbQpAQCAtNDUzLDcgKzQ1Myw4IEBAIHN0YXRpYyBD R0Zsb2F0IGNvbnRlbnRab29tU2NhbGUoV0tXZWJWaWV3KiB3ZWJWaWV3KQogCiAtICh2b2lkKV9k aWRDb21taXRMYXllclRyZWU6KGNvbnN0IFdlYktpdDo6UmVtb3RlTGF5ZXJUcmVlVHJhbnNhY3Rp b24mKWxheWVyVHJlZVRyYW5zYWN0aW9uCiB7Ci0gICAgQVNTRVJUKCFfY3VzdG9tQ29udGVudFZp ZXcpOworICAgIGlmIChfY3VzdG9tQ29udGVudFZpZXcpCisgICAgICAgIHJldHVybjsKIAogICAg IGlmIChfaXNBbmltYXRpbmdSZXNpemUpIHsKICAgICAgICAgW19jb250ZW50VmlldyBsYXllcl0u c3VibGF5ZXJUcmFuc2Zvcm0gPSBfcmVzaXplQW5pbWF0aW9uVHJhbnNmb3JtQWRqdXN0bWVudHM7 Cg==