109124 2013-02-06 18:35:00 -0800 [BlackBerry] Reader Mode: Opening two links quickly from reader mode causes browser bad state 2013-02-07 07:10:01 -0800 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 0 xuewen.ok webkit-unassigned mifenton rwlbuis tonikitoo webkit.review.bot yong.li.webkit oldest_to_newest 826968 0 xuewen.ok 2013-02-06 18:35:00 -0800 Description *********************** It is possible to put the browser into a broken state by rotating the device while following a link in Reader Mode. See attached images for examples of state in question. Steps to Reproduce *********************** 1. Load up the browser, navigate to a page with text links on it 2. Enter Reader Mode on the page 3. Tap a link, and immediately rotate the device (almost at the same time) Expected Results *********************** The browser should rotate and the page should load (not necessarily in that order) Actual Results *********************** Sometimes, the browser will enter a broken state If you click the links quickly, the browser can also be break to bad state since the creating window reentry. 826993 1 186978 xuewen.ok 2013-02-06 18:54:01 -0800 Created attachment 186978 patch 827263 2 186978 buildbot 2013-02-07 03:38:05 -0800 Comment on attachment 186978 patch Attachment 186978 did not pass win-ews (win): Output: http://queues.webkit.org/results/16445065 827476 3 186978 webkit.review.bot 2013-02-07 07:09:58 -0800 Comment on attachment 186978 patch Clearing flags on attachment: 186978 Committed r142117: <http://trac.webkit.org/changeset/142117> 827477 4 webkit.review.bot 2013-02-07 07:10:01 -0800 All reviewed patches have been landed. Closing bug. 186978 2013-02-06 18:54:01 -0800 2013-02-07 07:09:58 -0800 patch 0001-BlackBerry-Reader-Mode-Opening-two-links-quickly-fro.patch text/plain 2550 xuewen.ok RnJvbSBiMWMxMjVlZGVmMTI2YTJhZGEyZmRiZTlhNjg1YzYxODE4N2QwOTUwIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBTZWFuIFdhbmcgPFh1ZXdlbi5XYW5nQHRvcmNobW9iaWxlLmNv bS5jbj4KRGF0ZTogVGh1LCA3IEZlYiAyMDEzIDEwOjQ5OjM3ICswODAwClN1YmplY3Q6IFtQQVRD SF0gW0JsYWNrQmVycnldIFJlYWRlciBNb2RlOiBPcGVuaW5nIHR3byBsaW5rcyBxdWlja2x5IGZy b20KIHJlYWRlciBtb2RlIGNhdXNlcyBicm93c2VyIGJhZCBzdGF0ZQogaHR0cHM6Ly9idWdzLndl YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEwOTEyNAoKUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT ISkuCgpSSU0gQlVHIDI5MTI0NiBJbnRlcm5hbGx5IHJldmlld2VkIGJ5IFlvbmdMaS4KTW92ZSB0 aGUgV2ViUGFnZUdyb3VwTG9hZERlZmVycmVyIG9iamVjdCBmcm9tIENocm9tZUNsaWVudEJsYWNr QmVycnk6OmNyZWF0ZVdpbmRvdygpCmludG8gV2ViUGFnZUNsaWVudEltcGw6OmNyZWF0ZVdpbmRv dygpIHRvIG1ha2UgaXQgbW9yZSBjbG9zZSB0byBpdHMgcHJvdGVjdGluZyBwbGFjZS4KCiogV2Vi Q29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50QmxhY2tCZXJyeS5jcHA6CihXZWJDb3JlOjpDaHJvbWVD bGllbnRCbGFja0JlcnJ5OjpjcmVhdGVXaW5kb3cpOgotLS0KIFNvdXJjZS9XZWJLaXQvYmxhY2ti ZXJyeS9DaGFuZ2VMb2cgICAgICAgICAgICAgICAgICAgICAgIHwgICAxNCArKysrKysrKysrKysr KwogLi4uL2JsYWNrYmVycnkvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50QmxhY2tCZXJyeS5j cHAgfCAgICAzIC0tLQogMiBmaWxlcyBjaGFuZ2VkLCAxNCBpbnNlcnRpb25zKCspLCAzIGRlbGV0 aW9ucygtKQoKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFuZ2VMb2cg Yi9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkvQ2hhbmdlTG9nCmluZGV4IDJjMWUxZWQuLmI2N2E0 MmEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFuZ2VMb2cKKysrIGIv U291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEz LTAyLTA2ICBTZWFuIFdhbmcgIDxYdWV3ZW4uV2FuZ0B0b3JjaG1vYmlsZS5jb20uY24+CisKKyAg ICAgICAgW0JsYWNrQmVycnldIFJlYWRlciBNb2RlOiBPcGVuaW5nIHR3byBsaW5rcyBxdWlja2x5 IGZyb20gcmVhZGVyIG1vZGUgY2F1c2VzIGJyb3dzZXIgYmFkIHN0YXRlCisgICAgICAgIGh0dHBz Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDkxMjQKKworICAgICAgICBSZXZp ZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBSSU0gQlVHIDI5MTI0NiBJbnRlcm5h bGx5IHJldmlld2VkIGJ5IFlvbmdMaS4KKyAgICAgICAgTW92ZSB0aGUgV2ViUGFnZUdyb3VwTG9h ZERlZmVycmVyIG9iamVjdCBmcm9tIENocm9tZUNsaWVudEJsYWNrQmVycnk6OmNyZWF0ZVdpbmRv dygpCisgICAgICAgIGludG8gV2ViUGFnZUNsaWVudEltcGw6OmNyZWF0ZVdpbmRvdygpIHRvIG1h a2UgaXQgbW9yZSBjbG9zZSB0byBpdHMgcHJvdGVjdGluZyBwbGFjZS4KKworICAgICAgICAqIFdl YkNvcmVTdXBwb3J0L0Nocm9tZUNsaWVudEJsYWNrQmVycnkuY3BwOgorICAgICAgICAoV2ViQ29y ZTo6Q2hyb21lQ2xpZW50QmxhY2tCZXJyeTo6Y3JlYXRlV2luZG93KToKKwogMjAxMy0wMi0wNiAg SmFrb2IgUGV0c292aXRzICA8anBldHNvdml0c0ByaW0uY29tPgogCiAgICAgICAgIFtCbGFja0Jl cnJ5XSBSZWZhY3RvciByZW5kZXJDb250ZW50cygpIGZvciBjbGVhbmVyIGNvZGUuCmRpZmYgLS1n aXQgYS9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50 QmxhY2tCZXJyeS5jcHAgYi9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkvV2ViQ29yZVN1cHBvcnQv Q2hyb21lQ2xpZW50QmxhY2tCZXJyeS5jcHAKaW5kZXggMTA4NDdlMi4uNzIwY2M2ZSAxMDA2NDQK LS0tIGEvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L1dlYkNvcmVTdXBwb3J0L0Nocm9tZUNsaWVu dEJsYWNrQmVycnkuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9XZWJDb3JlU3Vw cG9ydC9DaHJvbWVDbGllbnRCbGFja0JlcnJ5LmNwcApAQCAtMjM2LDkgKzIzNiw2IEBAIFBhZ2Uq IENocm9tZUNsaWVudEJsYWNrQmVycnk6OmNyZWF0ZVdpbmRvdyhGcmFtZSogZnJhbWUsIGNvbnN0 IEZyYW1lTG9hZFJlcXVlc3QmCiAgICAgICAgIHJldHVybiAwOwogI2VuZGlmCiAKLSAgICBQYWdl R3JvdXBMb2FkRGVmZXJyZXIgZGVmZXJyZXIobV93ZWJQYWdlUHJpdmF0ZS0+bV9wYWdlLCB0cnVl KTsKLSAgICBUaW1lckJhc2U6OmZpcmVUaW1lcnNJbk5lc3RlZEV2ZW50TG9vcCgpOwotCiAgICAg aW50IHggPSBmZWF0dXJlcy54U2V0ID8gZmVhdHVyZXMueCA6IDA7CiAgICAgaW50IHkgPSBmZWF0 dXJlcy55U2V0ID8gZmVhdHVyZXMueSA6IDA7CiAgICAgaW50IHdpZHRoID0gZmVhdHVyZXMud2lk dGhTZXQ/IGZlYXR1cmVzLndpZHRoIDogLTE7Ci0tIAoxLjcuMTAuNAoK