167452 2017-01-26 01:09:16 -0800 [Threaded Compositor] Update also the contents size when creating the threaded compositor 2017-01-26 01:21:16 -0800 1 1 1 Unclassified WebKit WebKit2 WebKit Local Build Unspecified Unspecified RESOLVED FIXED Gtk P2 Normal --- 1 cgarcia webkit-unassigned bugs-noreply zan oldest_to_newest 1270159 0 cgarcia 2017-01-26 01:09:16 -0800 In r210954 we ensured that the threaded compositor and the viewport controller were created with the initial web page size. If we don't update the contents size, the visible rectangle will be empty and tiles won't be created until contentsSizeDidChange is called. It's even possible, when entering AC mode on demand, that we create the layer tree host after the contents size changed, in which case nothing is rendered unless we resize the window. 1270160 1 299796 cgarcia 2017-01-26 01:11:28 -0800 Created attachment 299796 Patch 1270162 2 cgarcia 2017-01-26 01:21:16 -0800 Committed r211205: <http://trac.webkit.org/changeset/211205> 299796 2017-01-26 01:11:28 -0800 2017-01-26 01:18:11 -0800 Patch wk2-initial-visible-rect.diff text/plain 2304 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCA0YmVjY2ZkZmRlYi4uYzc1YjIwOGU3Y2EgMTAwNjQ0Ci0tLSBhL1NvdXJj ZS9XZWJLaXQyL0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKQEAgLTEs MyArMSwxOCBAQAorMjAxNy0wMS0yNiAgQ2FybG9zIEdhcmNpYSBDYW1wb3MgIDxjZ2FyY2lhQGln YWxpYS5jb20+CisKKyAgICAgICAgW1RocmVhZGVkIENvbXBvc2l0b3JdIFVwZGF0ZSBhbHNvIHRo ZSBjb250ZW50cyBzaXplIHdoZW4gY3JlYXRpbmcgdGhlIHRocmVhZGVkIGNvbXBvc2l0b3IKKyAg ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE2NzQ1MgorCisg ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEluIHIyMTA5NTQg d2UgZW5zdXJlZCB0aGF0IHRoZSB0aHJlYWRlZCBjb21wb3NpdG9yIGFuZCB0aGUgdmlld3BvcnQg Y29udHJvbGxlciB3ZXJlIGNyZWF0ZWQgd2l0aCB0aGUgaW5pdGlhbCB3ZWIKKyAgICAgICAgcGFn ZSBzaXplLiBJZiB3ZSBkb24ndCB1cGRhdGUgdGhlIGNvbnRlbnRzIHNpemUsIHRoZSB2aXNpYmxl IHJlY3RhbmdsZSB3aWxsIGJlIGVtcHR5IGFuZCB0aWxlcyB3b24ndCBiZSBjcmVhdGVkCisgICAg ICAgIHVudGlsIGNvbnRlbnRzU2l6ZURpZENoYW5nZSBpcyBjYWxsZWQuIEl0J3MgZXZlbiBwb3Nz aWJsZSwgd2hlbiBlbnRlcmluZyBBQyBtb2RlIG9uIGRlbWFuZCwgdGhhdCB3ZSBjcmVhdGUgdGhl CisgICAgICAgIGxheWVyIHRyZWUgaG9zdCBhZnRlciB0aGUgY29udGVudHMgc2l6ZSBjaGFuZ2Vk LCBpbiB3aGljaCBjYXNlIG5vdGhpbmcgaXMgcmVuZGVyZWQgdW5sZXNzIHdlIHJlc2l6ZSB0aGUg d2luZG93LgorCisgICAgICAgICogV2ViUHJvY2Vzcy9XZWJQYWdlL0Nvb3JkaW5hdGVkR3JhcGhp Y3MvVGhyZWFkZWRDb29yZGluYXRlZExheWVyVHJlZUhvc3QuY3BwOgorICAgICAgICAoV2ViS2l0 OjpUaHJlYWRlZENvb3JkaW5hdGVkTGF5ZXJUcmVlSG9zdDo6VGhyZWFkZWRDb29yZGluYXRlZExh eWVyVHJlZUhvc3QpOgorCiAyMDE3LTAxLTI1ICBDYXJsb3MgR2FyY2lhIENhbXBvcyAgPGNnYXJj aWFAaWdhbGlhLmNvbT4KIAogICAgICAgICBbR1RLXSBJY29uIERhdGFiYXNlIHNob3VsZCBiZSBp biBwcml2YXRlIGJyb3dzaW5nIG1vZGUgZm9yIGVwaGVtZXJhbCB3ZWIgdmlld3MKZGlmZiAtLWdp dCBhL1NvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9Db29yZGluYXRlZEdyYXBoaWNz L1RocmVhZGVkQ29vcmRpbmF0ZWRMYXllclRyZWVIb3N0LmNwcCBiL1NvdXJjZS9XZWJLaXQyL1dl YlByb2Nlc3MvV2ViUGFnZS9Db29yZGluYXRlZEdyYXBoaWNzL1RocmVhZGVkQ29vcmRpbmF0ZWRM YXllclRyZWVIb3N0LmNwcAppbmRleCA1OGRhOGViNmUzYi4uYzU4NGM1ZGFjYTAgMTAwNjQ0Ci0t LSBhL1NvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9Db29yZGluYXRlZEdyYXBoaWNz L1RocmVhZGVkQ29vcmRpbmF0ZWRMYXllclRyZWVIb3N0LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0 Mi9XZWJQcm9jZXNzL1dlYlBhZ2UvQ29vcmRpbmF0ZWRHcmFwaGljcy9UaHJlYWRlZENvb3JkaW5h dGVkTGF5ZXJUcmVlSG9zdC5jcHAKQEAgLTU1LDYgKzU1LDEyIEBAIFRocmVhZGVkQ29vcmRpbmF0 ZWRMYXllclRyZWVIb3N0OjpUaHJlYWRlZENvb3JkaW5hdGVkTGF5ZXJUcmVlSG9zdChXZWJQYWdl JiB3ZWJQCiAgICAgLCBtX3N1cmZhY2UoQWNjZWxlcmF0ZWRTdXJmYWNlOjpjcmVhdGUod2ViUGFn ZSkpCiAgICAgLCBtX3ZpZXdwb3J0Q29udHJvbGxlcih3ZWJQYWdlLnNpemUoKSkKIHsKKyAgICBp ZiAoRnJhbWVWaWV3KiBmcmFtZVZpZXcgPSBtX3dlYlBhZ2UubWFpbkZyYW1lVmlldygpKSB7Cisg ICAgICAgIGF1dG8gY29udGVudHNTaXplID0gZnJhbWVWaWV3LT5jb250ZW50c1NpemUoKTsKKyAg ICAgICAgaWYgKCFjb250ZW50c1NpemUuaXNFbXB0eSgpKQorICAgICAgICAgICAgbV92aWV3cG9y dENvbnRyb2xsZXIuZGlkQ2hhbmdlQ29udGVudHNTaXplKGNvbnRlbnRzU2l6ZSk7CisgICAgfQor CiAgICAgSW50U2l6ZSBzY2FsZWRTaXplKG1fd2ViUGFnZS5zaXplKCkpOwogICAgIHNjYWxlZFNp emUuc2NhbGUobV93ZWJQYWdlLmRldmljZVNjYWxlRmFjdG9yKCkpOwogICAgIGZsb2F0IHNjYWxl RmFjdG9yID0gbV93ZWJQYWdlLmRldmljZVNjYWxlRmFjdG9yKCkgKiBtX3ZpZXdwb3J0Q29udHJv bGxlci5wYWdlU2NhbGVGYWN0b3IoKTsK