38431 2010-05-02 04:38:39 -0700 Web Inspector: debugger shortcuts are processed twice if source frame has focus. 2010-05-02 08:28:08 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 pfeldman pfeldman bweinstein commit-queue joepeck timothy yurys oldest_to_newest 219819 0 pfeldman 2010-05-02 04:38:39 -0700 Reason is the old code that delegates keyboard events from source frame to main document. It should have been deleted when SourceFrame stopped being an iframe. 219820 1 54870 pfeldman 2010-05-02 04:42:21 -0700 Created attachment 54870 [PATCH] Proposed fix. 219860 2 54870 commit-queue 2010-05-02 08:28:03 -0700 Comment on attachment 54870 [PATCH] Proposed fix. Clearing flags on attachment: 54870 Committed r58655: <http://trac.webkit.org/changeset/58655> 219861 3 commit-queue 2010-05-02 08:28:08 -0700 All reviewed patches have been landed. Closing bug. 54870 2010-05-02 04:42:21 -0700 2010-05-02 08:28:03 -0700 [PATCH] Proposed fix. debugger_shortcuts text/plain 2245 pfeldman ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg YmIxZDQ4ZS4uYTlkOTcyMiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNSBAQAorMjAxMC0wNS0wMiAgUGF2ZWwgRmVsZG1h biAgPHBmZWxkbWFuQGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg KE9PUFMhKS4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBkZWJ1Z2dlciBzaG9ydGN1dHMgYXJl IHByb2Nlc3NlZCB0d2ljZSBpZiBzb3VyY2UgZnJhbWUgaGFzIGZvY3VzLgorCisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zODQzMQorCisgICAgICAgICog aW5zcGVjdG9yL2Zyb250LWVuZC9Tb3VyY2VGcmFtZS5qczoKKyAgICAgICAgKFdlYkluc3BlY3Rv ci5Tb3VyY2VGcmFtZSk6CisgICAgICAgIChXZWJJbnNwZWN0b3IuU291cmNlRnJhbWUucHJvdG90 eXBlLl9jcmVhdGVWaWV3ZXJJZk5lZWRlZCk6CisKIDIwMTAtMDQtMjggIE5vYW0gUm9zZW50aGFs ICA8bm9hbS5yb3NlbnRoYWxAbm9raWEuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEtlbm5l dGggUm9oZGUgQ2hyaXN0aWFuc2VuLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9pbnNwZWN0b3IvZnJv bnQtZW5kL1NvdXJjZUZyYW1lLmpzIGIvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1NvdXJj ZUZyYW1lLmpzCmluZGV4IDYwZmRhNTguLjkzNDQ1ZjYgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvaW5z cGVjdG9yL2Zyb250LWVuZC9Tb3VyY2VGcmFtZS5qcworKysgYi9XZWJDb3JlL2luc3BlY3Rvci9m cm9udC1lbmQvU291cmNlRnJhbWUuanMKQEAgLTM5LDcgKzM5LDYgQEAgV2ViSW5zcGVjdG9yLlNv dXJjZUZyYW1lID0gZnVuY3Rpb24ocGFyZW50RWxlbWVudCwgYWRkQnJlYWtwb2ludERlbGVnYXRl LCByZW1vdmUKICAgICB0aGlzLl9yb3dNZXNzYWdlcyA9IHt9OwogICAgIHRoaXMuX21lc3NhZ2VC dWJibGVzID0ge307CiAgICAgdGhpcy5icmVha3BvaW50cyA9IFtdOwotICAgIHRoaXMuX3Nob3J0 Y3V0cyA9IHt9OwogCiAgICAgdGhpcy5fbG9hZGVkID0gZmFsc2U7CiAKQEAgLTE2Nyw3ICsxNjYs NiBAQCBXZWJJbnNwZWN0b3IuU291cmNlRnJhbWUucHJvdG90eXBlID0gewogCiAgICAgICAgIHRo aXMuX3RleHRWaWV3ZXIgPSBuZXcgV2ViSW5zcGVjdG9yLlRleHRWaWV3ZXIodGhpcy5fdGV4dE1v ZGVsLCBXZWJJbnNwZWN0b3IucGxhdGZvcm0sIHRoaXMuX3VybCk7CiAgICAgICAgIHZhciBlbGVt ZW50ID0gdGhpcy5fdGV4dFZpZXdlci5lbGVtZW50OwotICAgICAgICBlbGVtZW50LmFkZEV2ZW50 TGlzdGVuZXIoImtleWRvd24iLCB0aGlzLl9rZXlEb3duLmJpbmQodGhpcyksIHRydWUpOwogICAg ICAgICBlbGVtZW50LmFkZEV2ZW50TGlzdGVuZXIoImNvbnRleHRtZW51IiwgdGhpcy5fY29udGV4 dE1lbnUuYmluZCh0aGlzKSwgdHJ1ZSk7CiAgICAgICAgIGVsZW1lbnQuYWRkRXZlbnRMaXN0ZW5l cigibW91c2Vkb3duIiwgdGhpcy5fbW91c2VEb3duLmJpbmQodGhpcyksIHRydWUpOwogICAgICAg ICBlbGVtZW50LmFkZEV2ZW50TGlzdGVuZXIoIm1vdXNlbW92ZSIsIHRoaXMuX21vdXNlTW92ZS5i aW5kKHRoaXMpLCB0cnVlKTsKQEAgLTY3MiwxNyArNjcwLDYgQEAgV2ViSW5zcGVjdG9yLlNvdXJj ZUZyYW1lLnByb3RvdHlwZSA9IHsKICAgICAgICAgcmV0dXJuIGNvbmRpdGlvbkVsZW1lbnQ7CiAg ICAgfSwKIAotICAgIF9rZXlEb3duOiBmdW5jdGlvbihldmVudCkKLSAgICB7Ci0gICAgICAgIHZh ciBzaG9ydGN1dCA9IFdlYkluc3BlY3Rvci5LZXlib2FyZFNob3J0Y3V0Lm1ha2VLZXlGcm9tRXZl bnQoZXZlbnQpOwotICAgICAgICB2YXIgaGFuZGxlciA9IHRoaXMuX3Nob3J0Y3V0c1tzaG9ydGN1 dF07Ci0gICAgICAgIGlmIChoYW5kbGVyKSB7Ci0gICAgICAgICAgICBoYW5kbGVyKGV2ZW50KTsK LSAgICAgICAgICAgIGV2ZW50LnByZXZlbnREZWZhdWx0KCk7Ci0gICAgICAgIH0gZWxzZQotICAg ICAgICAgICAgV2ViSW5zcGVjdG9yLmRvY3VtZW50S2V5RG93bihldmVudCk7Ci0gICAgfSwKLQog ICAgIF9ldmFsU2VsZWN0aW9uSW5DYWxsRnJhbWU6IGZ1bmN0aW9uKGV2ZW50KQogICAgIHsKICAg ICAgICAgaWYgKCFXZWJJbnNwZWN0b3IucGFuZWxzLnNjcmlwdHMgfHwgIVdlYkluc3BlY3Rvci5w YW5lbHMuc2NyaXB0cy5wYXVzZWQpCg==