89507 2012-06-19 13:33:04 -0700 [BlackBerry] Missing notifyContentRendered() when resuming rendering 2012-06-19 15:48:24 -0700 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 jpetsovits jpetsovits tonikitoo webkit.review.bot oldest_to_newest 652736 0 jpetsovits 2012-06-19 13:33:04 -0700 If rendering without a backing store, we're forgetting to issue a notifyContentRendered() call because the forced rerendering in resumeBackingStore() doesn't go through the RenderQueue where all other calls to notifyContentRendered() currently sit. The patch below fixes this by adding a notification where it's necessary. 652742 1 148407 jpetsovits 2012-06-19 13:38:35 -0700 Created attachment 148407 Patch 652898 2 148407 webkit.review.bot 2012-06-19 15:48:19 -0700 Comment on attachment 148407 Patch Clearing flags on attachment: 148407 Committed r120763: <http://trac.webkit.org/changeset/120763> 652899 3 webkit.review.bot 2012-06-19 15:48:24 -0700 All reviewed patches have been landed. Closing bug. 148407 2012-06-19 13:38:35 -0700 2012-06-19 15:48:19 -0700 Patch bug-89507-20120619163813.patch text/plain 1845 jpetsovits U3VidmVyc2lvbiBSZXZpc2lvbjogMTIwNDgyCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2Js YWNrYmVycnkvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0NoYW5nZUxvZwpp bmRleCA0NDVlYmZhNzE2MGU1ZDcwMmVjYjA3NzM4ZmE5OTQxYzk4YjY4OGY3Li4wMzk4Y2NhZTIy NDc5OWVlYTQ3YzUwNzNmOTBhZjMzYWEyNzc2NWI3IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 L2JsYWNrYmVycnkvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFu Z2VMb2cKQEAgLTEsMyArMSwxOSBAQAorMjAxMi0wNi0xOSAgSmFrb2IgUGV0c292aXRzICA8anBl dHNvdml0c0ByaW0uY29tPgorCisgICAgICAgIFtCbGFja0JlcnJ5XSBNaXNzaW5nIG5vdGlmeUNv bnRlbnRSZW5kZXJlZCgpIHdoZW4gcmVzdW1pbmcgcmVuZGVyaW5nCisgICAgICAgIGh0dHBzOi8v YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04OTUwNworICAgICAgICBSSU0gUFIgMTUw MDQ5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQWxs IHRoZSBjdXJyZW50IGNhbGxzIHRvIG5vdGlmeUNvbnRlbnRSZW5kZXJlZCgpIGFyZQorICAgICAg ICBsb2NhdGVkIGluIHRoZSBSZW5kZXJRdWV1ZSwgYnV0IHRoaXMgb21pdHMgdGhlIGNhc2UKKyAg ICAgICAgd2hlbiB3ZSBmb3JjZSByZW5kZXJpbmcgZnJvbSByZXN1bWVCYWNraW5nU3RvcmUoKQor ICAgICAgICBhZnRlciByZW5kZXJpbmcgaGFkIGJlZW4gc3VzcGVuZGVkIGZvciBhIHdoaWxlLgor CisgICAgICAgICogQXBpL1dlYlBhZ2UuY3BwOgorICAgICAgICAoQmxhY2tCZXJyeTo6V2ViS2l0 OjpXZWJQYWdlUHJpdmF0ZTo6cmVzdW1lQmFja2luZ1N0b3JlKToKKwogMjAxMi0wNi0xNSAgQW50 b25pbyBHb21lcyAgPGFnb21lc0ByaW0uY29tPgogCiAgICAgICAgIFtCbGFja0JlcnJ5XSBEaXNh YmxlIFdlYkNvcmU6OkV2ZW50SGFuZGxlciBzeW50aGVzaXplZCBtb3VzZSBldmVudHMgZHVyaW5n IHRvdWNoIHNjcm9sbGluZwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0Fw aS9XZWJQYWdlLmNwcCBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9BcGkvV2ViUGFnZS5jcHAK aW5kZXggMzQ0ZWM0NzIwMzdhMjk4MmQ1MjZjMmQwNzQwYTAyNWEyNDdmZmM0OC4uYWI3MmEyZTVl ZWMzODYwYzRhNTQ0ZGUzOWEwMDUxZGJmMDMwYTE5OSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktp dC9ibGFja2JlcnJ5L0FwaS9XZWJQYWdlLmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVy cnkvQXBpL1dlYlBhZ2UuY3BwCkBAIC0zNTM2LDYgKzM1MzYsOCBAQCB2b2lkIFdlYlBhZ2VQcml2 YXRlOjpyZXN1bWVCYWNraW5nU3RvcmUoKQogICAgICAgICBkaXJlY3RSZW5kZXJpbmcgPSBtX2Jh Y2tpbmdTdG9yZS0+ZC0+c2hvdWxkRGlyZWN0UmVuZGVyaW5nVG9XaW5kb3coKTsKICAgICAgICAg aWYgKG1fYmFja2luZ1N0b3JlLT5kLT5yZW5kZXJWaXNpYmxlQ29udGVudHMoKSAmJiAhbV9iYWNr aW5nU3RvcmUtPmQtPmlzU3VzcGVuZGVkKCkgJiYgIWRpcmVjdFJlbmRlcmluZykKICAgICAgICAg ICAgIG1fYmFja2luZ1N0b3JlLT5kLT5ibGl0VmlzaWJsZUNvbnRlbnRzKCk7CisKKyAgICAgICAg bV9jbGllbnQtPm5vdGlmeUNvbnRlbnRSZW5kZXJlZChtX2JhY2tpbmdTdG9yZS0+ZC0+dmlzaWJs ZUNvbnRlbnRzUmVjdCgpKTsKICAgICB9IGVsc2UgewogICAgICAgICBpZiAobV9iYWNraW5nU3Rv cmUtPmQtPmlzT3BlbkdMQ29tcG9zaXRpbmcoKSkKICAgICAgICAgICAgc2V0Q29tcG9zaXRvckRy YXdzUm9vdExheWVyKGZhbHNlKTsK