95138 2012-08-27 15:30:12 -0700 [BlackBerry] UpdateViewport uses wrong size for frameView 2012-08-28 06:02:32 -0700 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 0 andchen webkit-unassigned mifenton tonikitoo webkit.review.bot oldest_to_newest 705762 0 andchen 2012-08-27 15:30:12 -0700 It uses screen size instead of actual visible size. It causes frame rect always larger than the actual visible one and ends up with find-on-page does not scroll correctly sometimes. 705807 1 160835 andchen 2012-08-27 15:50:25 -0700 Created attachment 160835 Patch 706268 2 160835 webkit.review.bot 2012-08-28 06:02:29 -0700 Comment on attachment 160835 Patch Clearing flags on attachment: 160835 Committed r126871: <http://trac.webkit.org/changeset/126871> 706269 3 webkit.review.bot 2012-08-28 06:02:32 -0700 All reviewed patches have been landed. Closing bug. 160835 2012-08-27 15:50:25 -0700 2012-08-28 06:02:29 -0700 Patch bug-95138-20120827185007.patch text/plain 1896 andchen U3VidmVyc2lvbiBSZXZpc2lvbjogMTI2Nzg3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2Js YWNrYmVycnkvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0NoYW5nZUxvZwpp bmRleCA1NmQ5ZWQ1NDkzZGJjYzZjMDcwM2YwMzYwMGZmZWU4ZTNlMGJjZWYzLi4xZTQ1MjYxYzM3 ZGU4MGVmOWYwY2FkYTg3YjQ2YjBmOTBkODEyZmEyIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 L2JsYWNrYmVycnkvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFu Z2VMb2cKQEAgLTEsMyArMSwxNyBAQAorMjAxMi0wOC0yNyAgQW5keSBDaGVuICA8YW5kY2hlbkBy aW0uY29tPgorCisgICAgICAgIFtCbGFja0JlcnJ5XSBVcGRhdGVWaWV3cG9ydCB1c2VzIHdyb25n IHNpemUgZm9yIGZyYW1lVmlldworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93 X2J1Zy5jZ2k/aWQ9OTUxMzgKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K KyAgICAgICAgSW50ZXJuYWxseSByZXZpZXdlZCBieSBKYWtvYiBQZXRzb3ZpdHMuCisKKyAgICAg ICAgVXNlIGFjdHVhbCB2aXNpYmxlIHNpemUgaW5zdGVhZCBvZiBzY3JlZW4gc2l6ZSB3aGVuIHVw ZGF0aW5nIHZpZXdwb3J0IHNpemUsCisgICAgICAgIG90aGVyd2lzZSwgZnJhbWVWaWV3IHNpemUg d291bGQgYmUgdG9vIGJpZy4KKworICAgICAgICAqIEFwaS9XZWJQYWdlLmNwcDoKKyAgICAgICAg KEJsYWNrQmVycnk6OldlYktpdDo6V2ViUGFnZVByaXZhdGU6OnVwZGF0ZVZpZXdwb3J0U2l6ZSk6 CisKIDIwMTItMDgtMjcgIExlbyBZYW5nICA8bGVveWFuZ0ByaW0uY29tPgogCiAgICAgICAgIFtC bGFja0JlcnJ5XSBOb3QgdXNpbmcgYWJzb2x1dGVWaXNpYmxlT3ZlclNpemUgd2hlbiByb3RhdGlu ZwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0FwaS9XZWJQYWdlLmNwcCBi L1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9BcGkvV2ViUGFnZS5jcHAKaW5kZXggYTk5ODIzZWY2 NmU1YmU4M2ViMWZmNjY2ODM0MjEyZjk1MzViY2NkMC4uYTI3NzZhYjEwZDczOGMwYTU1YmFiMjgz MTg0MGUyMDZkY2JhMDk1YiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0Fw aS9XZWJQYWdlLmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkvQXBpL1dlYlBhZ2Uu Y3BwCkBAIC0xNTY3LDEwICsxNTY3LDExIEBAIHZvaWQgV2ViUGFnZVByaXZhdGU6OnVwZGF0ZVZp ZXdwb3J0U2l6ZShib29sIHNldEZpeGVkUmVwb3J0ZWRTaXplLCBib29sIHNlbmRSZXNpCiAgICAg aWYgKCFtX2JhY2tpbmdTdG9yZSkKICAgICAgICAgcmV0dXJuOwogICAgIEFTU0VSVChtX21haW5G cmFtZS0+dmlldygpKTsKKyAgICBJbnRTaXplIHZpc2libGVTaXplID0gYWN0dWFsVmlzaWJsZVNp emUoKTsKICAgICBpZiAoc2V0Rml4ZWRSZXBvcnRlZFNpemUpCi0gICAgICAgIG1fbWFpbkZyYW1l LT52aWV3KCktPnNldEZpeGVkUmVwb3J0ZWRTaXplKGFjdHVhbFZpc2libGVTaXplKCkpOworICAg ICAgICBtX21haW5GcmFtZS0+dmlldygpLT5zZXRGaXhlZFJlcG9ydGVkU2l6ZSh2aXNpYmxlU2l6 ZSk7CiAKLSAgICBJbnRSZWN0IGZyYW1lUmVjdCA9IEludFJlY3Qoc2Nyb2xsUG9zaXRpb24oKSwg dmlld3BvcnRTaXplKCkpOworICAgIEludFJlY3QgZnJhbWVSZWN0ID0gSW50UmVjdChzY3JvbGxQ b3NpdGlvbigpLCB2aXNpYmxlU2l6ZSk7CiAgICAgaWYgKGZyYW1lUmVjdCAhPSBtX21haW5GcmFt ZS0+dmlldygpLT5mcmFtZVJlY3QoKSkgewogICAgICAgICBtX21haW5GcmFtZS0+dmlldygpLT5z ZXRGcmFtZVJlY3QoZnJhbWVSZWN0KTsKICAgICAgICAgbV9tYWluRnJhbWUtPnZpZXcoKS0+YWRq dXN0Vmlld1NpemUoKTsK