89369 2012-06-18 11:50:46 -0700 [BlackBerry] NetworkJob::startNewJobWithRequest() should check the frame pointer 2012-06-18 14:35:09 -0700 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 yong.li.webkit webkit-unassigned tonikitoo webkit.review.bot oldest_to_newest 651599 0 yong.li.webkit 2012-06-18 11:50:46 -0700 NetworkJob::startNewJobWithRequest() shouldn't proceed when the frame is null, because this means it is a PingLoader job and should just finish. 651619 1 148148 yong.li.webkit 2012-06-18 12:35:51 -0700 Created attachment 148148 the patch 651636 2 148148 yong.li.webkit 2012-06-18 12:55:09 -0700 Comment on attachment 148148 the patch thanks! 651737 3 148148 webkit.review.bot 2012-06-18 14:35:02 -0700 Comment on attachment 148148 the patch Clearing flags on attachment: 148148 Committed r120626: <http://trac.webkit.org/changeset/120626> 651738 4 webkit.review.bot 2012-06-18 14:35:09 -0700 All reviewed patches have been landed. Closing bug. 148148 2012-06-18 12:35:51 -0700 2012-06-18 14:35:02 -0700 the patch 89369.patch text/plain 1627 yong.li.webkit ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCBlNmYyMzE5Li5lMjliMzdiIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYg QEAKKzIwMTItMDYtMTggIFlvbmcgTGkgIDx5b2xpQHJpbS5jb20+CisKKyAgICAgICAgW0JsYWNr QmVycnldIE5ldHdvcmtKb2I6OnN0YXJ0TmV3Sm9iV2l0aFJlcXVlc3QoKSBzaG91bGQgY2hlY2sg dGhlIGZyYW1lIHBvaW50ZXIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTg5MzY5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK KyAgICAgICAgTmV0d29ya0pvYjo6c3RhcnROZXdKb2JXaXRoUmVxdWVzdCgpIHNob3VsZCBzdG9w IHByb2NlZWRpbmcgaWYgdGhlIGZyYW1lIGlzIG51bGwsCisgICAgICAgIHdoaWNoIG1lYW5zIGl0 IGlzIGEgUGluZ0xvYWRlciBqb2IgYW5kIHNob3VsZCBqdXN0IGZpbmlzaCBub3cuCisKKyAgICAg ICAgKiBwbGF0Zm9ybS9uZXR3b3JrL2JsYWNrYmVycnkvTmV0d29ya0pvYi5jcHA6CisgICAgICAg IChXZWJDb3JlOjpOZXR3b3JrSm9iOjpzdGFydE5ld0pvYldpdGhSZXF1ZXN0KToKKwogMjAxMi0w Ni0xMyAgQW5kcmV5IEtvc3lha292ICA8Y2FzZXFAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFdl YiBJbnNwZWN0b3I6IHVzZSBDU1MgZ3JhZGllbnRzIGluc3RlYWQgb2YgaW1hZ2VzIGZvciB0aW1l bGluZSBiYXJzLCBjaGVja2JveGVzICYgYnVsbGV0cwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNv cmUvcGxhdGZvcm0vbmV0d29yay9ibGFja2JlcnJ5L05ldHdvcmtKb2IuY3BwIGIvU291cmNlL1dl YkNvcmUvcGxhdGZvcm0vbmV0d29yay9ibGFja2JlcnJ5L05ldHdvcmtKb2IuY3BwCmluZGV4IDYw MTA3NjMuLjFjMjJiYjQ5IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3 b3JrL2JsYWNrYmVycnkvTmV0d29ya0pvYi5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZv cm0vbmV0d29yay9ibGFja2JlcnJ5L05ldHdvcmtKb2IuY3BwCkBAIC01NDEsNiArNTQxLDExIEBA IGJvb2wgTmV0d29ya0pvYjo6cmV0cnlBc0ZUUERpcmVjdG9yeSgpCiAKIGJvb2wgTmV0d29ya0pv Yjo6c3RhcnROZXdKb2JXaXRoUmVxdWVzdChSZXNvdXJjZVJlcXVlc3QmIG5ld1JlcXVlc3QsIGJv b2wgaW5jcmVhc1JlZGlyZWN0Q291bnQpCiB7CisgICAgLy8gbV9mcmFtZSBjYW4gYmUgbnVsbCBp ZiB0aGlzIGlzIGEgUGluZ0xvYWRlciBqb2IgKFNlZSBOZXR3b3JrSm9iOjppbml0aWFsaXplKS4K KyAgICAvLyBJbiB0aGlzIGNhc2Ugd2UgZG9uJ3Qgc3RhcnQgbmV3IHJlcXVlc3QuCisgICAgaWYg KCFtX2ZyYW1lKQorICAgICAgICByZXR1cm4gZmFsc2U7CisKICAgICBpZiAoaXNDbGllbnRBdmFp bGFibGUoKSkgewogICAgICAgICBSZWN1cnNpb25HdWFyZCBndWFyZChtX2NhbGxpbmdDbGllbnQp OwogICAgICAgICBtX2hhbmRsZS0+Y2xpZW50KCktPndpbGxTZW5kUmVxdWVzdChtX2hhbmRsZS5n ZXQoKSwgbmV3UmVxdWVzdCwgbV9yZXNwb25zZSk7Cg==