130962 2014-03-31 03:14:11 -0700 [CoordinatedGraphics] Content scale factor should not be applied to draw contents when fixed layout is not used. 2014-04-04 07:29:50 -0700 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 enmi.lee enmi.lee a.badowski bunhere cmarcelo commit-queue luiz noam sergio zeno oldest_to_newest 995802 0 enmi.lee 2014-03-31 03:14:11 -0700 Contents is painted by applying current page scale factor in the coordinated graphics even though page scale factor is already applied to the contents when fixed layout is not used. That means page scale factor is applied twice when fixed layout is not used. We can see wrong scaling results in the efl port's MiniBrowser using control+"-/+" button. - Bottom of the screen is not rendered (just white) when scale factor is less than 1.0. - We can not scroll down to the bottom of the page when scale factor is greater than 1.0. If fixed layout is not used delegating scaling is also not used, so we should not apply page scale factor to draw contents in that case. 995811 1 228167 enmi.lee 2014-03-31 03:37:58 -0700 Created attachment 228167 Patch 995815 2 228167 gyuyoung.kim 2014-03-31 03:44:03 -0700 Comment on attachment 228167 Patch I'm not coordinated graphics expert though, as far as I know, Coordinated graphics is only used by EFL port. So, rs=me for EFL port development. 996185 3 228167 commit-queue 2014-04-01 01:34:38 -0700 Comment on attachment 228167 Patch Clearing flags on attachment: 228167 Committed r166572: <http://trac.webkit.org/changeset/166572> 996186 4 commit-queue 2014-04-01 01:34:42 -0700 All reviewed patches have been landed. Closing bug. 997561 5 a.badowski 2014-04-04 07:29:50 -0700 *** Bug 130838 has been marked as a duplicate of this bug. *** 228167 2014-03-31 03:37:58 -0700 2014-04-01 01:34:38 -0700 Patch bug-130962-20140331193739.patch text/plain 2505 enmi.lee U3VidmVyc2lvbiBSZXZpc2lvbjogMTY2NDgwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggYzQ4ZmRlNTdiODY2Yjkz MjRiZTEyMmU3NjI5M2U1ZmJiMTMxMzhjNi4uODZhOTFlZWU4YzRkNTgwZWRiNWUxZGFhY2M1ZjYw ZWJhNmJkN2VkOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIzIEBACisyMDE0LTAzLTMxICBFdW5t aSBMZWUgIDxldW5taTE1LmxlZUBzYW1zdW5nLmNvbT4KKworICAgICAgICBbQ29vcmRpbmF0ZWRH cmFwaGljc10gQ29udGVudCBzY2FsZSBmYWN0b3Igc2hvdWxkIG5vdCBiZSBhcHBsaWVkIHRvIGRy YXcgY29udGVudHMgd2hlbiBmaXhlZCBsYXlvdXQgaXMgbm90IHVzZWQuCisgICAgICAgIGh0dHBz Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMzA5NjIKKworICAgICAgICBSZXZp ZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBEZWxlZ2F0aW5nIHNjYWxpbmcgaXMg dHVybmVkIG9mZiB3aGVuIGZpeGVkIGxheW91dCBpcyBub3QgdXNlZCBpbiB0aGUKKyAgICAgICAg Y29vcmRpbmF0ZWQgZ3JhcGhpY3MsIHNvIHBhZ2Ugc2NhbGUgZmFjdG9yIGlzIGFwcGxpZWQgdG8g dGhlIGNvbnRlbnRzCisgICAgICAgIGRpcmVjdGx5LgorICAgICAgICBJbiB0aGF0IGNhc2UsIHdl IHNob3VsZCBub3QgYXBwbHkgcGFnZSBzY2FsZSBmYWN0b3IgdG8gY29vcmRpbmF0ZWQKKyAgICAg ICAgZ3JhcGhpY3MgdG8gZHJhdyBjb250ZW50cyBhbmQgY2FsY3VsYXRlIHZpc2libGUgY29udGVu dCByZWN0LgorCisgICAgICAgIEN1cnJlbnRseSwgcGFnZSBzY2FsZSBmYWN0b3IgaXMgYXBwbGll ZCB0d2ljZSB0byB0aGUgY29vcmRpbmF0ZWQKKyAgICAgICAgZ3JhcGhpY3MgYW5kIGNvbnRlbnRz LCBzbyByZXN1bHQgb2Ygc2NhbGluZyBpcyBhYm5vcm1hbC4KKworICAgICAgICAqIFVJUHJvY2Vz cy9Db29yZGluYXRlZEdyYXBoaWNzL1dlYlZpZXcuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJW aWV3Ojp0cmFuc2Zvcm1Ub1NjZW5lKToKKyAgICAgICAgKFdlYktpdDo6V2ViVmlldzo6dmlzaWJs ZUNvbnRlbnRzU2l6ZSk6CisKIDIwMTQtMDMtMzAgIEFuZHkgRXN0ZXMgIDxhZXN0ZXNAYXBwbGUu Y29tPgogCiAgICAgICAgIFtDb2NvYV0gQWRkIGFuIG9yaWdpbmF0aW5nV2ViVmlldyBwcm9wZXJ0 eSB0byBfV0tEb3dubG9hZApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0Nv b3JkaW5hdGVkR3JhcGhpY3MvV2ViVmlldy5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3Mv Q29vcmRpbmF0ZWRHcmFwaGljcy9XZWJWaWV3LmNwcAppbmRleCBhYjI3ZjQ5OGU3MGY0NjcyZGQ4 ODYyNGUzMGU5YWIwMmE2YjI1YWE3Li5lNjE3ZGYxMjkzY2EyYjBjZWZhOWM0OTY1MDRlNTg0NWE2 NzU5ODI2IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQ29vcmRpbmF0ZWRH cmFwaGljcy9XZWJWaWV3LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQ29vcmRp bmF0ZWRHcmFwaGljcy9XZWJWaWV3LmNwcApAQCAtMjM0LDcgKzIzNCw5IEBAIEFmZmluZVRyYW5z Zm9ybSBXZWJWaWV3Ojp0cmFuc2Zvcm1Gcm9tU2NlbmUoKSBjb25zdAogQWZmaW5lVHJhbnNmb3Jt IFdlYlZpZXc6OnRyYW5zZm9ybVRvU2NlbmUoKSBjb25zdAogewogICAgIEZsb2F0UG9pbnQgcG9z aXRpb24gPSAtbV9jb250ZW50UG9zaXRpb247Ci0gICAgZmxvYXQgZWZmZWN0aXZlU2NhbGUgPSBj b250ZW50U2NhbGVGYWN0b3IoKSAqIG1fcGFnZS0+ZGV2aWNlU2NhbGVGYWN0b3IoKTsKKyAgICBm bG9hdCBlZmZlY3RpdmVTY2FsZSA9IG1fcGFnZS0+ZGV2aWNlU2NhbGVGYWN0b3IoKTsKKyAgICBp ZiAobV9wYWdlLT51c2VGaXhlZExheW91dCgpKQorICAgICAgICBlZmZlY3RpdmVTY2FsZSAqPSBj b250ZW50U2NhbGVGYWN0b3IoKTsKICAgICBwb3NpdGlvbi5zY2FsZShlZmZlY3RpdmVTY2FsZSwg ZWZmZWN0aXZlU2NhbGUpOwogCiAgICAgVHJhbnNmb3JtYXRpb25NYXRyaXggdHJhbnNmb3JtID0g bV91c2VyVmlld3BvcnRUcmFuc2Zvcm07CkBAIC0yNzQsNyArMjc2LDggQEAgaW5saW5lIFdlYkNv cmU6OkZsb2F0U2l6ZSBXZWJWaWV3OjpkaXBTaXplKCkgY29uc3QKIFdlYkNvcmU6OkZsb2F0U2l6 ZSBXZWJWaWV3Ojp2aXNpYmxlQ29udGVudHNTaXplKCkgY29uc3QKIHsKICAgICBGbG9hdFNpemUg dmlzaWJsZUNvbnRlbnRzU2l6ZShkaXBTaXplKCkpOwotICAgIHZpc2libGVDb250ZW50c1NpemUu c2NhbGUoMSAvIGNvbnRlbnRTY2FsZUZhY3RvcigpKTsKKyAgICBpZiAobV9wYWdlLT51c2VGaXhl ZExheW91dCgpKQorICAgICAgICB2aXNpYmxlQ29udGVudHNTaXplLnNjYWxlKDEgLyBjb250ZW50 U2NhbGVGYWN0b3IoKSk7CiAKICAgICByZXR1cm4gdmlzaWJsZUNvbnRlbnRzU2l6ZTsKIH0K