15380 2007-10-05 13:44:45 -0700 XPath: id('foo') doesn't resolve correctly 2007-10-15 23:58:45 -0700 1 1 1 Unclassified WebKit XML 523.x (Safari 3) All All RESOLVED FIXED P2 Normal --- 1 dan ap ap oldest_to_newest 57870 0 dan 2007-10-05 13:44:45 -0700 Calling document.evaluate with "id('foo')" doesn't return an element, even when there's an HTML element with ID "foo" in the page. To repro, open the attached xpathbug.html in WebKit nightly, Firefox 2.0, and Opera 9.2. In Firefox and Opera, the test passes as the xpath resolves correctly; in WebKit (and Safari 3 Public Beta) the test fails as document.evaluate returns null. 57871 1 16552 dan 2007-10-05 13:45:20 -0700 Created attachment 16552 example repro 57902 2 ap 2007-10-05 23:30:46 -0700 And I thought I fixed this in bug 12485 :-/ 57920 3 ap 2007-10-06 01:22:49 -0700 To work around this problem, append a space to the parameter: "id('nested1 ')" works fine to find an element with id "nested1". > And I thought I fixed this in bug 12485 :-/ That was a completely different bug with similar symptoms. 57933 4 16565 ap 2007-10-06 03:21:57 -0700 Created attachment 16565 proposed fix 57934 5 16565 mitz 2007-10-06 03:32:49 -0700 Comment on attachment 16565 proposed fix r=me 57936 6 ap 2007-10-06 04:10:16 -0700 Committed revision 26081 (feature branch). 58314 7 dan 2007-10-11 11:46:44 -0700 This isn't yet fixed in the official webkit nightly builds (r26359), but I think that's expected, since the change is currently only in the feature-branch...? How/when would we expect to see this in the nightly build? [I won't even ask about when we'd expect to see this fixed in Safari ... ;-)] 58315 8 ap 2007-10-11 12:09:48 -0700 There are nightly builds of the feature branch - the link is hidden at the bottom of <http://nightly.webkit.org/>. Also, as it was recently announced on the developer list, the feature branch will soon move to trunk. 58605 9 dan 2007-10-15 15:08:17 -0700 This is fixed in WebKit trunk r26595. Is this bug likely to get fixed in Safari 3? Seed 310A15 is from September 26 and has this bug (which I filed on 10/5). 58638 10 ap 2007-10-15 23:58:45 -0700 (In reply to comment #9) > Is this bug likely to get fixed in Safari 3? Apple doesn't comment on future products, and we outside Apple do not have any information about their plans. 16552 2007-10-05 13:45:20 -0700 2007-10-05 13:45:20 -0700 example repro xpathbug.html text/html 1182 dan PGh0bWw+DQo8aGVhZD48dGl0bGU+U2FmYXJpL1dlYktpdCBYUGF0aCBidWc8L3RpdGxlPg0KPHNj cmlwdD4NCiAgICB2YXIgcmVzdWx0cyA9ICIiOw0KICAgIHZhciByZXN1bHQ7DQogICAgdmFyIHRl c3ROdW0gPSAxOw0KICAgIGZ1bmN0aW9uIGluaXQoKSB7DQogICAgICAgIHZhciB0ZXN0cyA9IFsN CiAgICAgICAgICAgICJpZCgnbmVzdGVkMScpIiwNCiAgICAgICAgICAgICJpZCgnbmVzdGVkMScp L2RpdlsxXSIsDQogICAgICAgICAgICAiaWQoJ25lc3RlZDEnKS8vZGl2WzFdIiwNCiAgICAgICAg ICAgICJpZCgnbmVzdGVkMScpL2RpdlsxXS9pbnB1dFsyXSIsDQogICAgICAgICAgICAiaWQoJ25l c3RlZDEnKS9kaXZbMV0vL2lucHV0WzJdIiwNCiAgICAgICAgICAgICJpZCgnbmVzdGVkMScpLy9k aXZbMV0vaW5wdXRbMl0iLA0KICAgICAgICAgICAgImlkKCduZXN0ZWQxJykvL2RpdlsxXS8vaW5w dXRbMl0iDQogICAgICAgIF07DQogICAgICAgIGZvciAodmFyIGkgPSAwOyBpIDwgdGVzdHMubGVu Z3RoOyBpKyspIHsNCiAgICAgICAgICAgIHJ1blhQYXRoKHRlc3RzW2ldKTsNCiAgICAgICAgfQ0K ICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgncmVzdWx0cycpLmlubmVySFRNTCA9IHJl c3VsdHM7DQogICAgfQ0KICAgIA0KICAgIGZ1bmN0aW9uIHJ1blhQYXRoKHhwYXRoKSB7DQogICAg ICAgIHZhciByZXN1bHQgPSBkb2N1bWVudC5ldmFsdWF0ZSh4cGF0aCwgZG9jdW1lbnQsIG51bGws IFhQYXRoUmVzdWx0LkFOWV9UWVBFLCBudWxsKS5pdGVyYXRlTmV4dCgpOw0KICAgICAgICByZXN1 bHRzICs9IHhwYXRoICsgIiAiICsgKHJlc3VsdCA/ICJQQVNTRUQiIDogIkZBSUxFRCIpICsgIjog IiArIHJlc3VsdCArICI8YnIgLz4iOw0KICAgIH0NCjwvc2NyaXB0Pg0KPC9oZWFkPg0KPGJvZHkg b25sb2FkPSJpbml0KCkiPg0KPGRpdiBpZD0ibmVzdGVkMSI+DQogICAgICAgIDxkaXYgaWQ9Im5l c3RlZDIiPg0KICAgICAgICAgICAgPGlucHV0IGlkPSJuZXN0ZWQzYSIgdHlwZT0iYnV0dG9uIiB2 YWx1ZT0ibmVzdGVkM2EiPg0KICAgICAgICAgICAgPGlucHV0IGlkPSJuZXN0ZWQzYiIgdHlwZT0i YnV0dG9uIiB2YWx1ZT0ibmVzdGVkM2IiPg0KICAgICAgICA8L2Rpdj4NCiAgICA8L2Rpdj4NCiAg ICA8ZGl2IGlkPSJyZXN1bHRzIj48L2Rpdj4NCjwvYm9keT48L2h0bWw+ 16565 2007-10-06 03:21:57 -0700 2007-10-06 03:32:49 -0700 proposed fix 15380r1_patch.txt text/plain 4937 ap SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAyNjA3OSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTUgQEAKKzIwMDctMTAtMDYgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEB3ZWJr aXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg IGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE1MzgwCisgICAgICAgIFhQ YXRoOiBpZCgnZm9vJykgZG9lc24ndCByZXNvbHZlIGNvcnJlY3RseQorCisgICAgICAgIFRlc3Q6 IGZhc3QveHBhdGgvaWQtc2ltcGxlLmh0bWwKKworICAgICAgICAqIHhtbC9YUGF0aEZ1bmN0aW9u cy5jcHA6CisgICAgICAgIChXZWJDb3JlOjpYUGF0aDo6RnVuSWQ6OmV2YWx1YXRlKTogRml4ZWQg YSBsb2dpYyBlcnJvciBpbiBpZCgpIHBhcmFtZXRlciBwYXJzaW5nLgorCiAyMDA3LTEwLTA2ICBB bGV4ZXkgUHJvc2t1cnlha292ICA8YXBAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBi eSBEYXJpbi4KSW5kZXg6IFdlYkNvcmUveG1sL1hQYXRoRnVuY3Rpb25zLmNwcAo9PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 Ci0tLSBXZWJDb3JlL3htbC9YUGF0aEZ1bmN0aW9ucy5jcHAJKHJldmlzaW9uIDI2MDc3KQorKysg V2ViQ29yZS94bWwvWFBhdGhGdW5jdGlvbnMuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yODksMTMg KzI4OSwxMyBAQCBWYWx1ZSBGdW5JZDo6ZXZhbHVhdGUoKSBjb25zdAogICAgICAgICB3aGlsZSAo c3RhcnRQb3MgPCBsZW5ndGggJiYgaXNXaGl0ZXNwYWNlKGlkTGlzdFtzdGFydFBvc10pKQogICAg ICAgICAgICAgKytzdGFydFBvczsKICAgICAgICAgCisgICAgICAgIGlmIChzdGFydFBvcyA9PSBs ZW5ndGgpCisgICAgICAgICAgICBicmVhazsKKwogICAgICAgICBzaXplX3QgZW5kUG9zID0gc3Rh cnRQb3M7CiAgICAgICAgIHdoaWxlIChlbmRQb3MgPCBsZW5ndGggJiYgIWlzV2hpdGVzcGFjZShp ZExpc3RbZW5kUG9zXSkpCiAgICAgICAgICAgICArK2VuZFBvczsKIAotICAgICAgICBpZiAoZW5k UG9zID09IGxlbmd0aCkKLSAgICAgICAgICAgIGJyZWFrOwotCiAgICAgICAgIC8vIElmIHRoZXJl IGFyZSBzZXZlcmFsIG5vZGVzIHdpdGggdGhlIHNhbWUgaWQsIGlkKCkgc2hvdWxkIHJldHVybiB0 aGUgZmlyc3Qgb25lLgogICAgICAgICAvLyBJbiBXZWJLaXQsIGdldEVsZW1lbnRCeUlkIGJlaGF2 ZXMgc28sIHRvbywgYWx0aG91Z2ggaXRzIGJlaGF2aW9yIGluIHRoaXMgY2FzZSBpcyBmb3JtYWxs eSB1bmRlZmluZWQuCiAgICAgICAgIE5vZGUqIG5vZGUgPSBjb250ZXh0RG9jdW1lbnQtPmdldEVs ZW1lbnRCeUlkKFN0cmluZygmaWRMaXN0W3N0YXJ0UG9zXSwgZW5kUG9zIC0gc3RhcnRQb3MpKTsK SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDI2MDc5KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMDctMTAtMDYgIEFsZXhleSBQcm9za3VyeWFr b3YgIDxhcEB3ZWJraXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp LgorCisgICAgICAgIGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE1Mzgw CisgICAgICAgIFhQYXRoOiBpZCgnZm9vJykgZG9lc24ndCByZXNvbHZlIGNvcnJlY3RseQorCisg ICAgICAgICogZmFzdC94cGF0aC9pZC1zaW1wbGUtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAg ICAgKiBmYXN0L3hwYXRoL2lkLXNpbXBsZS5odG1sOiBBZGRlZC4KKwogMjAwNy0xMC0wNiAgQWxl eGV5IFByb3NrdXJ5YWtvdiAgPGFwQHdlYmtpdC5vcmc+CiAKICAgICAgICAgKiBmYXN0L2RvbS94 bWxodHRwcmVxdWVzdC1nZXQtZXhwZWN0ZWQudHh0OiBVcGRhdGVkIGV2ZW50IHByb3BlcnRpZXMK SW5kZXg6IExheW91dFRlc3RzL2Zhc3QveHBhdGgvaWQtc2ltcGxlLWV4cGVjdGVkLnR4dAo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L3hwYXRoL2lkLXNpbXBsZS1leHBlY3RlZC50eHQJ KHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L3hwYXRoL2lkLXNpbXBsZS1leHBlY3Rl ZC50eHQJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTEgQEAKK1Rlc3QgZm9yIGJ1ZyAxNTM4MDog WFBhdGg6IGlkKCdmb28nKSBkb2Vzbid0IHJlc29sdmUgY29ycmVjdGx5LgorCisgIAoraWQoJ25l c3RlZDEnKSBQQVNTRUQ6IFtvYmplY3QgSFRNTERpdkVsZW1lbnRdCitpZCgnbmVzdGVkMScpL2Rp dlsxXSBQQVNTRUQ6IFtvYmplY3QgSFRNTERpdkVsZW1lbnRdCitpZCgnbmVzdGVkMScpLy9kaXZb MV0gUEFTU0VEOiBbb2JqZWN0IEhUTUxEaXZFbGVtZW50XQoraWQoJ25lc3RlZDEnKS9kaXZbMV0v aW5wdXRbMl0gUEFTU0VEOiBbb2JqZWN0IEhUTUxJbnB1dEVsZW1lbnRdCitpZCgnbmVzdGVkMScp L2RpdlsxXS8vaW5wdXRbMl0gUEFTU0VEOiBbb2JqZWN0IEhUTUxJbnB1dEVsZW1lbnRdCitpZCgn bmVzdGVkMScpLy9kaXZbMV0vaW5wdXRbMl0gUEFTU0VEOiBbb2JqZWN0IEhUTUxJbnB1dEVsZW1l bnRdCitpZCgnbmVzdGVkMScpLy9kaXZbMV0vL2lucHV0WzJdIFBBU1NFRDogW29iamVjdCBIVE1M SW5wdXRFbGVtZW50XQorCgpQcm9wZXJ0eSBjaGFuZ2VzIG9uOiBMYXlvdXRUZXN0cy9mYXN0L3hw YXRoL2lkLXNpbXBsZS1leHBlY3RlZC50eHQKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpOYW1lOiBzdm46bWltZS10eXBl CiAgICsgdGV4dC9wbGFpbgpOYW1lOiBzdm46ZW9sLXN0eWxlCiAgICsgbmF0aXZlCgpJbmRleDog TGF5b3V0VGVzdHMvZmFzdC94cGF0aC9pZC1zaW1wbGUuaHRtbAo9PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlv dXRUZXN0cy9mYXN0L3hwYXRoL2lkLXNpbXBsZS5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0 VGVzdHMvZmFzdC94cGF0aC9pZC1zaW1wbGUuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSw0 MyBAQAorPGh0bWw+DQorPGhlYWQ+DQorPHNjcmlwdD4NCisgICAgaWYgKHdpbmRvdy5sYXlvdXRU ZXN0Q29udHJvbGxlcikNCisgICAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7 DQorDQorICAgIHZhciByZXN1bHRzID0gIiI7DQorICAgIHZhciByZXN1bHQ7DQorICAgIHZhciB0 ZXN0TnVtID0gMTsNCisgICAgZnVuY3Rpb24gaW5pdCgpIHsNCisgICAgICAgIHZhciB0ZXN0cyA9 IFsNCisgICAgICAgICAgICAiaWQoJ25lc3RlZDEnKSIsDQorICAgICAgICAgICAgImlkKCduZXN0 ZWQxJykvZGl2WzFdIiwNCisgICAgICAgICAgICAiaWQoJ25lc3RlZDEnKS8vZGl2WzFdIiwNCisg ICAgICAgICAgICAiaWQoJ25lc3RlZDEnKS9kaXZbMV0vaW5wdXRbMl0iLA0KKyAgICAgICAgICAg ICJpZCgnbmVzdGVkMScpL2RpdlsxXS8vaW5wdXRbMl0iLA0KKyAgICAgICAgICAgICJpZCgnbmVz dGVkMScpLy9kaXZbMV0vaW5wdXRbMl0iLA0KKyAgICAgICAgICAgICJpZCgnbmVzdGVkMScpLy9k aXZbMV0vL2lucHV0WzJdIg0KKyAgICAgICAgXTsNCisgICAgICAgIGZvciAodmFyIGkgPSAwOyBp IDwgdGVzdHMubGVuZ3RoOyBpKyspIHsNCisgICAgICAgICAgICBydW5YUGF0aCh0ZXN0c1tpXSk7 DQorICAgICAgICB9DQorICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgncmVzdWx0cycp LmlubmVySFRNTCA9IHJlc3VsdHM7DQorICAgIH0NCisgICAgDQorICAgIGZ1bmN0aW9uIHJ1blhQ YXRoKHhwYXRoKSB7DQorICAgICAgICB2YXIgcmVzdWx0ID0gZG9jdW1lbnQuZXZhbHVhdGUoeHBh dGgsIGRvY3VtZW50LCBudWxsLCBYUGF0aFJlc3VsdC5BTllfVFlQRSwgbnVsbCkuaXRlcmF0ZU5l eHQoKTsNCisgICAgICAgIHJlc3VsdHMgKz0geHBhdGggKyAiICIgKyAocmVzdWx0ID8gIlBBU1NF RCIgOiAiRkFJTEVEIikgKyAiOiAiICsgcmVzdWx0ICsgIjxiciAvPiI7DQorICAgIH0NCis8L3Nj cmlwdD4NCis8L2hlYWQ+DQorPGJvZHkgb25sb2FkPSJpbml0KCkiPg0KKzxwPlRlc3QgZm9yIDxh IGhyZWY9Imh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE1MzgwIj5idWcg MTUzODA8L2E+Og0KK1hQYXRoOiBpZCgnZm9vJykgZG9lc24ndCByZXNvbHZlIGNvcnJlY3RseS48 L3A+DQorPGRpdiBpZD0ibmVzdGVkMSI+DQorICAgICAgICA8ZGl2IGlkPSJuZXN0ZWQyIj4NCisg ICAgICAgICAgICA8aW5wdXQgaWQ9Im5lc3RlZDNhIiB0eXBlPSJidXR0b24iIHZhbHVlPSJuZXN0 ZWQzYSI+DQorICAgICAgICAgICAgPGlucHV0IGlkPSJuZXN0ZWQzYiIgdHlwZT0iYnV0dG9uIiB2 YWx1ZT0ibmVzdGVkM2IiPg0KKyAgICAgICAgPC9kaXY+DQorICAgIDwvZGl2Pg0KKyAgICA8ZGl2 IGlkPSJyZXN1bHRzIj48L2Rpdj4NCis8L2JvZHk+DQorPC9odG1sPg0KClByb3BlcnR5IGNoYW5n ZXMgb246IExheW91dFRlc3RzL2Zhc3QveHBhdGgvaWQtc2ltcGxlLmh0bWwKX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpO YW1lOiBzdm46bWltZS10eXBlCiAgICsgdGV4dC9odG1sCgo=