85033 2012-04-27 00:10:38 -0700 [Blackberry] remove redundant returns 2012-04-27 08:28:28 -0700 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 0 logingx webkit-unassigned tonikitoo webkit.review.bot yong.li.webkit oldest_to_newest 611288 0 logingx 2012-04-27 00:10:38 -0700 remove redundant returns in Webpage.cpp 611291 1 139141 logingx 2012-04-27 00:26:32 -0700 Created attachment 139141 Patch 611462 2 yong.li.webkit 2012-04-27 07:01:38 -0700 Looks very good to me :) 611525 3 139141 webkit.review.bot 2012-04-27 08:28:24 -0700 Comment on attachment 139141 Patch Clearing flags on attachment: 139141 Committed r115435: <http://trac.webkit.org/changeset/115435> 611526 4 webkit.review.bot 2012-04-27 08:28:28 -0700 All reviewed patches have been landed. Closing bug. 139141 2012-04-27 00:26:32 -0700 2012-04-27 08:28:24 -0700 Patch bug-85033-20120427152629.patch text/plain 2105 logingx U3VidmVyc2lvbiBSZXZpc2lvbjogMTE1NDEwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2Js YWNrYmVycnkvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0NoYW5nZUxvZwpp bmRleCA2MDRlMThkODYyMjcxMWFjYTIwNzM4ZGNkNzI3NmIzMDQ1ZDQ3ZjJhLi43MzEzYTJmN2Q3 Y2EzN2E0OGIzOTZhNTQ5ODMzMjYzZGQ3Zjg4ODE3IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 L2JsYWNrYmVycnkvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFu Z2VMb2cKQEAgLTEsMyArMSwxOCBAQAorMjAxMi0wNC0yNyAgQ2hyaXMgR3VhbiAgPGNocmlzLmd1 YW5AdG9yY2htb2JpbGUuY29tLmNuPgorCisgICAgICAgIFtCbGFja2JlcnJ5XSByZW1vdmUgcmVk dW5kYW50IHJldHVybnMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTg1MDMzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg ICAgICAgUmVtb3ZlIHR3byByZWR1bmRhbnQgcmV0dXJucyB3aGljaCB3ZXJlIGFkZGVkIGJ5IGFj Y2lkZW50LgorCisgICAgICAgIE5vIG5ldyB0ZXN0IGNhc2VzLCBiZWNhdXNlIG5vIGJlaGF2aW91 ciBoYXMgY2hhbmdlZC4KKworICAgICAgICAqIEFwaS9XZWJQYWdlLmNwcDoKKyAgICAgICAgKEJs YWNrQmVycnk6OldlYktpdDo6V2ViUGFnZTo6c2V0U2VsZWN0aW9uKToKKyAgICAgICAgKEJsYWNr QmVycnk6OldlYktpdDo6V2ViUGFnZTo6c2V0Q2FyZXRQb3NpdGlvbik6CisKIDIwMTItMDQtMjYg IFlvbmcgTGkgIDx5b2xpQHJpbS5jb20+CiAKICAgICAgICAgW0JsYWNrQmVycnldIEEgcXVpY2sg Zml4IHRvIHRoZSBwcmV2aW91cyBwYXRjaCByMTE1MjQ1LgpkaWZmIC0tZ2l0IGEvU291cmNlL1dl YktpdC9ibGFja2JlcnJ5L0FwaS9XZWJQYWdlLmNwcCBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJy eS9BcGkvV2ViUGFnZS5jcHAKaW5kZXggODM0NGE2NTk0ODIyYzdmZjEzZjc0MTdhNjU4MzI2Nzc1 ODdiOGUzNi4uMWJjNDRiZTlhMWQ2ZmU2ZTNmZWQ1ZTZlNmZiODg1YTg5YjkwYThmOCAxMDA2NDQK LS0tIGEvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0FwaS9XZWJQYWdlLmNwcAorKysgYi9Tb3Vy Y2UvV2ViS2l0L2JsYWNrYmVycnkvQXBpL1dlYlBhZ2UuY3BwCkBAIC00NDQ5LDcgKzQ0NDksNyBA QCB2b2lkIFdlYlBhZ2U6OnNldFNlbGVjdGlvbihjb25zdCBQbGF0Zm9ybTo6SW50UG9pbnQmIHN0 YXJ0UG9pbnQsIGNvbnN0IFBsYXRmb3JtOgogICAgIGludmFsaWRQb2ludCA9IEludFBvaW50KGVu ZFBvaW50KSA9PSBET01TdXBwb3J0OjpJbnZhbGlkUG9pbnQ7CiAgICAgSW50UG9pbnQgZW5kID0g aW52YWxpZFBvaW50ID8gRE9NU3VwcG9ydDo6SW52YWxpZFBvaW50IDogZC0+bWFwRnJvbVRyYW5z Zm9ybWVkKGVuZFBvaW50KTsKIAotICAgIHJldHVybiBkLT5tX3NlbGVjdGlvbkhhbmRsZXItPnNl dFNlbGVjdGlvbihzdGFydCwgZW5kKTsKKyAgICBkLT5tX3NlbGVjdGlvbkhhbmRsZXItPnNldFNl bGVjdGlvbihzdGFydCwgZW5kKTsKIH0KIAogdm9pZCBXZWJQYWdlOjpzZXRDYXJldFBvc2l0aW9u KGNvbnN0IFBsYXRmb3JtOjpJbnRQb2ludCYgcG9zaXRpb24pCkBAIC00NDU4LDcgKzQ0NTgsNyBA QCB2b2lkIFdlYlBhZ2U6OnNldENhcmV0UG9zaXRpb24oY29uc3QgUGxhdGZvcm06OkludFBvaW50 JiBwb3NpdGlvbikKICAgICAgICAgcmV0dXJuOwogICAgIC8vIEhhbmRsZWQgYnkgc2VsZWN0aW9u IGhhbmRsZXIgYXMgaXQncyBwb2ludCBiYXNlZC4KICAgICAvLyBUcmFuc2Zvcm0gdGhpcyBldmVu dHMgY29vcmRpbmF0ZXMgdG8gd2Via2l0IGNvbnRlbnQgY29vcmRpbmF0ZXMuCi0gICAgcmV0dXJu IGQtPm1fc2VsZWN0aW9uSGFuZGxlci0+c2V0Q2FyZXRQb3NpdGlvbihkLT5tYXBGcm9tVHJhbnNm b3JtZWQocG9zaXRpb24pKTsKKyAgICBkLT5tX3NlbGVjdGlvbkhhbmRsZXItPnNldENhcmV0UG9z aXRpb24oZC0+bWFwRnJvbVRyYW5zZm9ybWVkKHBvc2l0aW9uKSk7CiB9CiAKIHZvaWQgV2ViUGFn ZTo6c2VsZWN0QXRQb2ludChjb25zdCBQbGF0Zm9ybTo6SW50UG9pbnQmIGxvY2F0aW9uKQo=