29958 2009-10-01 03:19:39 -0700 Inspector console stops working while JS in IFRAME is paused 2009-10-05 02:06:21 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) Mac (Intel) OS X 10.6 RESOLVED FIXED P2 Normal --- 1 fds pfeldman pfeldman pmuellr timothy oldest_to_newest 151491 0 40431 fds 2009-10-01 03:19:39 -0700 Created attachment 40431 Test pages to demonstrate the regression In recent WebKit nightly builds (for example, AppleWebKit/532.2+ r48958), when you pause JavaScript execution in the Web Inspector, either with a breakpoint or by pressing the "Pause script execution" button, *if the script getting paused was running inside the IFRAME*, the inspector console stops working and only results in the following error message no matter what you enter: TypeError: Result of expression 'window._inspectorCommandLineAPI' [undefined] is not an object. The console only starts functioning again once script execution is resumed. It used to work fine even when paused in Safari 4.0.3. Simple test case attached: 1. Open "main.html", which includes iframe.html that has a script attached to the button's onclick event. 2. Open Web Inspector, enable script debugging and press the "Pause script execution" button or just place a breakpoint inside the iframe.html JS function. 3. Try pressing the button. 4. While JS execution is paused, open the Web Inspector console and enter anything. 151884 1 pmuellr 2009-10-02 10:20:41 -0700 Able to reproduce with the supplied test case - thanks! pfeldman, this seems like one for you ... 152081 2 40577 pfeldman 2009-10-03 09:31:55 -0700 Created attachment 40577 patch 152240 3 pfeldman 2009-10-05 02:06:21 -0700 Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/inspector/front-end/InjectedScript.js Committed r49085 40431 2009-10-01 03:19:39 -0700 2009-10-01 03:19:39 -0700 Test pages to demonstrate the regression bugtest.zip application/zip 630 fds UEsDBBQDAAAIAEFhQTu36fhvHgEAALABAAALAAAAaWZyYW1lLmh0bWxVUNtKAzEQfd58xZiXWijd D+i24KVCBbVoQXwzzY7d2DQJ2Ul3F/HfTRoLCiFzyeRcprq4fbrZvK2X0NBBL1h1DijqGEiRxsV1 2EGNBwuru+erh2VV5jZjVSu9cgQ0OJxzwp7KT3EUucvjwEcwkpQ1QNjS5Ri+WHEUHvoZK1jRwxz4 o+2AGkFw/wLYowyncdWCE6HFegLkB0BD6JXZgTADNSlRJv5CeMUtrEzrUJL1IK1prcYJaLVHeO+U qW03mnLGCqHR0+WoQa0tdNbrejSese9oocxqFyn9db219RDrTRNlxJOIsvMpq1x8V8aFs+ltILKG w1HoEMt1Ug0Z8q8f819qwjQgtZJ7oESTYaYcrDl15zxvjEe60qUri/oBUEsDBBQDAAAIAK9gQTs5 856BggAAAKgAAAAJAAAAbWFpbi5odG1ss1F08XcOiQxwVcgoyc2x47KBUamJKUCqJLMkJ9XOqTRd ISU1N1+hIDE91UYfLAiU1IcqSspPqbTjCsnILFYAopKMVIXcxMw8sGI9LpsCoIrMtKLE3FSF4qJk WyUIWw9kj5JCcUllTqqtUnlmSkmGlYKFgaqSnY0+WAXYggIQATYeAFBLAQIUAxQDAAAIAEFhQTu3 6fhvHgEAALABAAALAAAAAAAAAAAAIICkgQAAAABpZnJhbWUuaHRtbFBLAQIUAxQDAAAIAK9gQTs5 856BggAAAKgAAAAJAAAAAAAAAAAAIICkgUcBAABtYWluLmh0bWxQSwUGAAAAAAIAAgBwAAAA8AEA AAAA 40577 2009-10-03 09:31:55 -0700 2009-10-04 22:10:27 -0700 patch console_api text/plain 3455 pfeldman ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg ZDkzOTdmNy4uOGU4YzNjNiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBAQAorMjAwOS0xMC0wMyAgUGF2ZWwgRmVsZG1h biAgPHBmZWxkbWFuQGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBUaW1vdGh5 IEhhdGNoZXIuCisKKyAgICAgICAgV2ViIEluc3BlY3RvciBjb25zb2xlIHN0b3BzIHdvcmtpbmcg d2hpbGUgSlMgaW4gSUZSQU1FIGlzIHBhdXNlZC4KKworICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mjk5NTgKKworICAgICAgICAqIGluc3BlY3Rvci9mcm9u dC1lbmQvSW5qZWN0ZWRTY3JpcHQuanM6CisgICAgICAgIChJbmplY3RlZFNjcmlwdC5fZXZhbHVh dGVPbik6CisgICAgICAgIChJbmplY3RlZFNjcmlwdC5hZGRJbnNwZWN0ZWROb2RlKToKKyAgICAg ICAgKEluamVjdGVkU2NyaXB0Ll9lbnN1cmVDb21tYW5kTGluZUFQSUluc3RhbGxlZCk6CisKIDIw MDktMTAtMDIgIFBhdmVsIEZlbGRtYW4gIDxwZmVsZG1hbkBjaHJvbWl1bS5vcmc+CiAKICAgICAg ICAgUmV2aWV3ZWQgYnkgVGltb3RoeSBIYXRjaGVyLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9pbnNw ZWN0b3IvZnJvbnQtZW5kL0luamVjdGVkU2NyaXB0LmpzIGIvV2ViQ29yZS9pbnNwZWN0b3IvZnJv bnQtZW5kL0luamVjdGVkU2NyaXB0LmpzCmluZGV4IDg3MjkzYjguLmIwMmJmNzggMTAwNjQ0Ci0t LSBhL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9JbmplY3RlZFNjcmlwdC5qcworKysgYi9X ZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvSW5qZWN0ZWRTY3JpcHQuanMKQEAgLTU1Myw3ICs1 NTMsNyBAQCBJbmplY3RlZFNjcmlwdC5fZXZhbHVhdGVBbmRXcmFwID0gZnVuY3Rpb24oZXZhbEZ1 bmN0aW9uLCBvYmplY3QsIGV4cHJlc3Npb24pCiAKIEluamVjdGVkU2NyaXB0Ll9ldmFsdWF0ZU9u ID0gZnVuY3Rpb24oZXZhbEZ1bmN0aW9uLCBvYmplY3QsIGV4cHJlc3Npb24pCiB7Ci0gICAgSW5q ZWN0ZWRTY3JpcHQuX2Vuc3VyZUNvbW1hbmRMaW5lQVBJSW5zdGFsbGVkKCk7CisgICAgSW5qZWN0 ZWRTY3JpcHQuX2Vuc3VyZUNvbW1hbmRMaW5lQVBJSW5zdGFsbGVkKGV2YWxGdW5jdGlvbiwgb2Jq ZWN0KTsKICAgICAvLyBTdXJyb3VuZCB0aGUgZXhwcmVzc2lvbiBpbiB3aXRoIHN0YXRlbWVudHMg dG8gaW5qZWN0IG91ciBjb21tYW5kIGxpbmUgQVBJIHNvIHRoYXQKICAgICAvLyB0aGUgd2luZG93 IG9iamVjdCBwcm9wZXJ0aWVzIHN0aWxsIHRha2UgbW9yZSBwcmVjZWRlbnQgdGhhbiBvdXIgQVBJ IGZ1bmN0aW9ucy4KICAgICBleHByZXNzaW9uID0gIndpdGggKHdpbmRvdy5faW5zcGVjdG9yQ29t bWFuZExpbmVBUEkpIHsgd2l0aCAod2luZG93KSB7ICIgKyBleHByZXNzaW9uICsgIiB9IH0iOwpA QCAtNTcyLDcgKzU3Miw3IEBAIEluamVjdGVkU2NyaXB0LmFkZEluc3BlY3RlZE5vZGUgPSBmdW5j dGlvbihub2RlSWQpCiAgICAgaWYgKCFub2RlKQogICAgICAgICByZXR1cm4gZmFsc2U7CiAKLSAg ICBJbmplY3RlZFNjcmlwdC5fZW5zdXJlQ29tbWFuZExpbmVBUElJbnN0YWxsZWQoKTsKKyAgICBJ bmplY3RlZFNjcmlwdC5fZW5zdXJlQ29tbWFuZExpbmVBUElJbnN0YWxsZWQoSW5qZWN0ZWRTY3Jp cHQuX3dpbmRvdygpLmV2YWwsIEluamVjdGVkU2NyaXB0Ll93aW5kb3coKSk7CiAgICAgdmFyIGlu c3BlY3RlZE5vZGVzID0gSW5qZWN0ZWRTY3JpcHQuX3dpbmRvdygpLl9pbnNwZWN0b3JDb21tYW5k TGluZUFQSS5faW5zcGVjdGVkTm9kZXM7CiAgICAgaW5zcGVjdGVkTm9kZXMudW5zaGlmdChub2Rl KTsKICAgICBpZiAoaW5zcGVjdGVkTm9kZXMubGVuZ3RoID49IDUpCkBAIC04ODAsMTMgKzg4MCwx MSBAQCBJbmplY3RlZFNjcmlwdC5faW5zcGVjdE9iamVjdCA9IGZ1bmN0aW9uKG8pCiAgICAgfQog fQogCi1JbmplY3RlZFNjcmlwdC5fZW5zdXJlQ29tbWFuZExpbmVBUElJbnN0YWxsZWQgPSBmdW5j dGlvbihpbnNwZWN0ZWRXaW5kb3cpCitJbmplY3RlZFNjcmlwdC5fZW5zdXJlQ29tbWFuZExpbmVB UElJbnN0YWxsZWQgPSBmdW5jdGlvbihldmFsRnVuY3Rpb24sIGV2YWxPYmplY3QpCiB7Ci0gICAg dmFyIGluc3BlY3RlZFdpbmRvdyA9IEluamVjdGVkU2NyaXB0Ll93aW5kb3coKTsKLSAgICBpZiAo aW5zcGVjdGVkV2luZG93Ll9pbnNwZWN0b3JDb21tYW5kTGluZUFQSSkKKyAgICBpZiAoZXZhbEZ1 bmN0aW9uLmNhbGwoZXZhbE9iamVjdCwgIndpbmRvdy5faW5zcGVjdG9yQ29tbWFuZExpbmVBUEki KSkKICAgICAgICAgcmV0dXJuOwotICAgIAotICAgIGluc3BlY3RlZFdpbmRvdy5ldmFsKCJ3aW5k b3cuX2luc3BlY3RvckNvbW1hbmRMaW5lQVBJID0geyBcCisgICAgdmFyIGluc3BlY3RvckNvbW1h bmRMaW5lQVBJID0gZXZhbEZ1bmN0aW9uLmNhbGwoZXZhbE9iamVjdCwgIndpbmRvdy5faW5zcGVj dG9yQ29tbWFuZExpbmVBUEkgPSB7IFwKICAgICAgICAgJDogZnVuY3Rpb24oKSB7IHJldHVybiBk b2N1bWVudC5nZXRFbGVtZW50QnlJZC5hcHBseShkb2N1bWVudCwgYXJndW1lbnRzKSB9LCBcCiAg ICAgICAgICQkOiBmdW5jdGlvbigpIHsgcmV0dXJuIGRvY3VtZW50LnF1ZXJ5U2VsZWN0b3JBbGwu YXBwbHkoZG9jdW1lbnQsIGFyZ3VtZW50cykgfSwgXAogICAgICAgICAkeDogZnVuY3Rpb24oeHBh dGgsIGNvbnRleHQpIHsgXApAQCAtOTEzLDggKzkxMSw4IEBAIEluamVjdGVkU2NyaXB0Ll9lbnN1 cmVDb21tYW5kTGluZUFQSUluc3RhbGxlZCA9IGZ1bmN0aW9uKGluc3BlY3RlZFdpbmRvdykKICAg ICAgICAgZ2V0ICQ0KCkgeyByZXR1cm4gX2luc3BlY3RvckNvbW1hbmRMaW5lQVBJLl9pbnNwZWN0 ZWROb2Rlc1s0XSB9IFwKICAgICB9OyIpOwogCi0gICAgaW5zcGVjdGVkV2luZG93Ll9pbnNwZWN0 b3JDb21tYW5kTGluZUFQSS5jbGVhciA9IEluc3BlY3RvckNvbnRyb2xsZXIud3JhcENhbGxiYWNr KEluamVjdGVkU2NyaXB0Ll9jbGVhckNvbnNvbGVNZXNzYWdlcyk7Ci0gICAgaW5zcGVjdGVkV2lu ZG93Ll9pbnNwZWN0b3JDb21tYW5kTGluZUFQSS5pbnNwZWN0ID0gSW5zcGVjdG9yQ29udHJvbGxl ci53cmFwQ2FsbGJhY2soSW5qZWN0ZWRTY3JpcHQuX2luc3BlY3RPYmplY3QpOworICAgIGluc3Bl Y3RvckNvbW1hbmRMaW5lQVBJLmNsZWFyID0gSW5zcGVjdG9yQ29udHJvbGxlci53cmFwQ2FsbGJh Y2soSW5qZWN0ZWRTY3JpcHQuX2NsZWFyQ29uc29sZU1lc3NhZ2VzKTsKKyAgICBpbnNwZWN0b3JD b21tYW5kTGluZUFQSS5pbnNwZWN0ID0gSW5zcGVjdG9yQ29udHJvbGxlci53cmFwQ2FsbGJhY2so SW5qZWN0ZWRTY3JpcHQuX2luc3BlY3RPYmplY3QpOwogfQogCiBJbmplY3RlZFNjcmlwdC5fcmVz b2x2ZU9iamVjdCA9IGZ1bmN0aW9uKG9iamVjdFByb3h5KQo=