54858 2011-02-21 01:25:31 -0800 Web Inspector: elementsPanel are flaky, at startup it selects <html> or <body> element depends on timing 2013-06-03 07:56:51 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED WORKSFORME P2 Normal --- 116924 1 loislo loislo apavlov bweinstein joepeck keishi loislo pfeldman pmuellr rik rtakacs yurys oldest_to_newest 354454 0 loislo 2011-02-21 01:25:31 -0800 the root of problem is setDocument call. Sometimes we have additional setDocument call forced by domContentLoaded event because inspector window was opened just before inspecting page. The first one was at didCommitLoad but with almost empty DOM tree. In that case elementsPanel automatically selects documentRoot element at the first call because there are no other selectable elements in DOM tree. At the second call it selects the documentRoot again because it was selected before. 354456 1 83137 loislo 2011-02-21 01:30:46 -0800 Created attachment 83137 [patch] initial version 354484 2 83137 pfeldman 2011-02-21 02:29:26 -0800 Comment on attachment 83137 [patch] initial version View in context: https://bugs.webkit.org/attachment.cgi?id=83137&action=review > Source/WebCore/ChangeLog:5 > + Web Inspector: elementsPanel are flaky, at startup it selects <html> or <body> element depends on timing Elements panel is ... depending on the timing. > Source/WebCore/inspector/front-end/ElementsPanel.js:162 > + if (this.focusedDOMNode && !this._focusedNodeWasAutomaticallySelected) We should only restore selection upon domContentLoaded event instead. 894771 3 rtakacs 2013-05-29 01:32:17 -0700 It works fine for me (at statup the body element is the one that's selected). I will close the bug if noone minds. 83137 2011-02-21 01:30:46 -0800 2011-02-21 02:29:26 -0800 [patch] initial version patch text/plain 2961 loislo ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCA3MjIwNTM0Li5lZWI0YWZmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjMg QEAKKzIwMTEtMDItMjEgIElseWEgVGlraG9ub3Zza3kgIDxsb2lzbG9AY2hyb21pdW0ub3JnPgor CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFdlYiBJbnNw ZWN0b3I6IGVsZW1lbnRzUGFuZWwgYXJlIGZsYWt5LCBhdCBzdGFydHVwIGl0IHNlbGVjdHMgPGh0 bWw+IG9yIDxib2R5PiBlbGVtZW50IGRlcGVuZHMgb24gdGltaW5nCisgICAgICAgIGh0dHBzOi8v YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD01NDg1OAorCisgICAgICAgIFRoZSByb290 IG9mIHByb2JsZW0gaXMgc2V0RG9jdW1lbnQgY2FsbC4KKyAgICAgICAgU29tZXRpbWVzIHdlIGhh dmUgYWRkaXRpb25hbCBzZXREb2N1bWVudCBjYWxsIGZvcmNlZCBieSBkb21Db250ZW50TG9hZGVk CisgICAgICAgIGV2ZW50IGJlY2F1c2UgaW5zcGVjdG9yIHdpbmRvdyB3YXMgb3BlbmVkIGp1c3Qg YmVmb3JlIGluc3BlY3RpbmcgcGFnZS4KKyAgICAgICAgVGhlIGZpcnN0IG9uZSB3YXMgYXQgZGlk Q29tbWl0TG9hZCBidXQgd2l0aCBhbG1vc3QgZW1wdHkgRE9NIHRyZWUuCisgICAgICAgIEluIHRo YXQgY2FzZSBlbGVtZW50c1BhbmVsIGF1dG9tYXRpY2FsbHkgc2VsZWN0cyBkb2N1bWVudFJvb3Qg ZWxlbWVudCBhdCB0aGUgZmlyc3QgY2FsbAorICAgICAgICBiZWNhdXNlIHRoZXJlIGFyZSBubyBv dGhlciBzZWxlY3RhYmxlIGVsZW1lbnRzIGluIERPTSB0cmVlLgorICAgICAgICBBdCB0aGUgc2Vj b25kIGNhbGwgaXQgc2VsZWN0cyB0aGUgZG9jdW1lbnRSb290IGFnYWluIGJlY2F1c2UgaXQgd2Fz IHNlbGVjdGVkIGJlZm9yZS4KKworICAgICAgICAqIGluc3BlY3Rvci9mcm9udC1lbmQvRWxlbWVu dHNQYW5lbC5qczoKKyAgICAgICAgKFdlYkluc3BlY3Rvci5FbGVtZW50c1BhbmVsLnByb3RvdHlw ZS5yZXNldCk6CisgICAgICAgIChXZWJJbnNwZWN0b3IuRWxlbWVudHNQYW5lbC5wcm90b3R5cGUu c2V0RG9jdW1lbnQuc2VsZWN0Tm9kZSk6CisgICAgICAgIChXZWJJbnNwZWN0b3IuRWxlbWVudHNQ YW5lbC5wcm90b3R5cGUuc2V0IGZvY3VzZWRET01Ob2RlKToKKwogMjAxMS0wMi0yMCAgR2F2aW4g QmFycmFjbG91Z2ggIDxiYXJyYWNsb3VnaEBhcHBsZS5jb20+CiAKICAgICAgICAgQnVpbGQgZml4 IChyZW1vdmUgaW5jbHVkZXMpLgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9y L2Zyb250LWVuZC9FbGVtZW50c1BhbmVsLmpzIGIvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zy b250LWVuZC9FbGVtZW50c1BhbmVsLmpzCmluZGV4IDM2ZGI4MWUuLmYwMjhmMzEgMTAwNjQ0Ci0t LSBhL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvRWxlbWVudHNQYW5lbC5qcwor KysgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL0VsZW1lbnRzUGFuZWwuanMK QEAgLTE1OSw3ICsxNTksNyBAQCBXZWJJbnNwZWN0b3IuRWxlbWVudHNQYW5lbC5wcm90b3R5cGUg PSB7CiAKICAgICByZXNldDogZnVuY3Rpb24oKQogICAgIHsKLSAgICAgICAgaWYgKHRoaXMuZm9j dXNlZERPTU5vZGUpCisgICAgICAgIGlmICh0aGlzLmZvY3VzZWRET01Ob2RlICYmICF0aGlzLl9m b2N1c2VkTm9kZVdhc0F1dG9tYXRpY2FsbHlTZWxlY3RlZCkKICAgICAgICAgICAgIHRoaXMuX3Nl bGVjdGVkUGF0aE9uUmVzZXQgPSB0aGlzLmZvY3VzZWRET01Ob2RlLnBhdGgoKTsKIAogICAgICAg ICB0aGlzLnJvb3RET01Ob2RlID0gbnVsbDsKQEAgLTE4OSwxMyArMTg5LDEzIEBAIFdlYkluc3Bl Y3Rvci5FbGVtZW50c1BhbmVsLnByb3RvdHlwZSA9IHsKIAogICAgICAgICBmdW5jdGlvbiBzZWxl Y3ROb2RlKGNhbmRpZGF0ZUZvY3VzTm9kZSkKICAgICAgICAgewotICAgICAgICAgICAgaWYgKCFj YW5kaWRhdGVGb2N1c05vZGUpCi0gICAgICAgICAgICAgICAgY2FuZGlkYXRlRm9jdXNOb2RlID0g aW5zcGVjdGVkUm9vdERvY3VtZW50LmJvZHkgfHwgaW5zcGVjdGVkUm9vdERvY3VtZW50LmRvY3Vt ZW50RWxlbWVudDsKKyAgICAgICAgICAgIHZhciBub2RlVG9Gb2N1cyA9IGNhbmRpZGF0ZUZvY3Vz Tm9kZSB8fCBpbnNwZWN0ZWRSb290RG9jdW1lbnQuYm9keSB8fCBpbnNwZWN0ZWRSb290RG9jdW1l bnQuZG9jdW1lbnRFbGVtZW50OwogCi0gICAgICAgICAgICBpZiAoIWNhbmRpZGF0ZUZvY3VzTm9k ZSkKKyAgICAgICAgICAgIGlmICghbm9kZVRvRm9jdXMpCiAgICAgICAgICAgICAgICAgcmV0dXJu OwogCi0gICAgICAgICAgICB0aGlzLmZvY3VzZWRET01Ob2RlID0gY2FuZGlkYXRlRm9jdXNOb2Rl OworICAgICAgICAgICAgdGhpcy5mb2N1c2VkRE9NTm9kZSA9IG5vZGVUb0ZvY3VzOworICAgICAg ICAgICAgdGhpcy5fZm9jdXNlZE5vZGVXYXNBdXRvbWF0aWNhbGx5U2VsZWN0ZWQgPSAhY2FuZGlk YXRlRm9jdXNOb2RlOwogICAgICAgICAgICAgaWYgKHRoaXMudHJlZU91dGxpbmUuc2VsZWN0ZWRU cmVlRWxlbWVudCkKICAgICAgICAgICAgICAgICB0aGlzLnRyZWVPdXRsaW5lLnNlbGVjdGVkVHJl ZUVsZW1lbnQuZXhwYW5kKCk7CiAgICAgICAgIH0KQEAgLTM2OCw2ICszNjgsNyBAQCBXZWJJbnNw ZWN0b3IuRWxlbWVudHNQYW5lbC5wcm90b3R5cGUgPSB7CiAgICAgc2V0IGZvY3VzZWRET01Ob2Rl KHgpCiAgICAgewogICAgICAgICB0aGlzLnRyZWVPdXRsaW5lLmZvY3VzZWRET01Ob2RlID0geDsK KyAgICAgICAgdGhpcy5fZm9jdXNlZE5vZGVXYXNBdXRvbWF0aWNhbGx5U2VsZWN0ZWQgPSBmYWxz ZTsKICAgICB9LAogCiAgICAgX2F0dHJpYnV0ZXNVcGRhdGVkOiBmdW5jdGlvbihldmVudCkK