91476 2012-07-17 02:00:06 -0700 [Qt] REGRESSION(r122768, r122771): They broke jquery/data.html and inspector/elements/edit-dom-actions.html 2012-07-17 16:11:51 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 79666 0 jbadics webkit-unassigned fpizlo jbadics kadam oliver szledan oldest_to_newest 671094 0 jbadics 2012-07-17 02:00:06 -0700 jquery/data.html and inspector/elements/edit-dom-actions.html fails from r122768 or r122771. We couldn't reproduce the fails on local but we suppose that these two tests fail from the same revision that must be http://trac.webkit.org/changeset/122768 or http://trac.webkit.org/changeset/122771 . 671096 1 szledan 2012-07-17 02:07:57 -0700 I skipped these tests. Please unskip them after the proper fix. http://trac.webkit.org/changeset/122827/trunk/LayoutTests/platform/qt/Skipped 671107 2 jbadics 2012-07-17 02:39:57 -0700 I'm sorry I forgot to mention the diffs; inspector/elements/edit-dom-actions.html: --- /ramdisk/qt-linux-release/build/layout-test-results/inspector/elements/edit-dom-actions-expected.txt +++ /ramdisk/qt-linux-release/build/layout-test-results/inspector/elements/edit-dom-actions-actual.txt @@ -95,10 +95,37 @@ </div> </div> <div id="node-to-edit-as-html"><span id="span">Text</span></div> -==== after ==== -- <div id="testEditAsHTML"> - - <span foo="bar"> - <span id="inner-span">Span contents</span> - </span> - </div> +Exception while running: function onAllNodesAvailable() { + WebInspector.panels.elements.treeOutline._updateModifiedNodes(); + expand(WebInspector.panels.elements.treeOutline); + callback(); + } +didChange@qrc:/webkit/inspector/treeoutline.js:728 +title@qrc:/webkit/inspector/treeoutline.js:643 +updateTitle@qrc:/webkit/inspector/ElementsTreeOutline.js:1601 +onattach@qrc:/webkit/inspector/ElementsTreeOutline.js:828 +_attach@qrc:/webkit/inspector/treeoutline.js:777 +insertChild@qrc:/webkit/inspector/treeoutline.js:124 +insertChildElement@qrc:/webkit/inspector/ElementsTreeOutline.js:868 +updateChildrenOfNode@qrc:/webkit/inspector/ElementsTreeOutline.js:927 +_updateChildren@qrc:/webkit/inspector/ElementsTreeOutline.js:957 +[native code] +getChildNodes@qrc:/webkit/inspector/DOMAgent.js:275 +updateChildren@qrc:/webkit/inspector/ElementsTreeOutline.js:858 +onpopulate@qrc:/webkit/inspector/ElementsTreeOutline.js:848 +expand@qrc:/webkit/inspector/treeoutline.js:901 +expand +expand +expand +expand +expand +onAllNodesAvailable +result +processChildren +processChildren@[native code] +mycallback@qrc:/webkit/inspector/DOMAgent.js:286 +mycallback@[native code] +dispatch@qrc:/webkit/inspector/InspectorBackend.js:186 +qrc:/webkit/inspector/inspector.js:608 +[native code] jquery/data.html: crash log for DumpRenderTree (pid 2592): STDOUT: <empty> STDERR: 1 0x806509a /ramdisk/qt-linux-release/build/WebKitBuild/Release/bin/DumpRenderTree() [0x806509a] STDERR: 2 0xf774a400 [0xf774a400] STDERR: 3 0xf704ec00 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(_ZNK3JSC6JSCell8toNumberEPNS_9ExecStateE+0x18) [0xf704ec00] STDERR: 4 0xf707ed68 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(_ZNK3JSC7JSValue16toNumberSlowCaseEPNS_9ExecStateE+0x5a) [0xf707ed68] STDERR: 5 0xf69d6991 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0xd24991) [0xf69d6991] STDERR: 6 0xf704c256 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x139a256) [0xf704c256] STDERR: 7 0xf6f81a8a /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x12cfa8a) [0xf6f81a8a] STDERR: 8 0xee856fa7 [0xee856fa7] STDERR: 9 0xf6f3e417 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x128c417) [0xf6f3e417] STDERR: 10 0xf6f38963 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x1286963) [0xf6f38963] STDERR: 11 0xf70201bc /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(_ZN3JSC4callEPNS_9ExecStateENS_7JSValueENS_8CallTypeERKNS_8CallDataES2_RKNS_7ArgListE+0x60) [0xf70201bc] STDERR: 12 0xf5f708a6 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x2be8a6) [0xf5f708a6] STDERR: 13 0xf5f71180 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x2bf180) [0xf5f71180] STDERR: 14 0xf645eb68 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x7acb68) [0xf645eb68] STDERR: 15 0xf6547c33 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x895c33) [0xf6547c33] STDERR: 16 0xf6547cef /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0x895cef) [0xf6547cef] STDERR: 17 0xf67367b1 /ramdisk/qt-linux-release/build/WebKitBuild/Release/lib/libQtWebKit.so.4(+0xa847b1) [0xf67367b1] STDERR: 18 0xf43864e4 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(_ZN7QObject5eventEP6QEvent+0x84) [0xf43864e4] STDERR: 19 0xf478fe0c /usr/local/Trolltech/Qt-4.8.0/lib/libQtGui.so.4(_ZN19QApplicationPrivate13notify_helperEP7QObjectP6QEvent+0xac) [0xf478fe0c] STDERR: 20 0xf4795cc2 /usr/local/Trolltech/Qt-4.8.0/lib/libQtGui.so.4(_ZN12QApplication6notifyEP7QObjectP6QEvent+0x162) [0xf4795cc2] STDERR: 21 0xf43731ab /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(_ZN16QCoreApplication14notifyInternalEP7QObjectP6QEvent+0x7b) [0xf43731ab] STDERR: 22 0xf43a9b87 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(+0x1c5b87) [0xf43a9b87] STDERR: 23 0xf43a6b6a /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(+0x1c2b6a) [0xf43a6b6a] STDERR: 24 0xf43a6ba4 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(+0x1c2ba4) [0xf43a6ba4] STDERR: 25 0xf5711305 /lib/libglib-2.0.so.0(g_main_context_dispatch+0x1d5) [0xf5711305] STDERR: 26 0xf5714fe8 /lib/libglib-2.0.so.0(+0x3efe8) [0xf5714fe8] STDERR: 27 0xf57151c8 /lib/libglib-2.0.so.0(g_main_context_iteration+0x68) [0xf57151c8] STDERR: 28 0xf43a6855 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(_ZN20QEventDispatcherGlib13processEventsE6QFlagsIN10QEventLoop17ProcessEventsFlagEE+0x65) [0xf43a6855] STDERR: 29 0xf484a775 /usr/local/Trolltech/Qt-4.8.0/lib/libQtGui.so.4(+0x229775) [0xf484a775] STDERR: 30 0xf4371a69 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(_ZN10QEventLoop13processEventsE6QFlagsINS_17ProcessEventsFlagEE+0x49) [0xf4371a69] STDERR: 31 0xf4371f72 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(_ZN10QEventLoop4execE6QFlagsINS_17ProcessEventsFlagEE+0x182) [0xf4371f72] 671135 3 jbadics 2012-07-17 03:43:33 -0700 It also made ~160 fails on WebKit2. Please have a look at it when it's possible; thank you. 671489 4 fpizlo 2012-07-17 12:15:23 -0700 Is this 32-bit or 64-bit? 671628 5 fpizlo 2012-07-17 15:05:05 -0700 The code added by this patch doesn't appear to come into play in jquery/data.html. 671651 6 fpizlo 2012-07-17 15:21:59 -0700 Oh, actually, I found the bug. Patch on the way shortly. 671664 7 152851 fpizlo 2012-07-17 15:34:19 -0700 Created attachment 152851 the patch 671670 8 152851 mhahnenberg 2012-07-17 15:37:57 -0700 Comment on attachment 152851 the patch r=me 671706 9 fpizlo 2012-07-17 16:11:51 -0700 Landed in http://trac.webkit.org/changeset/122888 152851 2012-07-17 15:34:19 -0700 2012-07-17 15:37:57 -0700 the patch fix32bit_patch_1.diff text/plain 3649 fpizlo SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTIyODgwKQorKysgU291cmNl L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIwIEBA CisyMDEyLTA3LTE3ICBGaWxpcCBQaXpsbyAgPGZwaXpsb0BhcHBsZS5jb20+CisKKyAgICAgICAg W1F0XSBSRUdSRVNTSU9OKHIxMjI3NjgsIHIxMjI3NzEpOiBUaGV5IGJyb2tlIGpxdWVyeS9kYXRh Lmh0bWwgYW5kIGluc3BlY3Rvci9lbGVtZW50cy9lZGl0LWRvbS1hY3Rpb25zLmh0bWwKKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTkxNDc2CisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhlIDMyLWJpdCByZXBh dGNoaW5nIGNvZGUgd2FzIG5vdCBjb3JyZWN0bHkgYWRhcHRlZCB0byB0aGUgbmV3IHdvcmxkIHdo ZXJlIHRoZXJlIG1heSBub3QgYWx3YXlzCisgICAgICAgIGJlIGFuIGF2YWlsYWJsZSBzY3JhdGNo IHJlZ2lzdGVyLiBGaXhlZCBpdCBieSBlbnN1cmluZyB0aGF0IHRoZSBzY3JhdGNoIHJlZ2lzdGVy IHdlIHNlbGVjdCBkb2VzCisgICAgICAgIG5vdCBvdmVybGFwIHdpdGggdGhlIHZhbHVlIHRhZy4K KworICAgICAgICAqIGRmZy9ERkdSZXBhdGNoLmNwcDoKKyAgICAgICAgKEpTQzo6REZHOjpnZW5l cmF0ZVByb3RvQ2hhaW5BY2Nlc3NTdHViKToKKyAgICAgICAgKEpTQzo6REZHOjp0cnlDYWNoZUdl dEJ5SUQpOgorICAgICAgICAoSlNDOjpERkc6OnRyeUJ1aWxkR2V0QnlJRExpc3QpOgorICAgICAg ICAoSlNDOjpERkc6OmVtaXRQdXRSZXBsYWNlU3R1Yik6CisKIDIwMTItMDctMTcgIEdhYm9yIFJh cGNzYW55aSAgPHJnYWJvckB3ZWJraXQub3JnPgogCiAgICAgICAgIFVucmV2aWV3ZWQgYnVpbGRm aXggZnJvbSBab2x0YW4gSGVyY3plZyBhZnRlciAxMjI3NjguCkluZGV4OiBTb3VyY2UvSmF2YVNj cmlwdENvcmUvZGZnL0RGR1JlcGF0Y2guY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9KYXZhU2Ny aXB0Q29yZS9kZmcvREZHUmVwYXRjaC5jcHAJKHJldmlzaW9uIDEyMjg3MykKKysrIFNvdXJjZS9K YXZhU2NyaXB0Q29yZS9kZmcvREZHUmVwYXRjaC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTE2Niw3 ICsxNjYsMTEgQEAgc3RhdGljIHZvaWQgZ2VuZXJhdGVQcm90b0NoYWluQWNjZXNzU3R1YgogICAg IGJvb2wgbmVlZFRvUmVzdG9yZVNjcmF0Y2ggPSBmYWxzZTsKICAgICAKICAgICBpZiAoc2NyYXRj aEdQUiA9PSBJbnZhbGlkR1BSUmVnKSB7CisjaWYgVVNFKEpTVkFMVUU2NCkKICAgICAgICAgc2Ny YXRjaEdQUiA9IFNwZWN1bGF0aXZlSklUOjpzZWxlY3RTY3JhdGNoR1BSKGJhc2VHUFIsIHJlc3Vs dEdQUik7CisjZWxzZQorICAgICAgICBzY3JhdGNoR1BSID0gU3BlY3VsYXRpdmVKSVQ6OnNlbGVj dFNjcmF0Y2hHUFIoYmFzZUdQUiwgcmVzdWx0R1BSLCByZXN1bHRUYWdHUFIpOworI2VuZGlmCiAg ICAgICAgIHN0dWJKaXQucHVzaChzY3JhdGNoR1BSKTsKICAgICAgICAgbmVlZFRvUmVzdG9yZVNj cmF0Y2ggPSB0cnVlOwogICAgIH0KQEAgLTIzOCw3ICsyNDIsMTEgQEAgc3RhdGljIGJvb2wgdHJ5 Q2FjaGVHZXRCeUlEKEV4ZWNTdGF0ZSogZQogICAgICAgICBNYWNyb0Fzc2VtYmxlciBzdHViSml0 OwogICAgICAgICAKICAgICAgICAgaWYgKHNjcmF0Y2hHUFIgPT0gSW52YWxpZEdQUlJlZykgewor I2lmIFVTRShKU1ZBTFVFNjQpCiAgICAgICAgICAgICBzY3JhdGNoR1BSID0gU3BlY3VsYXRpdmVK SVQ6OnNlbGVjdFNjcmF0Y2hHUFIoYmFzZUdQUiwgcmVzdWx0R1BSKTsKKyNlbHNlCisgICAgICAg ICAgICBzY3JhdGNoR1BSID0gU3BlY3VsYXRpdmVKSVQ6OnNlbGVjdFNjcmF0Y2hHUFIoYmFzZUdQ UiwgcmVzdWx0R1BSLCByZXN1bHRUYWdHUFIpOworI2VuZGlmCiAgICAgICAgICAgICBzdHViSml0 LnB1c2goc2NyYXRjaEdQUik7CiAgICAgICAgICAgICBuZWVkVG9SZXN0b3JlU2NyYXRjaCA9IHRy dWU7CiAgICAgICAgIH0KQEAgLTQwNSw2ICs0MTMsNyBAQCBzdGF0aWMgYm9vbCB0cnlCdWlsZEdl dEJ5SURMaXN0KEV4ZWNTdGF0CiAgICAgICAgIGlmIChzbG90LmNhY2hlZFByb3BlcnR5VHlwZSgp ID09IFByb3BlcnR5U2xvdDo6R2V0dGVyCiAgICAgICAgICAgICB8fCBzbG90LmNhY2hlZFByb3Bl cnR5VHlwZSgpID09IFByb3BlcnR5U2xvdDo6Q3VzdG9tKSB7CiAgICAgICAgICAgICBpZiAoc2xv dC5jYWNoZWRQcm9wZXJ0eVR5cGUoKSA9PSBQcm9wZXJ0eVNsb3Q6OkdldHRlcikgeworICAgICAg ICAgICAgICAgIEFTU0VSVChzY3JhdGNoR1BSICE9IEludmFsaWRHUFJSZWcpOwogICAgICAgICAg ICAgICAgIEFTU0VSVChiYXNlR1BSICE9IHNjcmF0Y2hHUFIpOwogICAgICAgICAgICAgICAgIGlm IChpc0lubGluZU9mZnNldChzbG90LmNhY2hlZE9mZnNldCgpKSkgewogI2lmIFVTRShKU1ZBTFVF NjQpCkBAIC02NDIsNyArNjUxLDExIEBAIHN0YXRpYyB2b2lkIGVtaXRQdXRSZXBsYWNlU3R1YigK ICAgICBNYWNyb0Fzc2VtYmxlciBzdHViSml0OwogICAgIAogICAgIGlmIChzY3JhdGNoR1BSID09 IEludmFsaWRHUFJSZWcgJiYgKHdyaXRlQmFycmllck5lZWRlZCB8fCBpc091dE9mTGluZU9mZnNl dChzbG90LmNhY2hlZE9mZnNldCgpKSkpIHsKKyNpZiBVU0UoSlNWQUxVRTY0KQogICAgICAgICBz Y3JhdGNoR1BSID0gU3BlY3VsYXRpdmVKSVQ6OnNlbGVjdFNjcmF0Y2hHUFIoYmFzZUdQUiwgdmFs dWVHUFIpOworI2Vsc2UKKyAgICAgICAgc2NyYXRjaEdQUiA9IFNwZWN1bGF0aXZlSklUOjpzZWxl Y3RTY3JhdGNoR1BSKGJhc2VHUFIsIHZhbHVlR1BSLCB2YWx1ZVRhZ0dQUik7CisjZW5kaWYKICAg ICAgICAgbmVlZFRvUmVzdG9yZVNjcmF0Y2ggPSB0cnVlOwogICAgICAgICBzdHViSml0LnB1c2go c2NyYXRjaEdQUik7CiAgICAgfQpAQCAtNjUzLDcgKzY2NiwxMSBAQCBzdGF0aWMgdm9pZCBlbWl0 UHV0UmVwbGFjZVN0dWIoCiAgICAgICAgIE1hY3JvQXNzZW1ibGVyOjpUcnVzdGVkSW1tUHRyKHN0 cnVjdHVyZSkpOwogICAgIAogI2lmIEVOQUJMRShHR0MpIHx8IEVOQUJMRShXUklURV9CQVJSSUVS X1BST0ZJTElORykKKyNpZiBVU0UoSlNWQUxVRTY0KQogICAgIHNjcmF0Y2hHUFIyID0gU3BlY3Vs YXRpdmVKSVQ6OnNlbGVjdFNjcmF0Y2hHUFIoYmFzZUdQUiwgdmFsdWVHUFIsIHNjcmF0Y2hHUFIp OworI2Vsc2UKKyAgICBzY3JhdGNoR1BSMiA9IFNwZWN1bGF0aXZlSklUOjpzZWxlY3RTY3JhdGNo R1BSKGJhc2VHUFIsIHZhbHVlR1BSLCB2YWx1ZVRhZ0dQUiwgc2NyYXRjaEdQUik7CisjZW5kaWYK ICAgICBzdHViSml0LnB1c2goc2NyYXRjaEdQUjIpOwogICAgIFNwZWN1bGF0aXZlSklUOjp3cml0 ZUJhcnJpZXIoc3R1YkppdCwgYmFzZUdQUiwgc2NyYXRjaEdQUiwgc2NyYXRjaEdQUjIsIFdyaXRl QmFycmllckZvclByb3BlcnR5QWNjZXNzKTsKICAgICBzdHViSml0LnBvcChzY3JhdGNoR1BSMik7 Cg==